Skip to content

feat: add support for TLS protocol to Redis connection (rediss://)#425

Merged
mgruner merged 3 commits into
zammad:mainfrom
dreglad:redis-url-tls
May 12, 2026
Merged

feat: add support for TLS protocol to Redis connection (rediss://)#425
mgruner merged 3 commits into
zammad:mainfrom
dreglad:redis-url-tls

Conversation

@dreglad
Copy link
Copy Markdown
Contributor

@dreglad dreglad commented May 11, 2026

What this PR does / why we need it

  • Introduces a tls configuration option under zammadConfig.redis (defaulting to false for backwards compatibility).
  • Updates the REDIS_URL in _helpers.tpl to dynamically use the rediss:// protocol schema when TLS is enabled, instead of hardcoding redis://.
  • This fixes connectivity issues and enables native SSL/TLS connections when connecting Zammad to external Redis instances, or configuring Redis subchart to use TLS/SSL.

Special notes for your reviewer

  • Helm template generation was tested locally and correctly provisions rediss:// only when tls: true.

Checklist

  • Chart Version bumped

Signed-off-by: David Regla <dreglad@gmail.com>
monotek
monotek previously approved these changes May 11, 2026
Comment thread zammad/Chart.yaml Outdated
@dreglad dreglad changed the title fix: un-hardcode Redis connection protocol to support TLS (rediss://) feat: add support for TLS protocol to Redis connection (rediss://) May 11, 2026
@mgruner mgruner self-assigned this May 12, 2026
@mgruner mgruner merged commit b29ff4d into zammad:main May 12, 2026
8 checks passed
@mgruner
Copy link
Copy Markdown
Collaborator

mgruner commented May 12, 2026

Thanks and 👏 @dreglad

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