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
Restore context.set_ciphers() to create_urllib3_context() #1463
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1463 +/- ##
==========================================
- Coverage 99.45% 98.49% -0.97%
==========================================
Files 22 22
Lines 1835 1857 +22
==========================================
+ Hits 1825 1829 +4
- Misses 10 28 +18
Continue to review full report at Codecov.
|
Looks like gdal has gcc as a dependency now (it didn't in previous installs) and is failing because gcc is already installed on the system and it can't overwrite it? |
…MichaelLarson-patch-1
fc8e77a
to
6e2d616
Compare
test/test_ssl.py
Outdated
|
||
assert context.set_ciphers.call_count == 1 | ||
assert context.set_ciphers.call_args == mock.call(expected_ciphers) | ||
|
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 looks like CI is failing because of this empty line.
Thanks Thea, I'd say we're release-ready now. |
Fixes #1462.