[AMOS commit] MDL-50625 auth_ldap: Better check for paged results support Committed into Git: 2016-08-06 19:14 UTC

[AMOS commit] MDL-50625 auth_ldap: Better check for paged results support Committed into Git: 2016-08-06 19:14 UTC

by AMOS bot -
Number of replies: 0
Author: Iñaki Arenaza
MDL-50625 auth_ldap: Better check for paged results support
There is at least one LDAP server (Sun Directory Server) that doesn't
support Paged Results extension, even if it supports LDAP version 3. So
checking just for LDAP version is not enough.

If possible, we check the supportedControl attribute of the LDAP rootDSE
and see if the paged results control is available. This needs an LDAP
connection, which might not be possible to establish before we configure
some essential LDAP settings (server, bind user, password, etc.). Thus
we try to establish the connection and check the supportedControl
attribute. But if we fail, we perform only basic checks that are less
accurate and err on the side of cautiousness.


http://git.moodle.org/gw?p=moodle.git;a=commit;h=e47863e840fa6da779eb0816c5d3ad0ccad13504
http://github.com/moodle/moodle/commit/e47863e840fa6da779eb0816c5d3ad0ccad13504

+ 3.3 en [pagedresultsnotsupp,auth_ldap]