From b64e48335f63662a43d0b7198fd013d4af521996 Mon Sep 17 00:00:00 2001 From: Andreas Pflug Date: Tue, 31 Jan 2017 01:40:09 +0100 Subject: [PATCH] Don't try to use memberof on posixgroups, since it doesn't support memberUid attributes. Signed-off-by: Andreas Pflug --- apps/user_ldap/lib/Group_LDAP.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/apps/user_ldap/lib/Group_LDAP.php b/apps/user_ldap/lib/Group_LDAP.php index 49e5e72483..7ded4ef663 100644 --- a/apps/user_ldap/lib/Group_LDAP.php +++ b/apps/user_ldap/lib/Group_LDAP.php @@ -496,9 +496,11 @@ class Group_LDAP extends BackendUtility implements \OCP\GroupInterface { // if possible, read out membership via memberOf. It's far faster than // performing a search, which still is a fallback later. + // memberof doesn't support memberuid, so skip it here. if(intval($this->access->connection->hasMemberOfFilterSupport) === 1 && intval($this->access->connection->useMemberOfToDetectMembership) === 1 - ) { + && strtolower($this->access->connection->ldapGroupMemberAssocAttr) !== 'memberuid' + ) { $groupDNs = $this->_getGroupDNsFromMemberOf($userDN); if (is_array($groupDNs)) { foreach ($groupDNs as $dn) {