Issue 36029: Use title-case HTTP header fields (original) (raw)

Created on 2019-02-18 22:33 by maggyero, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 11924 closed maggyero,2019-02-18 22:34
Messages (5)
msg335870 - (view) Author: Géry (maggyero) * Date: 2019-02-18 22:33
Use consistent case for HTTP header fields, according to [RFC 7231](https://tools.ietf.org/html/rfc7231).
msg335908 - (view) Author: Stéphane Wirtel (matrixise) * (Python committer) Date: 2019-02-19 08:56
Thank you for your contribution and your issue, it's really appreciated. I had a doubt about this point and I wanted to verify about the HTTP headers. But in the previous version of this RFC7230: https://www.w3.org/Protocols/rfc2616/rfc2616.html Here is the section 4.2 "Message Headers" of this previous RFC: https://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2 where you have "Field names are case-insensitive". We can see the RFC2616 was replaced by multiple RFCs (7230-7237) and in the RFC 7230, there is a section 3.2 where there is the description of the header fields. In the section 3.2 of RFC7230: 3.2. Header Fields " Each header field consists of a case-insensitive field name followed by a colon (":"), optional leading whitespace, the field value, and optional trailing whitespace. " https://tools.ietf.org/html/rfc7230#section-3.2 So in this case, it's case-insensitive. I think we can close this issue and the associated Pull Request. Have a nice day and thank you,
msg336147 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2019-02-20 20:15
As stated on the PR, changing the code isn't going to be worth it, but updating the docs would be nice.
msg336465 - (view) Author: Géry (maggyero) * Date: 2019-02-24 12:19
Please see the changes and my reply in the PR.
msg337567 - (view) Author: Julien Palard (mdk) * (Python committer) Date: 2019-03-09 09:50
Closing as "not a bug" as discussed in the PR. Anyway thanks Géry for the work on this, even if not merged it's appreciated!
History
Date User Action Args
2022-04-11 14:59:11 admin set github: 80210
2019-03-09 09:50:28 mdk set status: open -> closedresolution: not a bugmessages: + stage: patch review -> resolved
2019-02-24 12:19:50 maggyero set messages: + title: Use consistent case for HTTP header fields -> Use title-case HTTP header fields
2019-02-20 20:15:54 brett.cannon set messages: +
2019-02-19 08:56:47 matrixise set nosy: + matrixisemessages: +
2019-02-18 22:34:44 maggyero set keywords: + patchstage: patch reviewpull_requests: + <pull%5Frequest11949>
2019-02-18 22:33:30 maggyero create