LibreBooking / app

Repository for the last open source version of Booked Scheduler. The "develop" branch contains the most current working code of the project and should be considered beta. The "master" branch is the most current stable release of BookedScheduler. Please read doc/README.md for further details.
GNU General Public License v3.0
399 stars 238 forks source link

LDAP authentication not working #166

Open sibelle-labs opened 1 year ago

sibelle-labs commented 1 year ago

Hello, I am unable to get LDAP working despite configuring. I have ldap.debug.enabled turned to True but do not see enough debug output. I am using the latest version of Booked Scheduler and PHP 8.1.8 + Oracle Linux 9 with php-ldap installed.

[1027] DEBUG default - [User= ()] Searching ldap for user testing [File=/var/www/html/librebooking/plugins/Authentication/Ldap/Ldap2Wrapper.php,Line=118] [1027] DEBUG default - [User= ()] Search result count: 1 [File=/var/www/html/librebooking/plugins/Authentication/Ldap/Ldap2Wrapper.php,Line=126] [1027] DEBUG default - [User= ()] Search result dn: [File=/var/www/html/librebooking/plugins/Authentication/Ldap/Ldap2Wrapper.php,Line=128] [1027] ERROR default - [User= ()] Current result is false [File=/var/www/html/librebooking/plugins/Authentication/Ldap/Ldap2Wrapper.php,Line=130] [1027] ERROR default - [User= ()] Could not find user testing [File=/var/www/html/librebooking/plugins/Authentication/Ldap/Ldap2Wrapper.php,Line=169] [1027] DEBUG default - [User= ()] Result of LDAP Authenticate for user testing: 0 [File=/var/www/html/librebooking/plugins/Authentication/Ldap/Ldap.php,Line=118]

Any hints or config examples ? Thanks a lot

colisee commented 1 year ago

Hi @sibelle-labs ,

I believe that php 8 broke the PEAR package Net_LDAP2 used by librebooking. I saw that a PR was pushed on the Net-LDAP2 repository, but not merged, yet...

Henceforth, I am afraid that ldap is unavailable on librebooking version 2.8.6 as long as the PEAR package is not updated for php 8...

TheHLee commented 1 year ago

Can confirm I'm having the same problem with LDAP:

colisee commented 1 year ago

Hi @effgarces ,

I just created a PR #235 that should make the ldap plugin compatible with php-8.0 and php-8.1