- Issue created by @aron novak
- 🇩🇪Germany jurgenhaas Gottmadingen
Never thought that Redis would have to be secured with authentication, but obviously that's a use case.
The MR implements only one of 2 possible auth mechanisms and it ends up with
redis://:password@host:port
. I think, this is a colon to much in front ofpassword
, isn't it?When we really implement auth, we should then also implement the ACL method which comes with a username, too. So, password is optional, and if a password is provided, there is also an optional username.
- 🇭🇺Hungary aron novak Hungary, Budapest
I just added support for username too, good point.
One of the large Drupal-specific PaaS providers does expose Redis to the public internet, so you can debug it easily, then, they of course protect it with authentication.
- 🇩🇪Germany jurgenhaas Gottmadingen
The problem with the extra colon in front of the password still remains if no username is provided. I've added a code suggestion to the MR for you.
-
jurgenhaas →
committed b617779c on 1.1.x authored by
aron novak →
Issue #3490036: Support Redis connection password
-
jurgenhaas →
committed b617779c on 1.1.x authored by
aron novak →
Automatically closed - issue fixed for 2 weeks with no activity.