Bug #55547 for Regexp-Common: URI regexp should be case insensitive wrt. scheme

Sun Mar 14 05:15:16 2010 scop [...] cpan.org - Ticket created

Subject:

URI regexp should be case insensitive wrt. scheme

The URI regexps seem to demand the URI scheme to be in lowercase, but RFC 2396 says: http://tools.ietf.org/html/rfc2396#section-3.1 "For resiliency, programs interpreting URI should treat upper case letters as equivalent to lower case in scheme names (e.g., allow "HTTP" as well as "http")." ...so I suggest making the scheme matching case insensitive. This is trivial to work around in code using Regexp-Common, but I think the default behavior should Just Work.

Sun Apr 17 10:32:25 2011 cpan [...] abigail.be - Correspondence added

Regexp::Common is to validate things, hence, it's strict. The RFC says lowercase, so the regexp only accepts lowercase.

Sun Apr 17 10:32:27 2011 The RT System itself - Status changed from 'new' to 'open'

Sun Apr 17 10:32:27 2011 cpan [...] abigail.be - Status changed from 'open' to 'rejected'

Sun Apr 17 10:33:27 2011 cpan [...] abigail.be - Correspondence added

Also note, that if you really want lowercase, you can use: $RE {URI} {HTTP} {-scheme => qr/(?i:http)/}

Sun Apr 17 10:33:29 2011 The RT System itself - Status changed from 'rejected' to 'open'

Sun Apr 17 10:33:53 2011 cpan [...] abigail.be - Status changed from 'open' to 'rejected'

Tue Apr 19 15:37:41 2011 scop [...] cpan.org - Correspondence added

On Sun Apr 17 10:32:25 2011, ABIGAIL wrote: Show quoted text

> Regexp::Common is to validate things, hence, it's strict. The RFC says > lowercase, so the regexp only accepts lowercase.

Oh, I think that's a very bad call. The RFC actually says applications *should* accept uppercase and lowercase scheme names and Regexp::Common behaved directly against that by default the last time I checked, so Regexp::Common is imposing additional restrictions over the relevant specification. That's not being strict, nor being "to validate things", it's something that in my opinion would deserve a loud and clear note in the documentation (don't know a good word for it).

Tue Apr 19 15:37:42 2011 The RT System itself - Status changed from 'rejected' to 'open'

Wed Mar 13 10:21:27 2013 cpan [...] abigail.be - Status changed from 'open' to 'rejected'