-
Notifications
You must be signed in to change notification settings - Fork 344
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
fix for min tls version to v1.2 #2119
Conversation
Signed-off-by: kangsheng89 <ks_wong_89@hotmail.com>
Codecov ReportBase: 87.32% // Head: 87.32% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #2119 +/- ##
=======================================
Coverage 87.32% 87.32%
=======================================
Files 100 100
Lines 6865 6865
=======================================
Hits 5995 5995
Misses 667 667
Partials 203 203 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: kangsheng89 <ks_wong_89@hotmail.com>
wdyt @rubenvp8510 ? |
@frzifus i will update for TLS cipher as well. Hehe pending..LOL |
Signed-off-by: kangsheng89 <ks_wong_89@hotmail.com>
This looks good to me! |
currently the CICD is failing @frzifus can u point me on how i can fix this? |
@frzifus can help to rerun the CICD pipeline? |
@kangsheng89 it did already pass :) |
Which problem is this PR solving?
Fix #2118
Short description of the changes
Adding min tls version for webhook server during bootstraping