homepage

Creating this bug for clarity, but was encountered while investigating 22041 (now set as not a bug).
When using set_tunnel with default port, the port value in the host header is set to "None":
"send: b'POST [PATH] HTTP/1.1\r\nHost: [HOST]:None\r\nAccept-Encoding: identity\r\nContent-Length: 41\r\nAccept: text/plain\r\nContent-type: application/x-www-form-urlencoded\r\n\r\n[FORM_DATA]'"
This issue seems to have been introduced in 3.4 when _tunnel_host and _tunnel_port were introduced to decouple the destination host and port from the proxy host and port.

Bump for review

If call _get_hostport() in set_tunnel() then it should be removed in _tunnel().
As for tests, it would be better do not rely on implementation details. Instead you can monkey-patch the send() method of of HTTPConnection instance and check passed argument.

Thanks Serhiy, new patch addresses your comments.

See also comments on Rietveld (e-mail notification likely fell in spam).

Thanks for the ping Serhiy, indeed the review notification email was sitting in spam. New patch addressing review comments as well as rectifying my own silliness.

New changeset 676d6bcfc031 by Serhiy Storchaka in branch '3.4':
Issue #22095: Fixed HTTPConnection.set_tunnel with default port. The port
https://hg.python.org/cpython/rev/676d6bcfc031
New changeset ebe2072e5472 by Serhiy Storchaka in branch 'default':
Issue #22095: Fixed HTTPConnection.set_tunnel with default port. The port
https://hg.python.org/cpython/rev/ebe2072e5472 

Thank you for your contribution Demian.

New changeset 3b6e0720a69d by Serhiy Storchaka in branch '2.7':
Issue #22095: Fixed HTTPConnection.set_tunnel with default port. The port
https://hg.python.org/cpython/rev/3b6e0720a69d 

Backported to 2.7 because this is needed to support proxy with a port in urllib/urllib2 as documented. See issue24311.