Skip Menu |

This queue is for tickets about the Dancer-Plugin-Database-Core CPAN distribution.

Report information
The Basics
Id: 101180
Status: patched
Priority: 0/
Queue: Dancer-Plugin-Database-Core

People
Owner: Nobody in particular
Requestors: MRDVT [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.08
Fixed in: (no value)



Subject: DSN regex match needs to be case insensitive
I'm running an older version of Dancer::Plugin::Database but this issue is still in Dancer::Plugin::Database::Core. Warning: Use of uninitialized value $driver in hash element at /usr/share/perl5/vendor_perl/Dancer/Plugin/Database.pm line 177. - ($driver) = $dsn =~ m{dbi:([^:]+)}; + ($driver) = $dsn =~ m{^dbi:([^:]+)}i; Issue is that the regex is not case insensitive but DBI allows mixed case. The documentation at http://search.cpan.org/~timb/DBI-1.632/DBI.pm is actually ambiguous. It says this. Show quoted text
> scheme is the first part of the DSN and is currently always 'dbi'.
but the first example is Show quoted text
> DBI->parse_dsn("DBI:MyDriver(RaiseError=>1):db=test;port=42");
Thanks, Mike mrdvt92
On Mon Dec 29 00:01:06 2014, MRDVT wrote: Show quoted text
> I'm running an older version of Dancer::Plugin::Database but this > issue is still in Dancer::Plugin::Database::Core. > > Warning: Use of uninitialized value $driver in hash element at > /usr/share/perl5/vendor_perl/Dancer/Plugin/Database.pm line 177. > > - ($driver) = $dsn =~ m{dbi:([^:]+)}; > + ($driver) = $dsn =~ m{^dbi:([^:]+)}i; > > Issue is that the regex is not case insensitive but DBI allows mixed > case. > > The documentation at http://search.cpan.org/~timb/DBI-1.632/DBI.pm is > actually ambiguous. > > It says this. >
> > scheme is the first part of the DSN and is currently always 'dbi'.
> > but the first example is >
> > DBI->parse_dsn("DBI:MyDriver(RaiseError=>1):db=test;port=42");
> > Than
Fixed in GIT. Hope to release soon. Thank you Alberto