Skip to content
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 Bug About Chunk Error #1697

Closed
wants to merge 1 commit into from
Closed

Fix Bug About Chunk Error #1697

wants to merge 1 commit into from

Conversation

DeeLMind
Copy link

To Fix : requests.exceptions.ChunkedEncodingError: ('Connection broken: IncompleteRead(0 bytes read)', IncompleteRead(0 bytes read))

To Fix : requests.exceptions.ChunkedEncodingError: ('Connection broken: IncompleteRead(0 bytes read)', IncompleteRead(0 bytes read))
@sethmlarson
Copy link
Member

Is this related to this thread: #1516

@sethmlarson
Copy link
Member

This fix has landed in master already. Thank you for this fix.

@sethmlarson sethmlarson closed this Nov 7, 2019
@sethmlarson sethmlarson reopened this Nov 7, 2019
@codecov-io
Copy link

Codecov Report

Merging #1697 into master will decrease coverage by 0.04%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1697      +/-   ##
=========================================
- Coverage   99.75%   99.7%   -0.05%     
=========================================
  Files          22      22              
  Lines        2006    2008       +2     
=========================================
+ Hits         2001    2002       +1     
- Misses          5       6       +1
Impacted Files Coverage Δ
src/urllib3/response.py 99.72% <66.66%> (-0.28%) ⬇️

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 f4bff6f...c7f0d2a. Read the comment docs.

@sethmlarson
Copy link
Member

I mistakenly closed this. My bad.

@DeeLMind
Copy link
Author

DeeLMind commented Nov 8, 2019

It does't matter , thank you for your reply , I am newer

@DeeLMind DeeLMind closed this Nov 8, 2019
@mdeuts
Copy link

mdeuts commented Jan 5, 2023

Dear @sethmlarson
This fix is absent in master...
Any chance to re-examine its necessity please?
Many thanks!!

@pquentin
Copy link
Member

pquentin commented Jan 11, 2023

@mdeuts The fix we landed is #1888 to report exactly what the issue is: it's a server-side issue and the servers reporting incorrect chunks should be fixed.

However note that #2860 will give a clearer message when the response is empty, which appears to be the usual reason for this error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants