This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author vstinner
Recipients berker.peksag, ezio.melotti, m.xhonneux, r.david.murray, rhettinger, vstinner, xiang.zhang
Date 2016-11-22.08:31:36
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1479803496.88.0.411993606581.issue27414@psf.upfronthosting.co.za>
In-reply-to
Content
Mathieu: "For consistency, all header values should be str."

Xiang: issue27414_v2.patch: Doc/library/http.server.rst
"*keyword* and *value* should both be :class:`str`."

I'm sorry, but I don't understand the whole point of this issue. The current code works, I don't see any good reason to change it.

The unit test is a little bit complex for a little tiny implementation detail. Usually, we try to avoid unit tests on implementation details in the Python test suite.

It's perfectly fine to pass an integer to an HTTP value, it's just super convenient.

I checked the HTTP client: the type of the 'argument' parameter of putheader() is not documented neither, so I think that it's just fine to leave the doc unchanged.

I close now this issue NOTABUG.
History
Date User Action Args
2016-11-22 08:31:36vstinnersetrecipients: + vstinner, rhettinger, ezio.melotti, r.david.murray, berker.peksag, xiang.zhang, m.xhonneux
2016-11-22 08:31:36vstinnersetmessageid: <1479803496.88.0.411993606581.issue27414@psf.upfronthosting.co.za>
2016-11-22 08:31:36vstinnerlinkissue27414 messages
2016-11-22 08:31:36vstinnercreate