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.

Sign up for GitHub

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

Jump to bottom

V10: DisableIndentity should be DisableIdentity #2924

Open
chayim opened this issue Feb 20, 2024 · 0 comments · May be fixed by #2923
Open

V10: DisableIndentity should be DisableIdentity #2924

chayim opened this issue Feb 20, 2024 · 0 comments · May be fixed by #2923
Labels
breakingchange v10
Milestone
connection-improv...

Comments

@chayim
Copy link
Contributor

chayim commented Feb 20, 2024

As part of the SETINFO changes in #2915, we noticed that the option is set as DisableIndentity rather than DisableIdentity. In order to ensure fewer breaking changes, this option should be renamed, ideally in V10.
@chayim chayim added this to the connection-improvements milestone Feb 20, 2024
@ofekshenawa ofekshenawa linked a pull request Feb 20, 2024 that will close this issue
Fix DisableIdentity and IdentitySuffix option forwarding" #2923
Open
@ofekshenawa ofekshenawa changed the title V10: DisableIndentity shoudl be DisableIdentity V10: DisableIndentity should be DisableIdentity Mar 4, 2024
@sk- sk- mentioned this issue May 6, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breakingchange v10
Projects
None yet
Milestone
connection-improvements
Development

Successfully merging a pull request may close this issue.

Fix DisableIdentity and IdentitySuffix option forwarding" reenjii/go-redis
1 participant
@chayim