[ mailman-Bugs-1242450 ] Scrubber URLs not properly delimited

[SourceForge.net]

Bugs item #1242450, was opened at 2005-07-21 10:18
Message generated for change (Comment added) made by msapiro
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1242450&group_id=103

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: mail delivery
Group: 2.1 (stable)
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Ian Eiloart (ianeiloart)
Assigned to: Nobody/Anonymous (nobody)
Summary: Scrubber URLs not properly delimited

Initial Comment:
The URLs inserted by the scrubber are not properly delimited. I've 
seen instances where Apple Mail tried to fetch a URL like this, for 
example:

http://mail.sussex.ac.uk/mailman/private/iant-test5/attachments/
20050721/3533d8c6/
uscsbullet.png____________________________

Where the underscores came from the list signature.

Wrapping the URLs in angle brackets, in accordance with the 
recommendation of  the appendix of http://www.faqs.org/rfcs/
rfc1738.html fixes this problem. 

Here's a diff (-e) of Mailman/Handlers/Scrubber.py which seems to 
do the job:

313c
Url : <%(url)s>
.
280c
Url: <%(url)s>
.
259c
URL: <%(url)s>
.
233c
URL: <%(url)s>
.
205c
Url: <%(url)s>
.




----------------------------------------------------------------------

>Comment By: Mark Sapiro (msapiro)
Date: 2007-11-12 15:28

Message:
Logged In: YES 
user_id=1123998
Originator: NO

Fixed for 2.1.10.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=1242450&group_id=103