Previously,http.client would always send content body immediately and ignore any 100 responses. This change makesHTTPClient.request() wait for a Continue response if theExpect: 100-Continue header is set, and adds a parameter toHTTPClient.getresponse() that will cause it to return 100 responses instead of eating them.

https://bugs.python.org/issue1346874

bpo-1346874: Add 'Expect: 100-Continue' support to httplib

3d8b7b1

Previously, http.client would always send content body immediately and ignore any 100 responses. This changemakes HTTPClient.request() wait for a `Continue` response if the `Expect: 100-Continue` header is set, andadds a parameter to HTTPClient.getresponse() that will cause it to return 100 responses instead of eating them.

the-knights-who-say-ni added the CLA signed label

Jul 21, 2019

bedevere-bot added the awaiting review label

Jul 21, 2019

Copy link

Author

tbartlett0 commentedJul 21, 2019

This PR is intended to be backwards-compatible, but I have identified two potential edge cases:

clients that currently setExpect: 100-Continue with a broken server that ignores the header: This will introduce a delay ofexpect_timeout seconds to each request. However, this can be fixed by a) not sending the header or b) settingexpect_timeout to 0.
anyone who redefinesHTTPClient.response_class: this is likely to be a breaking change, as a new kwarg is used onHTTPResponse.begin(). (This seems like a fairly rare situation, as it's not exposed as an__init__ param, but chances are that someone, somewhere has done it...)

Fix rstlint errors

7390814

csabella requested review fromvadmium andorsenthil

February 6, 2020 22:29

csabella requested review fromorsenthil and removed request fororsenthil andvadmium

June 12, 2020 22:32

Copy link

quadoss commentedApr 5, 2022

Is there an update on this one, its been in open state for couple of years and I too have been hitting this and wanting to see when this would be merged

ezio-melotti removed the CLA signed label

Jul 13, 2022

Copy link

emnoor-reef commentedAug 5, 2023

Just bumping. Is this PR on consideration for getting merged? Or, are there any other plans for#42550 ? I am encountering this problem at work.

Copy link

python-cla-botbot commentedApr 6, 2025•
edited
Loading

All commit authors signed the Contributor License Agreement.

Copy link

IngridMorstrad commentedApr 8, 2025•
edited
Loading

Hi,@tbartlett0
Is this PR still being worked on? Or can someone else (me) take it over?

Copy link

Member

AA-Turner commentedApr 8, 2025

Tim would need to (re-)sign the CLA.

Copy link

Author

tbartlett0 commentedApr 9, 2025

Hi, I haven't touched this since it was submitted, since there seemed to be limited interest and I ended up working around it in other ways. If someone else wants to take it on I have no problem with that (either using my code or rewriting their own, there might be a better way to achieve it)