Skip to content

Redis ACL/Username and Database Support #1044

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

Open
oxzi opened this issue Aug 8, 2024 · 2 comments · May be fixed by #1165
Open

Redis ACL/Username and Database Support #1044

oxzi opened this issue Aug 8, 2024 · 2 comments · May be fixed by #1165
Labels
area/configuration Affects the configuration enhancement New feature or improvement
Milestone

Comments

@oxzi
Copy link
Member

oxzi commented Aug 8, 2024

Is your feature request related to a problem? Please describe.

There were requests for both ACL and database support for the Redis server, e.g., Icinga/icingadb#529, Icinga/icingadb#676, Icinga/icinga2#9536.

I have started implementing this in the icinga-go-library, being available in Icinga DB after the next icinga-go-library release, and created a PR in Icinga 2, Icinga/icinga2#10102. To make this feature fully usable, Icinga DB Web also needs to support authentication via username and selecting a database directly after establishing the connection.

Describe the solution you'd like

As it seems, Icinga DB Web uses Predis as its Redis library. Fortunately, it supports both username authentication and database selection, https://github.com/predis/predis/wiki/Connection-Parameters. Thus it "just" needs to be made configurable.

Describe alternatives you've considered

N/A

Additional context

@nilmerg nilmerg added this to the 1.2.0 milestone Aug 13, 2024
@nilmerg nilmerg added enhancement New feature or improvement area/configuration Affects the configuration labels Aug 13, 2024
@w1ll-i-code
Copy link

Hi @nilmerg, we also need this feature. If you want I can develop it. Are there any strict requirements?

@nilmerg
Copy link
Member

nilmerg commented Apr 22, 2025

Are there any strict requirements?

None that I know of spontaneously 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/configuration Affects the configuration enhancement New feature or improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants