Skip Menu |

This queue is for tickets about the Catalyst-Authentication-Store-LDAP CPAN distribution.

Report information
The Basics
Id: 91831
Status: new
Priority: 0/
Queue: Catalyst-Authentication-Store-LDAP

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

Bug Information
Severity: Normal
Broken in: 1.014
Fixed in: (no value)



Subject: Undef warnings if false id and no username
At line 145 here: https://metacpan.org/source/BOBTFISH/Catalyst-Authentication-Store-LDAP-1.014/lib/Catalyst/Authentication/Store/LDAP/Backend.pm#L143 If $authinfo->{id} is false (like if the user submitted without providing a login) and $authinfo->{username} is undef (because it wasn't passed), then undef will be passed in to $self->get_user. This then causes warnings for regex operations on the undef value. Instead of ||, maybe // should be used (if you use that recent syntax in the library)? (I worked around this by passing in the username key instead of the id key) /J