-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]: LDAP user and group backend causing frequent errors as of NC 24 #33622
Comments
This appears to be regarding password policy attributes within the directory. Looking at "apps/user_ldap/lib/User/User.php" at line 672 we find this:
An exception is being thrown here because
In my case
That alone is not enough as it only tries to re-fetch password policy attributes. That too could fail, in which case we need to handle if it does.
After making that amend I can login with my LDAP account via web browser, and NextCloud for Windows immediately changed to a green tick. |
Here's how lines 664-678 should ideally look:
Never submitted a PR before. I'll try that shortly. |
…tory not having password policy attributes available. nextcloud#33622 Signed-off-by: Adam Reece <adam@reece.wales>
Hello, Exception: array_key_exists(): Argument #2 ($array) must be of type array, null given in file '/var/www/html/nextcloud/do.do.fr/apps/user_ldap/lib/User/User.php' line 653 And i request error when i try login. |
I was tearing my hair out trying to find and work out this bug too. As best as I can figure, the unhandled exception was happening in our case because our LDAP server default policy does not include any of the For those that want a quick temporary fix: I added an arbitrary value for |
Hi, please update to 25.0.7 or better 26.0.2 and report back if it fixes the issue. Thank you! My goal is to add a label like e.g. 26-feedback to this ticket of an up-to-date major Nextcloud version where the bug could be reproduced. However this is not going to work without your help. So thanks for all your effort! If you don't manage to reproduce the issue in time and the issue gets closed but you can reproduce the issue afterwards, feel free to create a new bug report with up-to-date information by following this link: https://github.com/nextcloud/server/issues/new?assignees=&labels=bug%2C0.+Needs+triage&template=BUG_REPORT.yml&title=%5BBug%5D%3A+ |
I've not noticed this problem recently. (Currently running NC 27.0.0.) |
I've not noticed this problem recently too. |
Bug description
Good afternoon,
The LDAP user and group backend functionality has been the source of a lot of errors recently as of updating to NC 24.
5HiSSTs2OKKG7whPwWY8
is an example error when trying to login.9FxSJKzzOUbrq65IDsk8
is an example error when trying to retrieve users or groups via the LDAP configuration wizard.drmZOACEwahqrtlOHPCf
is an example error from the Windows NextCloud client.Steps to reproduce
Expected behavior
The LDAP user and group backend functionality to not produce the errors quoted.
Installation method
Community Manual installation with Archive
Operating system
Debian/Ubuntu
PHP engine version
PHP 8.0
Web server
Apache (supported)
Database engine version
MariaDB
Is this bug present after an update or on a fresh install?
Updated to a major version (ex. 22.2.3 to 23.0.1)
Are you using the Nextcloud Server Encryption module?
No. App disabled.
What user-backends are you using?
Configuration report
List of activated Apps
Nextcloud Signing status
Nextcloud Logs
Additional info
No response
The text was updated successfully, but these errors were encountered: