Support TLS for Predis

Created on 5 October 2018, over 6 years ago
Updated 16 August 2024, 5 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

Needs work

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 over 1 year 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

  • Status changed to Needs work 5 months ago
  • 🇨🇭Switzerland berdir Switzerland

    Needs a rebase.

Production build 0.71.5 2024