Bug #52309 for Regexp-Common: URI rules rejects many example URIs from rfc3986, and hangs on news: example

Tue Dec 01 01:53:44 2009 SAMV [...] cpan.org - Ticket created

CC:	sam [...] vilain.net
Subject:	URI rules rejects many example URIs from rfc3986, and hangs on news: example

Silly XML Schema tends to use URNs, like "urn:oasis:names:specification:docbook:dtd:xml:4.1.2". In principle, 'scheme:opaque' matches an awful lot of rubbish that probably isn't a valid URI, but "urn" is probably worth adding. I checked a few of the ones in section 1.1.2 of RFC3986, and also found that the example 'ldap' and 'mailto' URIs also failed to validate, and the 'news' one never returned! To reproduce: perl -MRegexp::Common=URI -le '(print("$_:".(m{$RE{URI}}?"ok":"fail"))) for qw(ftp://ftp.is.co.za/rfc/rfc1808.txt http://www.ietf.org/rfc/rfc2396.txt ldap://[2001:db8::7]/c=GB?objectClass?one mailto:John.Doe@example.com tel:+1-816-555-1212 telnet://192.0.2.16:80/ urn:oasis:names:specification:docbook:dtd:xml:4.1.2 news:comp.infosystems.www.servers.unix )'

Mon Dec 28 22:13:17 2009 cpan [...] abigail.be - Correspondence added

On Tue Dec 01 01:53:44 2009, SAMV wrote:
Show quoted text

> Silly XML Schema tends to use URNs, like
> "urn:oasis:names:specification:docbook:dtd:xml:4.1.2". In principle,
> 'scheme:opaque' matches an awful lot of rubbish that probably isn't a
> valid URI, but "urn" is probably worth adding.
>
> I checked a few of the ones in section 1.1.2 of RFC3986, and also found
> that the example 'ldap' and 'mailto' URIs also failed to validate, and
> the 'news' one never returned!
>
> To reproduce:
>
> perl -MRegexp::Common=URI -le '(print("$_:".(m{$RE{URI}}?"ok":"fail"))) for
> qw(ftp://ftp.is.co.za/rfc/rfc1808.txt
> http://www.ietf.org/rfc/rfc2396.txt
> ldap://[2001:db8::7]/c=GB?objectClass?one
> mailto:John.Doe@example.com
> tel:+1-816-555-1212
> telnet://192.0.2.16:80/
> urn:oasis:names:specification:docbook:dtd:xml:4.1.2
> news:comp.infosystems.www.servers.unix
> )'

The 'ldap', 'mailto' and 'urn' schemes aren't supported. It's unlikely they ever will in Regexp::Common - but they maybe in its next generation (Regexp::Common510 - not yet released).

Mon Dec 28 22:13:18 2009 The RT System itself - Status changed from 'new' to 'open'

Tue Dec 29 20:40:58 2009 cpan [...] abigail.be - Correspondence added

The reason the pattern 'hangs' was that it took a long time for regexp engine to determine a certain branch was unsuccesfull. The pattern has been rewritten to fail faster (at the expensive of matching somewhat slower). This should be available in the next release (planned to occur this year)

Tue Dec 29 20:41:01 2009 cpan [...] abigail.be - Status changed from 'open' to 'resolved'