search

quanah / net-ldapapi

The Net::LDAPapi Perl Module uses the OpenLDAP and Mozilla C api's to directly access and manipulate an LDAP v2 or LDAP v3 server.
8 stars 6 forks source link

Fix cpan report 58270 #37

Open quanah opened 9 years ago

quanah commented 9 years ago

For our next release, we should determine if cpan report 58270 is valid:

https://rt.cpan.org/Ticket/Display.html?id=58270

If so, it should be fixed. It sounds reasonable at first glance.

lashtear commented 9 years ago

Frankly, the code looks correct as-is in trunk. If those diagnostics are troubling, they shouldn't set debug or we should implement a different debug option. In any case, if we do change then it should apply to all operations, not just rename.

A better approach might be to have an is_error test like in Net::LDAP and only emit the error messages in debug for things that are actually an error condition. That'd be something for a new release though, I would think.

quanah commented 9 years ago

Ok, so we can push this to 3.1 or 3.0.5