Skip Menu |
 

This queue is for tickets about the perl-ldap CPAN distribution.

Report information
The Basics
Id: 119916
Status: rejected
Priority: 0/
Queue: perl-ldap

People
Owner: Nobody in particular
Requestors: michele.r.berg [...] gmail.com
Cc:
AdminCc:

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



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>
Download (untitled) / with headers
text/plain 316b
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
RT-Send-CC: michele.r.berg [...] gmail.com
Download (untitled) / with headers
text/plain 783b
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"


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.