This patch fixes the login for me - thanks a lot.

Besides the one complaint I think this can go in. I didn't investigate the rules in detail, but it's standalone enough that pushing it seems fairly risk free.

Maybe something like this would help:

diff --git a/lib/Auth/LDAP.php b/lib/Auth/LDAP.php
index f5e33b2..2db826e 100644
--- a/lib/Auth/LDAP.php
+++ b/lib/Auth/LDAP.php
@@ -122,6 +122,12 @@ class LDAP extends Net_LDAP3 {
         $this->config_set("root_dn", $root_dn);
     }

In D5007#62760, @machniak wrote:

I feel uneasy about using binary data in sql queries, without special blob handling, but I guess it should work until we're proven otherwise.

I feel uneasy about using binary data in sql queries, without special blob handling, but I guess it should work until we're proven otherwise.

The net-ldap3 warning was already fixed in https://git.kolab.org/rPNL39a754269fe6e5d94b0fe87215e89232876bda01.

php-net-ldap3 is our main workhorse for ldap connections, php-net-ldap2 is a dependency there for a one or two features in use.

• Itip notifications