Skip Menu |

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

Report information
The Basics
Id: 73259
Status: rejected
Priority: 0/
Queue: perl-ldap
People
Owner: Nobody in particular
Requestors: bitcard [...] davel.me.uk
bobtfish [...] bobtfish.net
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in:
  • 0.39
  • 0.43
Fixed in: (no value)

History Show all quoted text
  Wed Dec 14 11:58:03 2011 bitcard [...] davel.me.uk - Ticket created
Subject: Unicode behaviour not consistent in escape_dn_value
I have written a patch with unit test, pushed to here, https://github.com/gbarr/perl-ldap/pull/5 Kind regards, Dave Lambley
  Mon Dec 19 06:03:55 2011 bobtfish [...] bobtfish.net - Requestor BOBTFISH added
  Wed Dec 05 15:16:49 2012 MARSCHAP [...] cpan.org - Status changed from 'new' to 'open'
  Mon Nov 04 19:33:03 2019 ether [...] cpan.org - Correspondence added
On 2011-12-14 08:58:03, bitcard@davel.me.uk wrote: Show quoted text
> I have written a patch with unit test, pushed to here, > > https://github.com/gbarr/perl-ldap/pull/5 > > Kind regards, > Dave Lambley
I see the fix is actually at https://github.com/perl-ldap/perl-ldap/pull/5.
  Tue Feb 18 17:38:32 2020 DLAMBLEY [...] cpan.org - Correspondence added
On Mon Nov 04 19:33:03 2019, ETHER wrote: Show quoted text
> I see the fix is actually at https://github.com/perl-ldap/perl-ldap/pull/5.
I have re-merged the branch with current master. It should cleanly merge again.
  Sat Feb 22 08:20:07 2020 MARSCHAP [...] cpan.org - Correspondence added
Hi, thanks for the pull equest, but I will not merge it. Encoding to/from UTF8 is not considered part of what escale_dn_value()/unescape_dn_value() is supposed to do. These functions expect resp. return a string in UTF8, and only do the escaping/unescaping necessary for DN values. Making sure the the strings are in the proper encoding is outside perl-ldap's realm, and should be done in the application using perl-ldap. Doing blind encoding-conversion from to UTF8 hard-hoces some expectations (like e.g. "encoding is latin1", ...) into perl-ldap that break perl-ldap for many users. E.g. I run Linux with a UTF8-based locale. Doing Encode::encode("utf-8", $_) in my environment breaks escale_dn_value.
  Sat Feb 22 08:20:47 2020 MARSCHAP [...] cpan.org - Status changed from 'open' to 'rejected'