- 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 3:47pm 14 August 2023 - π¬π§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
8 months ago 8:14pm 16 August 2024 - π¬π§United Kingdom mark-mackenzie-nexus
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.