Dump response from initial redirect
This PR modifies fixes a bug whereby the initial redirect is detected, but the response isn't actually read, which can lead to ResponseNotReady
errors when reusing the same connection for the authentication.
Merge request reports
Activity
Created by: codecov[bot]
Codecov Report
Merging #9 (2a0b3eeb) into main (d8d89f7d) will increase coverage by
4.62%
. The diff coverage is100.00%
.@@ Coverage Diff @@ ## main #9 +/- ## ========================================== + Coverage 70.65% 75.28% +4.62% ========================================== Files 3 3 Lines 167 178 +11 ========================================== + Hits 118 134 +16 + Misses 49 44 -5
Flag Coverage Δ Linux 75.28% <100.00%> (+4.62%)
Windows 100.00% <100.00%> (ø)
macOS 75.28% <100.00%> (+4.62%)
python3.6 73.96% <100.00%> (+4.78%)
python3.7 73.96% <100.00%> (+4.78%)
python3.8 75.28% <100.00%> (+4.62%)
python3.9 75.28% <100.00%> (+4.62%)
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files Coverage Δ requests_ecp/__init__.py 60.00% <100.00%> (+4.95%)
requests_ecp/tests/test_requests_ecp.py 100.00% <100.00%> (ø)
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update d8d89f7...2a0b3ee. Read the comment docs.By Duncan Macleod on 2021-03-19T10:15:15 (imported from GitLab)
Edited by Ghost UserMerged by: duncanmmacleod at 2021-03-19 10:21:23 UTC
By Duncan Macleod on 2024-06-12T16:02:10 (imported from GitLab)
Edited by Ghost Useradded api:patch label
assigned to @duncanmmacleod
By Duncan Macleod on 2021-03-19T10:22:45 (imported from GitLab)