[ mailman-Bugs-759841 ] Multipart/mixed issues in archives

Bugs item #759841, was opened at 2003-06-24 14:22 Message generated for change (Comment added) made by tkikuchi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=759841&...
Category: Pipermail Group: 2.1 (stable) Status: Open Resolution: None
Priority: 8
Submitted By: Pug Bainter (phelim_gervase) Assigned to: Nobody/Anonymous (nobody) Summary: Multipart/mixed issues in archives
Initial Comment: We are having problems with mailing lists that are not being properly stripped down to text content in the archives. When there is multipart/mixed, it doesn't pull the multipart/alternative sections into their appropriate text portions.
For example, from content such as the following:
==============================================================================
From ...
[...] Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: multipart/mixed; boundary=------------InterScan_NT_MIME_Boundary [...]
This is a multi-part message in MIME format.
--------------InterScan_NT_MIME_Boundary Content-Type: multipart/alternative; boundary="----_=_NextPart_001_01C336A1.2C7564BC" Content-Transfer-Encoding: 7bit
------_=_NextPart_001_01C336A1.2C7564BC Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: quoted-printable
Kevin has a pending checkin that addresses the minss/maxss issue. =20 [...] ------_=_NextPart_001_01C336A1.2C7564BC Content-Type: text/html; charset=us-ascii Content-Transfer-Encoding: quoted-printable
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN"> <HTML xmlns=3D"http://www.w3.org/TR/REC-html40" xmlns:v =3D=20 "urn:schemas-microsoft-com:vml" xmlns:o =3D=20 "urn:schemas-microsoft-com:office:office" xmlns:w =3D=20 "urn:schemas-microsoft-com:office:word" xmlns:x =3D=20 "urn:schemas-microsoft-com:office:excel" xmlns:st1 =3D=20 "urn:schemas-microsoft-com:office:smarttags"><HEAD><TITLE>Message</TITLE>=
[...] ==============================================================================
I only get the following:
============================================================================== [64bit-compiler-analysis] RE: vpr analysis Syyyy Kyyyyy syyyk at yyy.com Thu Jun 19 14:27:16 CDT 2003
Previous message: [64bit-compiler-analysis] 06-19-03 MSFT 64-Bit C/C++ compiler +improvement discussion Next message: [64bit-compiler-analysis] RE: vpr analysis Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
--------------------------------------------------------------------------------
Skipped content of type multipart/alternative
--------------------------------------------------------------------------------
Previous message: [64bit-compiler-analysis] 06-19-03 MSFT 64-Bit C/C++ compiler +improvement discussion Next message: [64bit-compiler-analysis] RE: vpr analysis Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
-------------------------------------------------------------------------------- More information about the 64bit-compiler-analysis mailing list ==============================================================================
As you can see, the actual content of the multipart/alternative portion [text/plain and text/html] were completely stripped out instead of being shown a plain text.
----------------------------------------------------------------------
Comment By: Tokio Kikuchi (tkikuchi)
Date: 2003-12-28 01:17
Message: Logged In: YES user_id=67709
The patch by q7joey is merged into my Scrubber.py patch #866238. I hope Barry can integrate it in 2.1.4.
----------------------------------------------------------------------
Comment By: Joe Pruett (q7joey) Date: 2003-09-27 16:48
Message: Logged In: YES user_id=559223
i have a few line patch that seems to make it do what is expected.
i can't see how to attach via sourceforge yet, so i'll paste it here:
--- /usr/local/src/mailman-2.1.2/Mailman/Handlers/Scrubber.py Fri Feb 7 23:13:50 2003 +++ ./Scrubber.py Sat Sep 27 08:58:46 2003 @@ -286,11 +286,13 @@ # BAW: Martin's original patch suggested we might want to try # generalizing to utf-8, and that's probably a good idea (eventually). text = [] - for part in msg.get_payload(): + for part in msg.walk(): + if part.get_main_type() == 'multipart': + continue # All parts should be scrubbed to text/plain by now. partctype = part.get_content_type() if partctype <> 'text/plain': - text.append(_('Skipped content of type %(partctype)s')) + text.append(_('Skipped content of type %(partctype)s\n')) continue try: t = part.get_payload(decode=1)
----------------------------------------------------------------------
Comment By: Martin RJ. Cleaver (mrjc) Date: 2003-09-27 07:23
Message: Logged In: YES user_id=50125
This fails for many of my users as they habitually attach a photo of themselves in their signatures. They are incredulous at the idea that mailman can't handle it.
Thanks
----------------------------------------------------------------------
Comment By: Joe Pruett (q7joey) Date: 2003-09-27 01:26
Message: Logged In: YES user_id=559223
i agree that this should be a high priority issue. a simple message with just multipart/alternative will show up in the archive ok, but if there is any other kind of attachment, then the entire multipart section is skipped and you just get a link for the extra attachment for download/view ability. i haven't started to look at the code (and i'm not a python/mailman person), but i'll report anything i can find.
----------------------------------------------------------------------
Comment By: Martin RJ. Cleaver (mrjc) Date: 2003-09-22 13:34
Message: Logged In: YES user_id=50125
Additionally I think it is appropriate to up the priority on this bug as it causes key functionality to fail.
----------------------------------------------------------------------
Comment By: Martin RJ. Cleaver (mrjc) Date: 2003-09-22 13:26
Message: Logged In: YES user_id=50125
This is causing me real problems! Is there any known workarounds?
If I can't fix this I might have to use a different package as presently all my archives are useless!
----------------------------------------------------------------------
Comment By: Pug Bainter (phelim_gervase) Date: 2003-06-24 17:01
Message: Logged In: YES user_id=484284
This appears to be within:
def process(mlist, msg, msgdata=None):
at around line 276, but I saw no way of making it recurse for multipart/[mixed|alternative] sub-MIME parts.
----------------------------------------------------------------------
You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=100103&aid=759841&...
participants (1)
-
SourceForge.net