Berker Peksag <[email protected]> added the comment:
Thank you for tracking down the problem, William! I just took a quick look at
your patch and it seems reasonable to me.
Setting protocol_version to 'HTTP/1.1' means setting the close_connection
attribute of BaseHTTPRequestHandler to False which is expected and a feature of
HTTP/1.1 (persistent connections)
Quoting RFC 2616:
HTTP/1.1 defines the "close" connection option for the sender to
signal that the connection will be closed after completion of the
response.
HTTP/1.1 applications that do not support persistent connections MUST
include the "close" connection option in every message.
So I think it's OK to send a "Connection: close" header in the test (it would
be nice to add a comment though)
----------
components: -Library (Lib)
nosy: +berker.peksag
_______________________________________
Python tracker <[email protected]>
<https://bugs.python.org/issue25095>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
