?
avatar arraylabs
arraylabs
19 Sep 2017

Steps to reproduce the issue

Configure LDAP, try to login with LDAP user

Expected result

LDAP to authenticate properly, currently failing with line 119 of ldap.php file.

Actual result

Currently failing with line 119 of ldap.php file, specifically the $ldap->escape method that upon further inspection does not exist in the LdapClient.php file.

System information (as much as possible)

LAMP, php 6.x, J3.8

Additional comments

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar arraylabs arraylabs - open - 19 Sep 2017
avatar joomla-cms-bot joomla-cms-bot - change - 19 Sep 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 Sep 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Sep 2017
Category Authentication
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 Sep 2017
Priority Medium Critical
avatar mbabker
mbabker - comment - 19 Sep 2017

Replace the file with https://github.com/joomla-framework/ldap/blob/1.2.0/src/LdapClient.php

Apparently some pull request updating the file didn't get applied (or submitted?).

avatar mbabker
mbabker - comment - 19 Sep 2017

d35924e fixes this. Will be included in 3.8.1.

Leaving this open for a little bit just so we don't get a plethora of similar opened issues.

avatar arraylabs
arraylabs - comment - 19 Sep 2017

Appreciate it, thanks!!

avatar brianteeman
brianteeman - comment - 19 Sep 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Sep 2017
Status New Discussion
avatar kdhoe1
kdhoe1 - comment - 21 Sep 2017

This sollution doesn't work with registered users, only super users can login
regards

avatar mbabker
mbabker - comment - 21 Sep 2017

@kdhoe1 not knowing anything more about your problem I can say with certainty that your problem is not related to this issue and you should file a separate report with appropriate details.

avatar kdhoe1
kdhoe1 - comment - 21 Sep 2017

before update to version 3.8.0 no issues during logon of all users through ldap in frontend.
after update: 0 - Call to undefined method Joomla\Ldap\LdapClient::escape() for all users
when replacing ldap file, only super users can login, regular users get: 0 - Class 'Joomla\Ldap\LdapClient' not found
???

avatar mbabker
mbabker - comment - 21 Sep 2017

Then you have not replaced the file correctly as the file contains that class.

avatar kdhoe1
kdhoe1 - comment - 21 Sep 2017

just deleted the file on the server and copied the new file
is there another way?

regards

avatar mbabker
mbabker - comment - 21 Sep 2017

If you downloaded the HTML page and used that, then that would break it. Make sure you hit the "raw" button on the page so you get the actual PHP file that should be uploaded without any extra HTML. Additionally, the file must be uploaded with the correct casing in the name otherwise the file loader won't pick it up.

avatar kdhoe1
kdhoe1 - comment - 21 Sep 2017

OK, Big thanks, it works

Regards

Kris

avatar mbabker mbabker - change - 21 Sep 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-09-21 14:28:34
Closed_By mbabker
avatar mbabker mbabker - close - 21 Sep 2017
avatar EHJ-52n
EHJ-52n - comment - 22 Sep 2017

We had the same error and the solution with replacing the libraries/vendor/joomla/ldap/src/LdapClient.php with the new version solved the issue for us!

avatar gojets
gojets - comment - 27 Sep 2017

Thanks mbabker! The fix works for me.

avatar OlofBailey
OlofBailey - comment - 16 Apr 2019

Getting the same error in 3.9.5

avatar mbabker
mbabker - comment - 16 Apr 2019

@OlofBailey Please don't randomly find 18 month old issues and comment "getting same error on new release". You can't be having the same error as this issue identified as this was an issue of a file not being correctly updated with the 3.8.0 release. If you believe you have found a bug with the new release, the proper action is to file a new issue report with all required details needed to either replicate the issue or explain why it is not a bug in the core package.

Add a Comment

Login with GitHub to post a comment