[request-tracker-maintainers] Bug#1094406: Bug in LDAP.pm module

[Rudy Godoy]

Package: request-tracker5
Version: 5.0.3+dfsg-3~deb12u2


The LDAP module has a bug in the stable version, and as per Andrew's 
message[1], also in Sid.

Below is my proposed patch. I know there is also another patch, but my 
reasoning is that the purpose of the debug logging is to display the 
values the user has configured, so that he can figure out quickly what 
is wrong in his configuration.

The $search_filter, which is an Net::LDAP::Filter's object, of course 
has the as_string method. Unlike the variable $filter, which value is 
what the user has defined.


1- 
https://forum.bestpractical.com/t/samba-ldap-y-rt-5/40705/4?u=rudygodoy


--- LDAP-orig.pm	2025-01-25 17:32:21.447008081 -0500
+++ LDAP.pm	2025-01-25 17:44:39.383008081 -0500
@@ -622,7 +622,7 @@

          unless ( $user_found->code == LDAP_SUCCESS || $user_found->code 
== LDAP_PARTIAL_RESULTS ) {
              $RT::Logger->debug( "search for",
-                                $filter->as_string,
+                                $filter,
                                  "failed:",
                                  ldap_error_name($user_found->code),
                                  $user_found->code );