#3833 enhancement closed fixed (fixed)
_newclient does not support bare LF newlines, unlike web browsers
Reported by: | ivank | Owned by: | |
---|---|---|---|
Priority: | low | Milestone: | |
Component: | web | Keywords: | |
Cc: | Branch: |
branches/newclient-bare-lf-3833
branch-diff, diff-cov, branch-cov, buildbot |
|
Author: | jonathanj |
Description
Some servers send broken HTTP responses with only \n to delimit status lines and headers, instead of \r\n. All modern web browsers support this, but _newclient does not.
Originally discovered at http://twistedmatrix.com/trac/ticket/886#comment:23
Change History (12)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
http://www.thinkgeek.com/ also has this defect, as of this posting.
comment:3 Changed 13 years ago by
RFC 2616 section 19.3 Tolerant Applications:
The line terminator for message-header fields is the sequence CRLF. However, we recommend that applications, when parsing such headers, recognize a single LF as a line terminator and ignore the leading CR.
comment:4 Changed 13 years ago by
(via http://news.ycombinator.com/item?id=705095 , which lists other HTTP clients that had to be modified to work with LF)
comment:6 Changed 12 years ago by
http://www.thinkgeek.com/ no longer appears to have this problem.
But a new bad site was discovered: http://hdl.loc.gov/loc.uscongress/legislation.111s1448
(And I can't believe ycombinator is still so broken. Sigh.)
comment:7 Changed 11 years ago by
Keywords: | review added |
---|---|
Owner: | jknight deleted |
The patch on #886 appears to be a more or less complete attempt at fixing this issue. Please review.
comment:8 Changed 11 years ago by
Author: | → jonathanj |
---|---|
Branch: | → branches/newclient-bare-lf-3833 |
(In [30570]) Branching to 'newclient-bare-lf-3833'
comment:9 Changed 11 years ago by
comment:10 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:11 Changed 11 years ago by
comment:12 Changed 6 years ago by
Keywords: | review removed |
---|
[mass edit] Removing the review tag from closed tickets
The only server that was found to do this was http://news.ycombinator.com/ (as of 2009-02). Nothing else has been found so far.