[Tickets #9196] H4: Userlist empty (in admin gui)

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/9196
------------------------------------------------------------------------------
  Ticket             | 9196
  Created By         | hutter at ihs.ac.at
  Summary            | H4: Userlist empty (in admin gui)
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 2. Medium
  Milestone          |
  Patch              | 1
  Owners             |
+New Attachment     | Ldap.php.diff
------------------------------------------------------------------------------


hutter at ihs.ac.at (2010-08-24 12:26) wrote:

Clicking on Users in the admin GUI (logged in as an administrator to
horde), we get only an empty list (icons are visible but no username is
shown).
I tracked this problem down in the code and it seems to be the following line
in function listUsers in framework/Auth/lib/Horde/Auth/Ldap.php
$uid = Horde_String::lower($this->_params['uid']);

See the attachment for a fix/workaround (diff -u output).

I assume the problem is, that a Ldap query is not case sensitive.
However, the result returned from our Windows 2008 Active Directory server
has the attribute sAMAccountName. By only converting the parameter uid
(which holds sAMAccountName) to lowercase, but not the returned result too,
the line "$userlist[] = $val[$uid][0];" will fail. If conversion to lowercase
is needed for other (real) ldap implementations, ensuring to convert the
returned attributes too, will be necessary for AD to work.

The disadvantage of the provided solution is one must enter  
sAMAccountName in the config, thus having a case sensitive  
configuration entry.

best,
Harald Hutter