Bug 35222

Summary: [patch] getmsg.cgi: mailing list archive URL regexp suboptimal
Product: Documentation Reporter: Tony Finch <dot>
Component: Books & ArticlesAssignee: freebsd-doc (Nobody) <doc>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   

Description Tony Finch 2002-02-22 22:40:00 UTC
If you look at the following message you will see that the > bracket
after the URL has been included in the href link, so clicking on it
doesn't work.

http://docs.freebsd.org/cgi/getmsg.cgi?fetch=488915+0+current/freebsd-arch
Comment 1 Tony Finch freebsd_committer freebsd_triage 2002-05-15 00:33:37 UTC
I have come up with a patch for this, although I haven't tested it yet...

Index: getmsg.cgi
===================================================================
RCS file: /home/ncvs/www/en/cgi/getmsg.cgi,v
retrieving revision 1.34
diff -u -r1.34 getmsg.cgi
--- getmsg.cgi	19 Mar 2002 00:53:52 -0000	1.34
+++ getmsg.cgi	8 May 2002 20:39:13 -0000
@@ -210,7 +210,7 @@
     my ($text) = @_;
     my $cvsweb = 'http://cvsweb.FreeBSD.org/';
 
-    $text =~ s/(http|https|ftp|gopher|mailto|news|file)(:[^\s]*?\/?)(\W?\s)/<a href="$1$2">$1$2<\/a>$3/goi;
+    $text =~ s/(http|https|ftp|gopher|mailto|news|file)(:[^\s>"]*)/<a href="$1$2">$1$2<\/a>/goi;
 
     if ($text =~ /Revision\s+Changes\s+Path/) {
Comment 2 Wolfram Schneider freebsd_committer freebsd_triage 2007-07-16 19:12:14 UTC
State Changed
From-To: open->closed

fixed, thanks for your report!