Skip to content
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

Adds password capabilities to icingadb-redis configuration #202

Merged
merged 2 commits into from
Nov 10, 2023

Conversation

mocdaniel
Copy link
Contributor

For now we added basic requirepass capabilities. We might want to look into other, more advanced available configuration options?

@cla-bot cla-bot bot added the cla/signed label Oct 31, 2023
@mocdaniel mocdaniel requested a review from mkayontour October 31, 2023 14:35
@gianmarco-mameli
Copy link
Contributor

I'm interested in this feature, actually I was thinking to add the config myself, if you need help I'm available

@mocdaniel mocdaniel requested a review from mkayontour November 10, 2023 08:46
Copy link
Member

@mkayontour mkayontour left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the PR, I'll merge it.

@mkayontour mkayontour merged commit a63eb0d into main Nov 10, 2023
@mkayontour mkayontour linked an issue Nov 10, 2023 that may be closed by this pull request
@mocdaniel mocdaniel deleted the feature/redis-password branch December 7, 2023 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IcingaDB-Redis: Allow to set a Redis password
3 participants