Support Redis connection password

Created on 26 November 2024, 4 months ago

Problem/Motivation

The module throws constant DB log errors if the Redis connection is password protected.

📌 Task
Status

Active

Version

1.1

Component

Code

Created by

🇭🇺Hungary aron novak Hungary, Budapest

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @aron novak
  • Merge request !4Handle password at Redis connection → (Merged) created by aron novak
  • 🇭🇺Hungary aron novak Hungary, Budapest
  • Pipeline finished with Success
    4 months ago
    Total: 143s
    #351114
  • 🇩🇪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 of password, 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.

  • Pipeline finished with Success
    4 months ago
    Total: 264s
    #351802
  • 🇩🇪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.

  • Pipeline finished with Success
    4 months ago
    Total: 139s
    #351882
  • Pipeline finished with Skipped
    4 months ago
    #351945
  • 🇩🇪Germany jurgenhaas Gottmadingen

    Thx.

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024