starred/asynq

Fork 0

mirror of https://github.com/hibiken/asynq.git synced 2026-04-26 15:35:55 +03:00

[GH-ISSUE #450] [FEATURE REQUEST] choose EVAL for `func (r *RDB) runScript` #205

New issue

Closed

opened 2026-03-02 05:19:37 +03:00 by kerem · 5 comments

kerem commented

2026-03-02 05:19:37 +03:00

Owner

Originally created by @chenlujjj on GitHub (May 6, 2022).
Original GitHub issue: https://github.com/hibiken/asynq/issues/450

Originally assigned to: @hibiken on GitHub.

Is your feature request related to a problem? Please describe.
I am using this awesome lib in my work, however when I released my application to production, I found that the evalsha is forbidden in my company.

Describe the solution you'd like
Maybe we can choose to use eval or evalsha in func (r *RDB) runScript and func (r *RDB) runScriptWithErrorCode.
It can be a bool parameter in NewClient function or something.

Describe alternatives you've considered

Additional context

Originally created by @chenlujjj on GitHub (May 6, 2022). Original GitHub issue: https://github.com/hibiken/asynq/issues/450 Originally assigned to: @hibiken on GitHub. **Is your feature request related to a problem? Please describe.** I am using this awesome lib in my work, however when I released my application to production, I found that the evalsha is forbidden in my company. **Describe the solution you'd like** Maybe we can choose to use `eval` or `evalsha` in `func (r *RDB) runScript` and `func (r *RDB) runScriptWithErrorCode`. It can be a bool parameter in `NewClient` function or something. **Describe alternatives you've considered** **Additional context**

kerem

2026-03-02 05:19:37 +03:00

closed this issue
added the
enhancement
label

kerem commented

2026-03-02 05:19:38 +03:00

Author

Owner

@chenlujjj commented on GitHub (May 6, 2022):

It turns out that some script calls func (r *RDB) runScript or func (r *RDB) runScriptWithErrorCode mentioned above, others call func (s *Script) Run directly, such as res, err := dequeueCmd.Run(context.Background(), r.client, keys, argv...).Result() in rdb.go.

@chenlujjj commented on GitHub (May 6, 2022): It turns out that some script calls `func (r *RDB) runScript` or `func (r *RDB) runScriptWithErrorCode` mentioned above, others call `func (s *Script) Run` directly, such as `res, err := dequeueCmd.Run(context.Background(), r.client, keys, argv...).Result()` in `rdb.go`.

kerem commented

2026-03-02 05:19:38 +03:00

Author

Owner

@chenlujjj commented on GitHub (May 6, 2022):

So there woule be more work than just modifying func (r *RDB) runScript and func (r *RDB) runScriptWithErrorCode

@chenlujjj commented on GitHub (May 6, 2022): So there woule be more work than just modifying `func (r *RDB) runScript` and `func (r *RDB) runScriptWithErrorCode`

kerem commented

2026-03-02 05:19:38 +03:00

Author

Owner

@chenlujjj commented on GitHub (May 6, 2022):

I have fored a repo: https://github.com/chenlujjj/asynq/tree/eval-script, and replaced all Run to Eval. It's just a quick and dirty update for my usage and should not be merged into main branch.

@chenlujjj commented on GitHub (May 6, 2022): I have fored a repo: https://github.com/chenlujjj/asynq/tree/eval-script, and replaced all `Run` to `Eval`. It's just a quick and dirty update for my usage and should not be merged into main branch.

kerem commented

2026-03-02 05:19:38 +03:00

Author

Owner

@hibiken commented on GitHub (May 10, 2022):

@chenlujjj thank you for opening an issue!
Do you think you can keep using your fork?

@hibiken commented on GitHub (May 10, 2022): @chenlujjj thank you for opening an issue! Do you think you can keep using your fork?

kerem commented

2026-03-02 05:19:38 +03:00

Author

Owner

@chenlujjj commented on GitHub (May 10, 2022):

@chenlujjj thank you for opening an issue! Do you think you can keep using your fork?

yes, it works now.

@chenlujjj commented on GitHub (May 10, 2022): > @chenlujjj thank you for opening an issue! Do you think you can keep using your fork? yes, it works now.

kerem referenced this issue

2026-03-02 05:54:29 +03:00

[PR #205] [MERGED] Add tls support to command line tool #664

kerem referenced this issue

2026-03-07 22:11:52 +03:00

[PR #205] [MERGED] Add tls support to command line tool #1673

kerem referenced this issue

2026-03-15 20:57:43 +03:00

[PR #205] [MERGED] Add tls support to command line tool #2681