[Bug 616] New - links missing common operators

bugs@bugs.horde.org bugs@bugs.horde.org
Mon, 12 Feb 2001 18:37:43 -0400


http://bugs.horde.org/show_bug.cgi?id=616

*** shadow/616	Mon Feb 12 18:37:43 2001
--- shadow/616.tmp.17969	Mon Feb 12 18:37:43 2001
***************
*** 0 ****
--- 1,32 ----
+ Bug#: 616
+ Product: Horde
+ Version: 2.2 Stable
+ Platform: PHP Code
+ OS/Version: All
+ Status: NEW   
+ Resolution: 
+ Severity: normal
+ Priority: P2
+ Component: IMP
+ Area: BUILD
+ AssignedTo: chuck@horde.org                            
+ ReportedBy: bill@parducci.net               
+ URL: 
+ Summary: links missing common operators
+ 
+ the routine used to generate hot links in e-mail messages does not handle 
+ URL strings with common operators properly. for instance, mail the following url 
+ to yourself:
+ 
+ http://whois.nic.or.kr/cgi-bin/newwhois.cgi?VALUE=216.244.174.242&TYPE=++
+ 
+ note that the '++' at the end is not included in the link.
+ 
+ a *quick look* makes me think the culprit is here in the replace: 
+ 
+  $tmsg = preg_replace('|(\w+)://([^\s"<]*)([\w#?/&=])|', '<A href="\1://\2\3" 
+ target="_blank">\1://\2\3</A>', $tmsg);
+ 
+ (imp/message.php3" line 113)
+ 
+ b