- 🇫🇷France andypost
Support for Redis 6 and 7 was added in https://pecl.php.net/package/redis/6.0.0 version only, probably it means that you need to add version check somewhere
Moreover Redis 7 added ACLv2 https://redis.com/blog/introducing-redis-7/
- 🇨🇦Canada fengtan Montreal, Canada
Okay, I have added a version check to both
/admin/reports/status
and/admin/reports/redis
.From what I can see, ACL's are supported by:
- PhpRedis 5.3.0+
- Predis 1.1.4+
As far as I can tell ACLv2 brings server-specific enhancements (mainly selectors and key-based permissions), but does not introduce incompatibilities with regards to how the client connects to the server. In short this patch brings support to both ACLv1 and ACLv2.
- Status changed to RTBC
over 1 year ago 9:00pm 6 October 2023 - First commit to issue fork.
- 🇩🇪Germany Harlor Berlin
Without the patch a username in PhpRedis can be configured by using:
$settings['redis.connection']['password'] = ['user' => '...', 'pass' => '...'];
But it makes totally sense to have a meaningful wording, a consistent configuration and a reporting for PhpRedis and Predis - which the patch offers.
We have tested the patch with PhpRedis sentinel and it worked!
We tested with the old settings (see above) and the new settings (see below).
$settings['redis.connection']['password'] = '...'; $settings['redis.connection']['username'] = '...';
+1 RTBC
- Status changed to Needs work
8 months ago 7:56pm 16 August 2024 - 🇨🇭Switzerland berdir Switzerland
Please provide a merge request so tests of all backend can be run.
- 🇩🇪Germany Harlor Berlin
Not sure how the test results have to be interpreted but here is the MR.