Preserve chunked flag when re-calling urlopen() in retry loop

[lmvlmv]

No description provided.

[lmvlmv]

Attempts to fix #1714

[codecov-io]

Codecov Report

Merging #1715 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1715   +/-   ##
=======================================
  Coverage   99.65%   99.65%           
=======================================
  Files          22       22           
  Lines        2006     2006           
=======================================
  Hits         1999     1999           
  Misses          7        7

Impacted Files	Coverage Δ
src/urllib3/connectionpool.py	100% <ø> (ø)	⬆️

---

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 79e81f9...6f4da3b. Read the comment docs.

[pquentin]

Thank you for your pull request! That's a nice and important fix, and thank you for helping urllib3 and requests play well together.

There are two other recursive calls that are also missing chunked=chunked, but we'll worry about them later.

I added a test to make sure that we don't his this issue again, so someone else will need to review this :)

[lmvlmv]

NP. Thanks for that. Couldn't get the test env working locally. Since the change to requests more people will be visiting this function call so good to get it fixed first. Not sure when the next requests release will be...

[sethmlarson]

Wow, great catch!

[sethmlarson]

Thanks @lmvlmv and @pquentin! 🎉