Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Protocol error: invalid multibulk length. #299

Closed
Trial97 opened this issue Oct 11, 2021 · 2 comments
Closed

Protocol error: invalid multibulk length. #299

Trial97 opened this issue Oct 11, 2021 · 2 comments

Comments

@Trial97
Copy link

Trial97 commented Oct 11, 2021

Hi @mediocregopher,

First, let me thank you for this awesome library.
I encountered the following error when I tried to use HMSet: -ERR Protocol error: invalid multibulk length.
After some investigation regarding this, I found out that it was because of the large number of arguments I used in that command(StackExchange/StackExchange.Redis#201 (comment)).
I am confused on whether we should handle the error in our code or maybe better in your library?

Thanks,
Trial97

@mediocregopher
Copy link
Owner

@Trial97 In my opinion that error should be handled in your code. There's not really anything for radix to do about that except forward the error along, is there?

@Trial97
Copy link
Author

Trial97 commented Oct 14, 2021

Hi @mediocregopher,

Thanks for your quick reply.
I followed your advice and fixed it in my code: https://github.com/cgrates/cgrates/blob/3e2ff64ee1f6b52ad664a7cf5be526c3415186c3/engine/storage_redis.go#L1202-L1235

Thanks,
Trial97

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants