Skip to content

Redis Sentinel : add password field for DSN to documentation #16310

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
Closed
aricci95 opened this issue Dec 22, 2021 · 1 comment
Closed

Redis Sentinel : add password field for DSN to documentation #16310

aricci95 opened this issue Dec 22, 2021 · 1 comment
Labels
Cache hasPR A Pull Request has already been submitted for this issue.

Comments

@aricci95
Copy link

Hi,

It took me 2 days of hard research to figure out how to add the password for the Redis Sentinel DSN, please add this information.

Correct Sentinel DSN with a password should look like this :
'redis:REDIS_PASSWORD@?host[REDIS_HOST1:REDIS_PORT1]&host[REDIS_HOST2:REDIS_PORT2]&redis_sentinel=SENTINEL_NAME&dbindex=DB_INDEX_NUMBER'

Regards,

Antoine.
@ToshY
Copy link
Contributor

ToshY commented Aug 26, 2022

I will make a PR for this, as this caused me so much pain when trying to figure it out (in this discussion).

@ToshY ToshY mentioned this issue Aug 26, 2022
Merged
@xabbuh xabbuh added hasPR A Pull Request has already been submitted for this issue. Cache labels Aug 27, 2022
javiereguiluz added a commit that referenced this issue Sep 14, 2022
@javiereguiluz
minor #17234 [Cache] fixes #16310 unclear alternate Redis DSN format …
7e1f09f 
…(ToshY)

This PR was merged into the 4.4 branch.

Discussion
----------

[Cache] fixes #16310 unclear alternate Redis DSN format

Fixes #16310

Resolved unclear documentation for providing credentials on alternate Redis DSN format.

Commits
-------

6286e3b fixes #16310 unclear alternate DSN format
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this issue Sep 14, 2022
@javiereguiluz
Merge branch '4.4' into 5.4
b1242c8 
* 4.4:
  Removing heading
  fixes symfony#16310 unclear alternate DSN format
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this issue Sep 14, 2022
@javiereguiluz
Merge branch '5.4' into 6.0
0725a38 
* 5.4:
  Fixing "HTML5 color format" link
  Fixing path
  Removing heading
  fixes symfony#16310 unclear alternate DSN format
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this issue Sep 14, 2022
@javiereguiluz
Merge branch '6.0' into 6.1
d8d242d 
* 6.0:
  Replace remaining annotations with attributes, remove remaining annotations configuration block
  Fixing "HTML5 color format" link
  Fixing path
  Removing heading
  fixes symfony#16310 unclear alternate DSN format
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this issue Sep 14, 2022
@javiereguiluz
Merge branch '6.1' into 6.2
9a28537 
* 6.1:
  Replace remaining annotations with attributes, remove remaining annotations configuration block
  Fixing "HTML5 color format" link
  Fixing path
  Removing heading
  fixes symfony#16310 unclear alternate DSN format
This was referenced Oct 27, 2022
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Cache hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix/ci OskarStark/symfony-docs
fix/ci 2 symfony/symfony-docs
3 participants
@xabbuh @aricci95 @ToshY