-
Notifications
You must be signed in to change notification settings - Fork 1.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
Finagle Redis client auth command fails in 6.29.0 #422
Comments
Hey, sorry about the regression. Can you try the following and let me know if it fixes your issue:
|
Hi @roanta, Thanks for helping. I implemented the change you suggested, and my application works as expected. LMK if there's anything else I can do to help debug this issue. Cheers. |
Thanks! A silly mistake on our part. I'll push a patch through and it should be fixed in the next release. |
Great! Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have a working finagle-redis 6.28.0 client application that uses the redis
AUTH
command. When I upgraded to 6.29.0, the only change I made was to change this line of code:to
(I presume this is the correct port. I cannot find user documentation on the new client API in 6.29.0.)
Immediately following, I call
redisClient.auth(password)
. In 6.29.0, it barfs:The text was updated successfully, but these errors were encountered: