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

[bug] Redis error: Lua redis lib command arguments must be strings or integers #732

Closed
VanderY opened this issue Oct 30, 2023 · 8 comments · Fixed by centrifugal/centrifuge#328

Comments

@VanderY
Copy link

VanderY commented Oct 30, 2023

I'm constantly getting this error. This happens without any actions from me so I can't give steps to reproduce.

Message
Error calling history
Error

Lua redis lib command arguments must be strings or integers script: c9735f3dd5ac99ee9c4e59d9f5ac297b036f9443, on @user_script:25.

Versions

Centrifugo version is 5.1
Redis version is 7.2

Running on docker

@FZambia
Copy link
Member

FZambia commented Oct 30, 2023

Hello @VanderY ,

thanks - reproduced and prepared the fix in centrifugal/centrifuge#328

@FZambia
Copy link
Member

FZambia commented Oct 31, 2023

The fix is a part of v5.1.1

@FZambia FZambia closed this as completed Oct 31, 2023
@VanderY
Copy link
Author

VanderY commented Nov 2, 2023

Hello again @FZambia,

Now it's the same error but in a different place

Lua redis lib command arguments must be strings or integers script: 9f5e715bc195d5830e60915df951da8f4b6fe940, on @user_script:34.

@FZambia FZambia reopened this Nov 2, 2023
@FZambia
Copy link
Member

FZambia commented Nov 2, 2023

Unfortunate..

The only reason I see now how this could happen is inconsistent state of meta key in Redis. This is generally not possible - but maybe I am missing sth. But unfortunately to precisely look at it we need to know the channel which is now not printed to the error log.

As a first step I'll add channel logging for the next release.

Probably you can write to me in Telegram? Find me in our Telegram community - https://centrifugal.dev/docs/getting-started/community - and we can somehow speed up the investigation of this.

@FZambia FZambia changed the title [bug] Redis error [bug] Redis error: Lua redis lib command arguments must be strings or integers Nov 2, 2023
@FZambia
Copy link
Member

FZambia commented Nov 2, 2023

Realized why this can happen - this is one more possible scenario, when meta key has only epoch field. Centrifugo v5.1.1 does not have this issue anymore for new installations or new Redis keys, but old keys result into this error.

@VanderY
Copy link
Author

VanderY commented Nov 3, 2023

Actually, this is really what happened.
I just removed all redis data, and it seems to be working now.

Thank you so much for your support!

@FZambia
Copy link
Member

FZambia commented Nov 5, 2023

Thanks! I also made some changes for next Centrifugo version to work correctly with both new and old keys

@FZambia
Copy link
Member

FZambia commented Nov 25, 2023

Hope should be now fine for all cases with v5.1.2

@FZambia FZambia closed this as completed Nov 25, 2023
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

Successfully merging a pull request may close this issue.

2 participants