Thanks to visit codestin.com
Credit goes to github.com

Skip to content

BC Break when using Redis transport with an empty username #43306

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
alexsegura opened this issue Oct 4, 2021 · 2 comments · Fixed by #43484
Closed

BC Break when using Redis transport with an empty username #43306

alexsegura opened this issue Oct 4, 2021 · 2 comments · Fixed by #43484
Labels
Bug Messenger Status: Needs Review

Comments

@alexsegura
Copy link
Contributor

Symfony version(s) affected: 5.3.8

Description

I'm using Symfony Messenger with Redis transport (it's Redis 5.x), and when upgrading symfony/redis-messenger 5.3.8, my current setup is broken.

The errors are:

Symfony\Component\Messenger\Exception\InvalidArgumentException
Redis connection failed: ERR wrong number of arguments for 'auth' command

ErrorException
Warning: Redis::auth() expects parameter 1 to be string, array given

It used to work with the following configuration:

MESSENGER_TRANSPORT_DSN="redis://:password@host/messages/symfony/consumer"

There are some changes between 5.3.7 and 5.3.8 that might explain the bug

symfony/redis-messenger@v5.3.7...v5.3.8

How to reproduce

Configure Redis transport with the following env var:

MESSENGER_TRANSPORT_DSN="redis://:password@host/messages/symfony/consumer"
@alexsegura alexsegura added the Bug label Oct 4, 2021
@alexsegura
Copy link
Contributor Author

I confirm that downgrading symfony/redis-messenger to 5.3.7 fixes my issue

coopcycle/coopcycle-web@b02ba7a

@n0rbyt3
Copy link
Contributor

n0rbyt3 commented Oct 4, 2021

@alexsegura There has been a bug where user AND password authentication was not possible. You only need to remove the colon in your dsn string to fix your problem.

But it's indeed a BC break.

@villfa villfa mentioned this issue Oct 13, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Messenger Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Messenger] Fix Redis Transport when username is empty
4 participants
@alexsegura @xabbuh @n0rbyt3 @carsonbot