-
-
Notifications
You must be signed in to change notification settings - Fork 299
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add option to set aclfile #485
Conversation
Hey @simonasr Thanks for this. Can you drop an update to the change log under the ## Unreleased section then I can get this shipped asap |
758a7d9
to
8cf3a60
Compare
Released as: 6.6.0 |
@simonasr do you want to take point on removing some of the older configuration options for redis? Looks like we really only want to be supporting 6+ at a minimum |
Hi @damacus Sure, I'd be happy to help with that in my free time. Just to be clear, are we talking about removing obsolete conditional statements from the configuration templates and related code if such, or is there more to it? As far as I know, Redis hasn't actually removed any configuration options, so I just want to understand the scope of what we're doing. |
Yeah, pretty much want to make sure 6 and 7 work. It's easier if it's headed up by someone who still uses it |
Description
Adding
aclfile
option to Redis configuration file.Issues Resolved
Enables configuring external ACL file for Redis v6+
Check List
## Unreleased