MDL-57558 ldap: fix ldap_get_entries_moodle()
authorIñaki Arenaza <iarenaza@mondragon.edu>
Fri, 30 Dec 2016 23:01:52 +0000 (00:01 +0100)
committerDan Poltawski <dan@moodle.com>
Tue, 16 May 2017 09:48:27 +0000 (10:48 +0100)
commit67bebb69eb6e30be40564a9108dee86bdf0c50b8
treec2d8a467774d552cf2b58119851f97aa014378a2
parent9695ca7698170aeb82ced11298c831dd7d9000a3
MDL-57558 ldap: fix ldap_get_entries_moodle()

While ldap_get_entries_moodle() PHPdocs state that it returns "array
ldap-entries with lower-cased attributes as indexes.", this is not true. It
uses ldap_get_attributes() internally, which returns both numerically indexed
attribute names, and dictionary-like entries indexed by attribute names.

Current code lowercases the dictionary-like entries, but then uses the
numerically indexed entries for the attribute names used as keys in the
returned array. The numerically indexed names might or might not be lowercased,
depending on the LDAP server and PHP version) version. E.g., OpenLDAP 2.x,
Novell eDirectory 8.x and MS Active Directory return mixed-cased attribute
names, and PHP 5.x and PHP 7.x don't lowercase them inside ldap_get_entries().

This is probably why all calls to ldap_get_entries_moodle() are followed by
calls to array_change_key_case(), even if that shouldn't be necessary.

So make sure we always return lower-cased attributs as indexes and add some
unit tests to avoid regressions in the future.
auth/ldap/auth.php
lib/ldaplib.php
lib/tests/ldaplib_test.php
lib/upgrade.txt