avoid double prefixes for mimed subjects

Bug #266583 reported by Dk379
2
Affects Status Importance Assigned to Milestone
GNU Mailman
Invalid
Medium
Unassigned

Bug Description

this patch checks subject: line for being mime-encoded
with charset, and if this the case, decodes is for the
purpose of prefix checking.

this avoids replies with subject
encoded( Re: [listname] old subject.. )

to be converted to
[listname] encoded( Re: [listname] old subject...)

ad infinitum.

patch originally developed by vaget at vaget dot org
and beautified by me.

[http://sourceforge.net/tracker/index.php?func=detail&aid=528031&group_id=103&atid=300103]

Revision history for this message
Dk379 (dk379) wrote :
Revision history for this message
Loewis (loewis) wrote :
Revision history for this message
Mikhail Zabaluev (mzabaluev) wrote :

This patch does the job, although EncWord functionality is
somewhat moot: it assumes that any string is encoded using
at most one non-ASCII charset. Which _might_ be wrong.
Decoding into a Unicode string with regard to charsets would
be cleaner, but OTOH it would require codec support and
hence would bring additional complexity and trigger more
untrampled bugs.

Revision history for this message
Barry Warsaw (barry) wrote :

Since this patch is against MM2.0, I'm going to reject it.
Security patches only now for MM2.0.x.

On a related note, CookHeaders.py in MM2.1 should have
enough smarts to avoid adding the prefix to encoded headers,
although it could miss prefixes which contain spaces and
where each word is in a separate encoded chunk. I think in
practice this is unlikely.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.