Skip to content

gh-66897: Upgrade HTTP CONNECT to protocol HTTP/1.1 #8305

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.

Sign up for GitHub

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

Merged
orsenthil merged 17 commits into from
Apr 5, 2023
Merged

gh-66897: Upgrade HTTP CONNECT to protocol HTTP/1.1 #8305

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
9f5ce09
bpo-22708: Upgrade HTTP CONNECT to protocol HTTP/1.1 (GH-NNNN)
handlerbot Jul 16, 2018
c3927ed
Refactor tests to pass under -bb (fix ByteWarnings); missed some line…
handlerbot Jul 16, 2018
2fe3ade
Use consistent 'tunnelling' spelling in Lib/http/client.py
handlerbot Jul 20, 2018
b2f76d7
Lib/test/test_httplib: Remove remnant of obsoleted test.
handlerbot Jul 20, 2018
9990baa
Use dict.copy() not copy.copy()
handlerbot Jul 24, 2018
367b407
fix version changed
merwok Oct 7, 2022
f370042
Merge branch 'main' into fix-bpo-22708
handlerbot Oct 8, 2022
8b55642
Update Lib/http/client.py
handlerbot Oct 8, 2022
aa1508e
Switch to for/else: syntax, as suggested
handlerbot Oct 10, 2022
ef8b896
Don't use for: else:
handlerbot Oct 25, 2022
5d2f166
Merge branch 'main' into fix-bpo-22708
handlerbot Oct 25, 2022
87e514c
Sure, fine, w/e
handlerbot Oct 26, 2022
d224052
Oops
handlerbot Oct 26, 2022
70ba474
1nm to the left
handlerbot Oct 26, 2022
0a5db5f
Merge branch 'main' into fix-bpo-22708
handlerbot Oct 26, 2022
d3b5726
Merge branch 'main' into fix-bpo-22708
handlerbot Jan 12, 2023
92c4f20
Merge branch 'main' into fix-bpo-22708
arhadthedev Mar 22, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Switch to for/else: syntax, as suggested
  • Loading branch information
@handlerbot
handlerbot authored Oct 10, 2022
commit aa1508ee7c91686fffa587c9c3846d3cdef69fc9
4 changes: 1 addition & 3 deletions Lib/http/client.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -901,12 +901,10 @@ def set_tunnel(self, host, port=None, headers=None):
else:
self._tunnel_headers.clear()

saw_host_header = False
for header in self._tunnel_headers.keys():
if header.lower() == "host":
saw_host_header = True
break
if not saw_host_header:
else:
encoded_host = self._tunnel_host.encode("idna").decode("ascii")
self._tunnel_headers["Host"] = "%s:%d" % (
encoded_host, self._tunnel_port)
Expand Down