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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature]: Support for redis sentinel #632

Closed
onedr0p opened this issue Aug 18, 2022 · 2 comments 路 Fixed by #757
Closed

[Feature]: Support for redis sentinel #632

onedr0p opened this issue Aug 18, 2022 · 2 comments 路 Fixed by #757
Assignees
Labels
馃敤 enhancement New feature or request
Milestone

Comments

@onedr0p
Copy link
Sponsor Contributor

onedr0p commented Aug 18, 2022

Hi 馃憢馃徏

It would be great if blocky could connect to a highly available redis instance.

Here's a couple examples of applications that support this method of redis.

https://www.authelia.com/configuration/session/redis/
https://github.com/outline/outline/blob/93bb9d067d0f61a2f5d6f1d302774b128095a4d0/.env.sample#L22L29

Thanks!

@kwitsch
Copy link
Collaborator

kwitsch commented Aug 21, 2022

Looks reasonable to me.
Will work on it as I get some time.

@0xERR0R 0xERR0R added the 馃敤 enhancement New feature or request label Aug 21, 2022
@kwitsch kwitsch self-assigned this Aug 22, 2022
@0xERR0R 0xERR0R added this to the future milestone Nov 4, 2022
@0xERR0R 0xERR0R modified the milestones: future, 0.21 Nov 21, 2022
0xERR0R added a commit that referenced this issue Nov 24, 2022
Co-authored-by: Dimitri Herzog <dimitri.herzog@gmail.com>
@onedr0p
Copy link
Sponsor Contributor Author

onedr0p commented Nov 24, 2022

Thanks! This makes blocky more resilient to failure if a redis master goes offline. Much appreciated 馃憤

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
馃敤 enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants