Skip to content

Automatic TLS Certificate Reload#2650

Closed
satheeshaGowda wants to merge 1 commit into
valkey-io:unstablefrom
satheeshaGowda:tls_reload
Closed

Automatic TLS Certificate Reload#2650
satheeshaGowda wants to merge 1 commit into
valkey-io:unstablefrom
satheeshaGowda:tls_reload

Conversation

@satheeshaGowda

Copy link
Copy Markdown
Member

The issue being addressed - #2649

Signed-off-by: Satheesha Chattenahalli Hanume Gowda <satheesha@apple.com>

Automatic TLS Certificate Reload

Signed-off-by: Satheesha Chattenahalli Hanume Gowda <satheesha@apple.com>
@satheeshaGowda

Copy link
Copy Markdown
Member Author

Will add the test if the proposal is accepted .

@zuiderkwast

Copy link
Copy Markdown
Contributor

In which way is this different to #1870? Please review and compare.

@madolson madolson added the major-decision-pending Major decision pending by TSC team label Oct 11, 2025
@madolson

Copy link
Copy Markdown
Member

@satheeshaGowda I think this is a good feature to work on. Based on the last discussion, I think the two important things to incorporate are:

  1. Not re-loading the certificate if it is the same.
  2. Doing the parsing off the main thread, since it can be CPU intensive.

@YiwenZhang12

Copy link
Copy Markdown
Contributor

Thanks for the initial work on this @satheeshaGowda !
Since the original author has left the company, I’ll be taking this effort forward and will handle any remaining updates or follow-ups.

@yang-z-o yang-z-o mentioned this pull request Jan 8, 2026
@madolson

Copy link
Copy Markdown
Member

Replaced with #3020

@madolson madolson closed this Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major-decision-pending Major decision pending by TSC team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants