-
-
Notifications
You must be signed in to change notification settings - Fork 950
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
feat: hot-reload TLS certificates #2744
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2744 +/- ##
==========================================
- Coverage 75.22% 75.16% -0.06%
==========================================
Files 294 294
Lines 17081 17082 +1
==========================================
- Hits 12849 12840 -9
- Misses 3253 3265 +12
+ Partials 979 977 -2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice 🎉
Sorry for the long review time, somehow forgot about this PR 😅
I just have some ideas how to simplify and improve the code a bit further 😉
return p.getTSLCertificates( | ||
type CertFunc = func(*tls.ClientHelloInfo) (*tls.Certificate, error) | ||
|
||
func (p *Config) GetTLSCertificatesForPublic(ctx context.Context) CertFunc { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It took me a few seconds to spot the typo 😂 nice find
f87b62d
to
ee8f300
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, this looks great now 😍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks! Just a two questions :)
ory/hydra#2850 but for Kratos