Bug #119916 for perl-ldap: Net::LDAP::Entry bug fix

Thu Jan 19 16:29:29 2017 michele.r.berg [...] gmail.com - Ticket created

Subject:	Net::LDAP::Entry bug fix
Date:	Thu, 19 Jan 2017 15:28:29 -0600
To:	bug-perl-ldap [...] rt.cpan.org
From:	Michele Berg <michele.r.berg [...] gmail.com>

This may or may not be something you want to implement on a widespread basis, but I've implemented it locally to avoid getting unnecessary warnings, as $type is not necessary to make LDAP connections: sub get_value { my $self = shift; my $type = shift || ''; $type = lc($type) if $type; ... Thanks, Michele

Sat Jan 21 08:41:15 2017 MARSCHAP [...] cpan.org - Severity Wishlist added

Sat Jan 21 08:50:08 2017 MARSCHAP [...] cpan.org - Correspondence added

RT-Send-CC:

michele.r.berg [...] gmail.com

Hi, On Thu Jan 19 16:29:29 2017, michele.r.berg@gmail.com wrote: Show quoted text

> This may or may not be something you want to implement on a widespread > basis, but I've implemented it locally to avoid getting unnecessary > warnings, as $type is not necessary to make LDAP connections: > > sub get_value { > my $self = shift; > my $type = shift || ''; > $type = lc($type) if $type; > ...

I do not see the reason why anyone may need that change. Please elaborate the cases where you get warnings. get_value() is meant to be called with the name of an attribute. According to RFC 4512 section 1.4, attribute names are non empty: oid = descr / numericoid descr = keystring keystring = leadkeychar *keychar leadkeychar = ALPHA ALPHA = %x41-5A / %x61-7A ; "A"-"Z" / "a"-"z"

Sat Jan 21 08:50:08 2017 The RT System itself - Status changed from 'new' to 'open'

Tue Apr 16 09:51:33 2019 MARSCHAP [...] cpan.org - Status changed from 'open' to 'rejected'