[Mailman-Users] Attachment URL in the archive is mangled.

[Mark Dale]

Hi Mark

Your patch has worked a treat. Thank you very much for your patience and
generosity. Many of us would be a long way up the proverbial creek if it
weren't for your support.

In summary, for those interested:

Using Mailman 2.1.5 - in the Archives, the URL to attachments was being
malformed by it getting wrapped and an "=" sign inserted at the line break.

This was fixed (albeit lightly tested so far) by replacing, in the
Scrubber.py file, as follows:

--------
OLD CODE
--------
        # Now join the text and set the payload
        sep = _('-------------- next part --------------\n')
        del msg['content-type']
        msg.set_payload(sep.join(text), charset)
        del msg['content-transfer-encoding']
        msg.add_header('Content-Transfer-Encoding', '8bit')
    return msg

--------
NEW CODE
--------
        # Now join the text and set the payload
        sep = _('-------------- next part --------------\n')
        del msg['content-type']
        del msg['content-transfer-encoding']
        msg.set_payload(sep.join(text), charset)
    return msg


cheers
Mark



Mark Sapiro wrote:
> Mark Sapiro wrote:
>> I have been able to duplicate the problem with the 2.1.5 version of
>> Scrubber.py. I'll try to come up with a simple patch that you can
>> apply to fix it. It is fixed in recent Scrubber.py versions.
> 
> 
> I think you'd be better off upgrading your Mailman, but if you want to
> try a patch to the 2.1.5 Scrubber.py, I think this should do it. It is
> only lightly tested, but I think it's OK.
> 
> --- Scrubber.py 2008-11-22 20:21:38.375000000 -0800
> +++ Scrubberx.py        2008-11-22 20:54:47.250000000 -0800
> @@ -326,9 +326,8 @@
>          # Now join the text and set the payload
>          sep = _('-------------- next part --------------\n')
>          del msg['content-type']
> -        msg.set_payload(sep.join(text), charset)
>          del msg['content-transfer-encoding']
> -        msg.add_header('Content-Transfer-Encoding', '8bit')
> +        msg.set_payload(sep.join(text), charset)
>      return msg