Support TLS for Predis

Created on 5 October 2018, over 5 years ago
Updated 14 August 2023, 11 months ago

Hi there,

I need to use a tls connection for redis, and there is no way to set this schema in the settings.php file.

Furthermore, Predis Client, from redis contribute module, does not offer the possibility to pass a password either.

✨ Feature request
Status

RTBC

Version

1.0

Component

Code

Created by

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • First commit to issue fork.
  • πŸ‡¨πŸ‡΄Colombia jidrone

    I completely agree with @eelkeblok, if the issue already provide a MR is not needed/good to add patch and even worse without an interdiff.

    The only difference I see between last patch and the MR is the following, which should not be responsibility of this issue.

    +++ b/src/ClientFactory.php
    @@ -27,6 +32,11 @@ class ClientFactory {
    +  /**
    +   * Redis default scheme: will not authenticate.
    +   */
    +  const REDIS_DEFAULT_SCHEME = 'tcp';
    

    That being said, I tested the patch from the MR and it is working fine, so I just re-rolled, moved the new parameter to the end of some config arrays and fixed some CS.

  • Status changed to RTBC 11 months ago
  • πŸ‡¬πŸ‡§United Kingdom Alina Basarabeanu

    The patch from #37 fixed the issue in Drupal Version 9.5.10 and redis 1.7.0.
    Can this be merged into the next stable release?
    Thank you

Production build 0.69.0 2024