-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Load system CA certificates by default on empty HTTPSConnection #1608
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1608 +/- ##
==========================================
- Coverage 99.73% 99.47% -0.27%
==========================================
Files 22 22
Lines 1892 1901 +9
==========================================
+ Hits 1887 1891 +4
- Misses 5 10 +5
Continue to review full report at Codecov.
|
7639407
to
bd33f3e
Compare
It's a nice change, but unfortunately, this pull request decreases coverage |
:'-( |
Right, only wrote tests for the Connection level logic. Dang, I'm on it. |
Closes #1603.