-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[configtls] add include_system_ca_certs_pool #9142
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9142 +/- ##
==========================================
+ Coverage 90.34% 90.37% +0.02%
==========================================
Files 340 340
Lines 17989 17996 +7
==========================================
+ Hits 16253 16263 +10
+ Misses 1412 1410 -2
+ Partials 324 323 -1 ☔ View full report in Codecov by Sentry. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
…pool is not initialized properly by the system
472de3f
to
073f6b1
Compare
Rebased, anything else to do here? |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
Description:
Add
include_system_ca_certs_pool
to configtls, allowing to load system certs and additional custom certs.Link to tracking Issue:
Fixes #7774
Testing:
UTs
Documentation:
Added to README