[wp-trac] [WordPress Trac] #5421: Commas within urls dont get parsed correctly (Pingback)

[WordPress Trac]

#5421: Commas within urls dont get parsed correctly (Pingback)
---------------------+------------------------------------------------------
 Reporter:  torty    |       Owner:  anonymous
     Type:  defect   |      Status:  new      
 Priority:  normal   |   Milestone:  2.3.2    
Component:  General  |     Version:  2.3.1    
 Severity:  normal   |    Keywords:           
---------------------+------------------------------------------------------
 Hi,

 Wordpress does not extract urls with a "," like
 http://www.myWebsite.de/tests/digicam/0,39030924,39159407,00/enorm+anpassungsfaehig+casio+exilim+zoom+ex_z1080+video.htm

 Returns as URL : http://www.cnet.de/tests/digicam/0

 After looking at the code i think we found the problem:

 the do_enclose function uses this regular expression:

 $ltrs = '\w';
 $gunk = '/#~:.?+=&%@!\-';
 $punc = '.:?\-';
 $any = $ltrs . $gunk . $punc;

 preg_match_all("{\b http : [$any] +? (?= [$punc] * [^$any] | $)}x",
 $content, $post_links_temp);

 which is missing ',' at all.
 As far as we found out commas are allowed within urls.
 See also: http://gbiv.com/protocols/uri/rfc/rfc3986.html#examples

 Maybe sombody could fix that so that sites with 'comma'-urls could get
 pinged also.

 Thanks

 Torsten

-- 
Ticket URL: <http://trac.wordpress.org/ticket/5421>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software