[issue11492] email.header.Header doesn't fold headers correctly

STINNER Victor added the comment: I wrote PR 10378 to show that I don't think that this bug must be fixed in Python 2: it would break any application relying on the current folding algorithm. -- nosy: +vstinner ___ Python tracker

[issue11492] email.header.Header doesn't fold headers correctly

Change by STINNER Victor : -- pull_requests: +9680 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue11492] email.header.Header doesn't fold headers correctly

Changes by Srikanth S sriks...@gmail.com: -- nosy: +srikanths ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11492 ___ ___ Python-bugs-list mailing

[issue11492] email.header.Header doesn't fold headers correctly

Roundup Robot devnull@devnull added the comment: New changeset 51a551acb60d by R David Murray in branch '3.2': #11492: rewrite header folding algorithm. Less code, more passing tests. http://hg.python.org/cpython/rev/51a551acb60d New changeset fcd20a565b95 by R David Murray in branch

[issue11492] email.header.Header doesn't fold headers correctly

Changes by R. David Murray rdmur...@bitdance.com: -- resolution: - fixed stage: patch review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11492

[issue11492] email.header.Header doesn't fold headers correctly

R. David Murray rdmur...@bitdance.com added the comment: This was quite the adventure. The more I worked on fixing the tests, the more if/else cases the existing splitting algorithm grew. When I reached the point where fixing one test broke two others, I thought maybe it was time to try a

[issue11492] email.header.Header doesn't fold headers correctly

R. David Murray rdmur...@bitdance.com added the comment: Note that this fix solves issue 11772, so I've closed that one as a duplicate. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11492

[issue11492] email.header.Header doesn't fold headers correctly

R. David Murray rdmur...@bitdance.com added the comment: Here is a patch containing three test cases that demonstrate three different failings of the header folding algorithm. I'm working on the fix, but it is non-trivial. -- components: +Library (Lib) -None keywords: +patch title:

[issue11492] email.header.Header doesn't fold headers correctly

Changes by R. David Murray rdmur...@bitdance.com: Removed file: http://bugs.python.org/file21563/header_folding_tests.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11492 ___

[issue11492] email.header.Header doesn't fold headers correctly

R. David Murray rdmur...@bitdance.com added the comment: Note that 2.7 fails two of these tests as well, but for different reasons. I'm not currently planning to fix 2.7, as its behavior at least (a) doesn't lose non-whitespace information and (b) doesn't exceed the maxheaderlen. --

[issue11492] email.header.Header doesn't fold headers correctly

Changes by R. David Murray rdmur...@bitdance.com: Removed file: http://bugs.python.org/file21564/header_folding_tests.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11492 ___

[issue11492] email.header.Header doesn't fold headers correctly

R. David Murray rdmur...@bitdance.com added the comment: Here is an updated test patch that brings the test coverage of the relevant code much closer to 100%. There are still three lines and one branch uncovered, but it appears as though one of the bugs is preventing the test case that would

[issue11492] email.header.Header doesn't fold headers correctly

Roundup Robot devnull@devnull added the comment: New changeset 10725fc76e11 by R David Murray in branch '3.1': #11492: fix header truncation on folding when there are runs of split chars. http://hg.python.org/cpython/rev/10725fc76e11 New changeset 74ec64dc3538 by R David Murray in branch '3.2':