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

"ERR invalid password" when creating a RedisURI with a username #131

Closed
jsiebens opened this issue Sep 8, 2015 · 3 comments
Closed

"ERR invalid password" when creating a RedisURI with a username #131

jsiebens opened this issue Sep 8, 2015 · 3 comments
Assignees
Labels
type: bug A general bug
Milestone

Comments

@jsiebens
Copy link

jsiebens commented Sep 8, 2015

When deploying an application to Heroku with the Heroku Redis addon, a Redis URI with a username is provided. Eg: redis://h:@<...>.compute-1.amazonaws.com:14589
This results in an "ERR invalid password" error because the username is not correctly stripped from the password.

@mp911de mp911de self-assigned this Sep 8, 2015
@mp911de mp911de added the type: bug A general bug label Sep 8, 2015
@mp911de
Copy link
Collaborator

mp911de commented Sep 8, 2015

Please not that Redis authentication is based only on passwords. Passing passwords within URL's is supported but it misses somehow the point since the URI UserInfo syntax is USER:PASSWORD. Passwords with special chars such as : and @ cannot be used because they conflict to the URI syntax.

@mp911de
Copy link
Collaborator

mp911de commented Sep 8, 2015

Fixed in 3.4 and 4.0

@mp911de mp911de added this to the Lettuce 3.3 milestone Sep 10, 2015
@mp911de
Copy link
Collaborator

mp911de commented Sep 10, 2015

Fixed also in 3.3

@mp911de mp911de closed this as completed Sep 10, 2015
@mp911de mp911de modified the milestones: 3.3.1.Final, Lettuce 3.3 Sep 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

No branches or pull requests

2 participants