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

[SourceForge.net]

Bugs item #1242450, was opened at 2005-07-21 18:18
Message generated for change (Tracker Item Submitted) made by Item Submitter
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: Open
Resolution: None
Priority: 5
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>
.




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

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