add optional TLS for Redis Sentinel connections via env flag #13359
Devenir-Glorieux
started this conversation in
Share your Work
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Describe the feature or potential improvement
Langfuse currently supports Redis TLS via
REDIS_TLS_ENABLED, but in Sentinel mode it does not expose a dedicated env-controlled way to enable TLS for Sentinel endpoints themselves.In TLS-only Sentinel setups, this forces downstream users to patch built artifacts in Docker images.
In
createRedisSentinelInstancewe already pass...tlsOptions, but Sentinel-specific TLS options are not conditionally controlled via env:enableTLSForSentinelModesentinelTLSAs a result, users cannot configure Sentinel TLS cleanly through documented environment variables.
Proposal: add a new env variable:
REDIS_SENTINEL_TLS_ENABLED(true|false, defaultfalse)REDIS_SENTINEL_TLS_ENABLEDonly has effect whenREDIS_SENTINEL_ENABLED=true.REDIS_TLS_ENABLED=true(so TLS options exist).REDIS_SENTINEL_TLS_ENABLED=trueis explicitly set.https://redis.github.io/ioredis/interfaces/SentinelConnectionOptions.html#enableTLSForSentinelMode
Additional information
No response
All reactions