[issue12455] urllib2 forces title() on header names, breaking some requests

Change by Mark Lawrence : -- nosy: -BreamoreBoy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue12455] urllib2 forces title() on header names, breaking some requests

Changes by Demian Brecht demianbre...@gmail.com: -- nosy: -demian.brecht ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455 ___ ___

[issue12455] urllib2 forces title() on header names, breaking some requests

karl added the comment: Mark, I'm happy to followup. I will be in favor of removing any capitalization and not to change headers whatever they are. Because it doesn't matter per spec. Browsers do not care about the capitalization. And I haven't identified Web Compatibility issues regarding

[issue12455] urllib2 forces title() on header names, breaking some requests

Mark Lawrence added the comment: @Karl do you intend following up on this issue? -- nosy: +BreamoreBoy ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455 ___

[issue12455] urllib2 forces title() on header names, breaking some requests

karl added the comment: terry.reedy: You said: and that has_header and get_header *require* the .capitalize() form and reject the .title() form. I made a patch for these two. See http://bugs.python.org/issue5550 -- ___ Python tracker

[issue12455] urllib2 forces title() on header names, breaking some requests

karl added the comment: Are there issues related to removing the capitalize() and title() appears? # title() * http://hg.python.org/cpython/file/886df716cd09/Lib/urllib/request.py#l1239 # capitalize() * http://hg.python.org/cpython/file/886df716cd09/Lib/urllib/request.py#l359 *

[issue12455] urllib2 forces title() on header names, breaking some requests

karl added the comment: tests in http://hg.python.org/cpython/file/886df716cd09/Lib/test/test_wsgiref.py#l370 also checking that everything is case insensitive. And the method to get the headers in wsgiref, make sure they are lower-case

[issue12455] urllib2 forces title() on header names, breaking some requests

karl added the comment: Note that HTTP header fields are case-insensitive. See http://tools.ietf.org/html/draft-ietf-httpbis-p1-messaging#section-3.2 Each HTTP header field consists of a case-insensitive field name followed by a colon (:), optional whitespace, and the field value.

[issue12455] urllib2 forces title() on header names, breaking some requests

karl added the comment: So looking at the casing of headers, I discovered other issues. I opened another bug. http://bugs.python.org/issue17322 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455

[issue12455] urllib2 forces title() on header names, breaking some requests

Changes by Demian Brecht demianbre...@gmail.com: -- nosy: +dbrecht ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455 ___ ___ Python-bugs-list

[issue12455] urllib2 forces title() on header names, breaking some requests

Terry J. Reedy added the comment: The comment about urllib.request forcing .title() is consistent with 'Content-Length' and 'Content-Type' in the docs but puzzling and inconsistent given that in 3.3, header names are printed .capitalize()'ed and not .title()'ed and that has_header and

[issue12455] urllib2 forces title() on header names, breaking some requests

Changes by Éric Araujo mer...@netwok.org: Removed file: http://bugs.python.org/file22537/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455 ___

[issue12455] urllib2 forces title() on header names, breaking some requests

Changes by Éric Araujo mer...@netwok.org: -- nosy: +eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455 ___ ___ Python-bugs-list

[issue12455] urllib2 forces title() on header names, breaking some requests

Senthil Kumaran sent...@uthcode.com added the comment: AFAIR, that capitalize part is somewhere a requirement in RFC, if the server did not behave in proper manner, it may not be a good idea for the client to change (or be permissive the flag). -- nosy: +orsenthil

[issue12455] urllib2 forces title() on header names, breaking some requests

Senthil Kumaran sent...@uthcode.com added the comment: Sorry, not Capitalize, but the Title part. One can some bugs which lead to this change in the urllib2. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455

[issue12455] urllib2 forces title() on header names, breaking some requests

R. David Murray rdmur...@bitdance.com added the comment: Quoting http://tools.ietf.org/html/rfc2068#section-4.2: Field names are case-insensitive. Which is only logical, since they are modeled on email headers, and email header names are case insensitive. So, the server in question is

[issue12455] urllib2 forces title() on header names, breaking some requests

Cal Leeming cal.leem...@simplicitymedialtd.co.uk added the comment: Thats full understandable that the default won't change. I'll put this in my todo list to write a patch in a week or two. On 1 Jul 2011 08:45, R. David Murray rep...@bugs.python.org wrote: R. David Murray rdmur...@bitdance.com

[issue12455] urllib2 forces title() on header names, breaking some requests

R. David Murray rdmur...@bitdance.com added the comment: A feature request for a way to control this is reasonable. However, new features can only go into 3.3. -- stage: - needs patch title: urllib2 Request() forces capitalize() on header names, breaking some requests - urllib2

[issue12455] urllib2 forces title() on header names, breaking some requests

Cal Leeming cal.leem...@simplicitymedialtd.co.uk added the comment: Damn 3.3 huh? Ah well, at least it's in the pipeline ^_^ Thanks for your help on this @r.david.murray! -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455

[issue12455] urllib2 forces title() on header names, breaking some requests

Changes by Santoso Wijaya santoso.wij...@gmail.com: -- nosy: +santa4nt ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue12455 ___ ___