testUri underscore fix
6 views
Skip to first unread message
Nick Ramsay
Sep 27, 2009, 1:33:33 PM9/27/09
to inspekt
There needs to be an underscore in the isUri regex in Inspekt.php,
otherwise urls with underscores don't parse. Here's the fixed path
line:
regex .= '(/((%[0-9a-f]{2}|[-_a-z0-9/~;:@=+$,.!*()\'\&]*)
*)/?)?'; // path
Most BBC urls have an underscore, so they're good for testing. Here's
one:
http://news.bbc.co.uk/2/hi/middle_east/8277040.stm
Nick
HotaruCMS.org
otherwise urls with underscores don't parse. Here's the fixed path
line:
regex .= '(/((%[0-9a-f]{2}|[-_a-z0-9/~;:@=+$,.!*()\'\&]*)
*)/?)?'; // path
Most BBC urls have an underscore, so they're good for testing. Here's
one:
http://news.bbc.co.uk/2/hi/middle_east/8277040.stm
Nick
HotaruCMS.org
Ed Finkler
Oct 5, 2009, 11:55:00 PM10/5/09
to ins...@googlegroups.com
I've pushed this fix and added a unit test for it
--
Ed Finkler
http://funkatron.com
Twitter:@funkatron
AIM: funka7ron
ICQ: 3922133
XMPP:funk...@gmail.com
--
Ed Finkler
http://funkatron.com
Twitter:@funkatron
AIM: funka7ron
ICQ: 3922133
XMPP:funk...@gmail.com
Reply all
Reply to author
Forward
0 new messages