Support TLS for Predis

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.

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

Needs a rebase.

Please can the patch at comment #37 please be added to a production release. It is necessary to allow the module to work with Azure Cache for Redis. Tested successfully from Azure App Service with Drupal 10.3.

Thanks for the fix. I can confirm the patch at #37 works for me in an Azure hosted environment with Predis. I would be very interested in seeing it merged so I don't have to manually track this issue anymore.

@berdir I have merged the 8.x-1.9 tag which is at 8.x-1.v HEAD into the 3004561-support-tls-for branch.

The patch at #37 ✨ Support TLS for Predis Needs review works for me using AWS Elasticache-Redis with Drupal 10.4.5, drupal/redis 1.9.0, and predis 2.4.0.

Thank you @Eelke Blok for you MR.
The diff let me solve a TLS issue on Drupal Predis.
REgards