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

Enable TLS connection for Redis #2334

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Ziris85
Copy link

@Ziris85 Ziris85 commented Nov 16, 2024

Hello!

This is a hopefully simple PR to allow users to configure Redis to connect to a TLS-enabled Redis endpoint. It proposes a new environment variable REDIS_PROTOCOL that can be passed to the container, and replaces all areas (that I found) where tcp:// was either hard-coded or the assumed (omitted) default. All adjustments will continue to use tcp as the default if the env var isn't passed, so this shouldn't impact existing installations.

I don't do PR's very frequently so apologies if I missed something here, but I'm interested in thoughts/feedback!

Thank you!

@LordRobinCbz
Copy link

Hello,

I was here to add this feature, but you've done it.
Do you think tat you can add suport for mTLS to ?

Seems not complicated if cert,key and ca are given with base64 string.
With something like REDIS_CLIENT_CERT_BASE64 and so on ?

@LordRobinCbz
Copy link

I will probably add support for mTLS with Mariadb to ...

@J0WI
Copy link
Contributor

J0WI commented Jan 11, 2025

Needs documentation in REDME

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants