rpj
commented
8 months ago This is fantastic, thank you @ilyaBykonya for submitting it! 🙇
Overall it looks really good, but we do require that a test be added for newly-added or changed functionality, which this PR definitely qualifies for.
Fortunately the GitHub Actions CI pipeline is using redis version 7.2.4 so it will be fully testable within the CI pipeline setup here.
I've just removed the Actions restriction so you will be able to run further pipelines without my approval, so you can be sure your tests succeed in the CI pipeline by just pushing the changes here (or to a separate branch if you prefer) and letting the pipeline run.
Please get in touch with any questions, I'm more than happy to help guide you in any way I can!
Thank you again, I really appreciate the contribution!
Redis 6.2 introduced the function of user ACL lists, which limit access and increase the level of system security: https://redis.io/docs/management/security/acl/
The changes in the fork were aimed only at adding support for this functionality.
In the first commit, an overload of the authenticate method was added to work with the new ACL, and the old method simply called: authenticate("default", password); which for new versions of Redis is identical to authenticate(password);
The second commit was caused by the fact that the old version, being the only (old) authentication method for requirepass, was no longer compatible with it. And I've added code duplication to maintain backward compatibility with Redis < 6.2