#475 Error with LDAP authentication - ?typo in AuthLdap.php

[Elistia]

Just upgraded from v1.8 to v1.9 and was unable to authenticate using LDAP (no config change).

I think there may be a typo in lib\MRBS\Auth\AuthLdap.php on line 584:

$filter = "(" . self::$all_ldap_opts['ldap_dn_search_attrib'][$idx] . "=$user)";

Debug messages show that $user is empty.

Changed to the following and it works.

$filter = "(" . self::$all_ldap_opts['ldap_dn_search_attrib'][$idx] . "=$username)";

Had problems trying to get the errors to log, but will put that issue in a seperate ticket :-)

Thanks.

[Campbell Morrison]

Thanks. Now fixed in bc2920.

[Elistia]

Great - thanks :-)

Logged the issue I had tracking down the error here https://sourceforge.net/p/mrbs/support-requests/2132/

[Anonymous]

I have spent the best part of the day trying to get ldap to work on a clean install of 1.9- this fixed it. Thank you!

[Anonymous]

Did an upgrade last night (01/09/2020) from v1.5 to v1.9 and LDAP login to AD was the only thing which wasn't working... thanks for the fix.

[Campbell Morrison]

Fixed in MRBS 1.9.1.