[issue1372770] email.Header should preserve original FWS
Changes by Srikanth S sriks...@gmail.com: -- nosy: +srikanths ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
R. David Murray rdmur...@bitdance.com added the comment: As of the fix for issue 11492, the email package only uses continuation_ws when folding RFC2047 encoded words. So I consider this issue fixed. (I have, by the way, come around to the view that we should never be introducing or deleting whitespace except when RFC 2047 encoding/decoding...we are still deleting it in a couple places, and I will address that by and by). -- components: +Library (Lib) -Extension Modules resolution: - duplicate stage: test needed - committed/rejected status: open - closed superseder: - email.header.Header doesn't fold headers correctly ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
R. David Murray rdmur...@bitdance.com added the comment: I agree that when dealing with prexisting folding it is better to preserve it. The case I was talking about is, say, prepending re to a subject and refolding it. It is the transformation step where I think turning fws into a single space makes sense. But I absolutly want this to be controllable and hope that experience can inform the chosen defaults. I also expect to be getting feedback from the mailman folks well before anything gets set in stone. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
Nathan Herring nathan.herr...@technosloth.com added the comment: You are certainly correct about (some, perhaps many) e-mail generators using tabs when folding, which is AFAICT, much more likely an incorrect implementation of RFC 2822 rather than an intentional transformation of the user's specified Subject line. Some*, however, dutifully unfold those tabs back into the Subject line, causing all sorts of strangeness -- explosion of conversations that are logically identical, except that in one, it's all spaces, and in the others, some of the spaces have, seemingly arbitrarily, turned into tabs. Fixing all of the e-mail generators to Do The Right Thing wasn't what I had in mind, but making sure that mailman, a rather commonly used mail service, and its reliance on python's Header class, would no longer permute the messages made my e-mail generators that do Do The Right Thing, was one step. I would be inclined to, were I a contributor, make it so that it would conform to RFC 2822 unfolding and preserve the FWS accordingly. I don't know if there'd be enough desire for your alternative, but if there were, I'd make it a non-default option to de-tabify Subject lines (or other headers). As it stands, I am newly freed from any restrictions from contributing, and so I might try and see what mailman's intentions are surrounding taking updates to python before trying to propose a patch that they might never use (i.e., they may just take the version you're working on now.) *Microsoft Entourage (which I worked on while at Microsoft) and Microsoft Outlook both behave correctly, as far as I could determine in '05, with regards to the folding/unfolding headers. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
Changes by Terry J. Reedy tjre...@udel.edu: -- nosy: -terry.reedy ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
R. David Murray rdmur...@bitdance.com added the comment: Header parsing and formatting is one of the major changes in email6, and it should handle this much more sensibly. email6 won't land until 3.3, though it will be available on pypy for testing before that. If you want propose a patch for fixing this in the current email module I will consider it (but see below). Absent that, I'm marking this for email6/3.3. Note that in email6 the primary interface for creating headers will be a factory function, and in the current design it will reject header values that contain /n (and/or /r). The parser will deal with unfolding values when parsing existing messages. Also, while I agree with you about what the RFC *says*, what email programs actually *do* seems to be a bit different. Email generators in general use a single leading tab as folding whitespace, but if you unfold the resulting value, it is clear that the tabs are noise, and should be replaced by single spaces on unfolding. This becomes obvious when you consider things like an unstructured Subject header that has been wrapped. It will be wrapped with tabs by any mailer I've so far encountered, but if you unwrap it, add a 'Re:', and rewrap it, preserving the tab is clearly the wrong thing to do. This is in fact what email used to do, and this has annoyed many many people over the years (including me) because the header in the reply message has this tab stuck in the middle of the subject... So currently my plan is to special case tabs on folding and unfolding. When unfolding single leading tabs will become blanks, when folding a single tab will be used as folding white space, replacing single blanks at the point of folding. I haven't tested this algorithm on any other mailers yet, because I haven't got enough of the code finished yet to generate parseable messages. (Maybe I'll do some by hand.) This folding policy will be a controllable policy setting, so it will be possible to produce the strictly-RFC-conformant folding and unfolding on a per-message or even (when creating them) a per-header basis. I welcome your thoughts on this subject (and if you are so moved your participation on the email-sig, which while it is pretty quiet right now will probably get less so soon when I post the next API iteration for email6). -- versions: +Python 3.3 -Python 3.2 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
Terry J. Reedy tjre...@udel.edu added the comment: The email module has several problems. RDM is working on overhauling the email module for 3.2. Existing issues may not get individual attention. -- nosy: +terry.reedy versions: +Python 3.2 -Python 2.6, Python 3.0 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
Changes by Barry A. Warsaw ba...@python.org: -- assignee: barry - r.david.murray nosy: +r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
Changes by Daniel Diniz aja...@gmail.com: -- keywords: +easy ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue1372770] email.Header should preserve original FWS
Changes by Daniel Diniz aja...@gmail.com: -- nosy: +ajaksu2 stage: - test needed type: - behavior versions: +Python 2.6, Python 3.0 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1372770 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
