-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Support for CLIENT SETINFO #2659
Support for CLIENT SETINFO #2659
Conversation
SoulPancake
commented
Jul 23, 2023
- Fixes Clients should identify themselves on connect #2654
@ofekshenawa thought you should take a look given what we'd discussed. |
@SoulPancake LGTM! |
redis.doc: As described in the Redis documentation, it represents the state of a network connection rather than just a simple command. |
We should invoke this method during connection initialization and ignore failures (for versions < 7.2) link: #2654 (comment) |