Merge pull request #10783 from nextcloud/backport/10775/stable13

[stable13] Resolve all group memberships properly
nextcloud · Aug 22, 2018 · 5bc7af2 · 5bc7af2
2 parents a4e989b + 4f6f3b2
commit 5bc7af2
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/apps/user_ldap/lib/Group_LDAP.php b/apps/user_ldap/lib/Group_LDAP.php
@@ -828,7 +828,7 @@ public function usersInGroup($gid, $search = '', $limit = -1, $offset = 0) {
 			if($isMemberUid) {
 				//we got uids, need to get their DNs to 'translate' them to user names
 				$filter = $this->access->combineFilterWithAnd(array(
-					str_replace('%uid', $member, $this->access->connection->ldapLoginFilter),
+					str_replace('%uid', trim($member), $this->access->connection->ldapLoginFilter),
 					$this->access->getFilterPartForUserSearch($search)
 				));
 				$ldap_users = $this->access->fetchListOfUsers($filter, $attrs, 1);