Support for paginated LDAP requests
Bug #1496690 reported by
Aaron Wells
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mahara |
Confirmed
|
Medium
|
Unassigned | ||
Bug Description
As described here: https:/
The Mahara LDAP auth plugin (which was originally written for PHP 5.3) doesn't do paginated LDAP requests. This causes problems with ActiveDirectory, which will only return 1000 results at a time.
PHP 5.4 added support for paginated LDAP. We should update the auth plugin to use this functionality. In order to still work in 5.3, we should use a function_exists() call to check for the existence of the paginated LDAP methods and only use them if they're available.
Revision history for this message
| Stefan Bäcker (stefan-baecker-29) wrote | #1 |
To post a comment you must log in.
This patch has an implementation of paged results for the user synchronisation via ldap. Also catching errors when hitting a size limit that is not taken care of in the current implementation. Patch is against 22.04.2, but also works in 22.04.3 and should be good for 22.10 also, but untested.