Skip to content

Adds user parsing to redis-cli URIs#8048

Merged
itamarhaber merged 1 commit intoredis:unstablefrom
itamarhaber:fix-8047
Nov 12, 2020
Merged

Adds user parsing to redis-cli URIs#8048
itamarhaber merged 1 commit intoredis:unstablefrom
itamarhaber:fix-8047

Conversation

@itamarhaber
Copy link
Member

Until v6's ACL, users weren't generally supported and specifically ignored when parsing the URI. This fixes this behavior.

Ref: #8047

@itamarhaber itamarhaber requested a review from oranagra November 12, 2020 16:57
@itamarhaber
Copy link
Member Author

Also updated #4507

@itamarhaber itamarhaber merged commit 92ec592 into redis:unstable Nov 12, 2020
JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Nov 19, 2020
@oranagra oranagra mentioned this pull request Jan 13, 2021
oranagra pushed a commit that referenced this pull request Nov 19, 2023
Add documentation of the URI format in the `--help` output of
`redis-cli` and `redis-benchmark`.

In particular, it's good for users to know that they need to specify
"default" as the username when authenticating without a username. Other
details of the URI format are described too, like scheme and dbnum.

It used to be possible to connect to Redis using an URL with an empty
username, like `redis-cli -u redis://:PASSWORD@localhost:6379/0`. This
was broken in 6.2 (#8048), and there was a discussion about it #9186.
Now, users need to specify "default" as the username and it's better to
document it.

Refer to #12746 for more details.
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 this pull request may close these issues.

2 participants