Merge pull request #25366 from nextcloud/backport/25361/stable20

[stable20] fix parameter provided as string not array

Author: juliusknorr

apps/user_ldap/lib/Group_LDAP.php

@@ -222,7 +222,7 @@ public function getDynamicGroupMembers(string $dnGroup): array {
 			$pos = strpos($memberURLs[0], '(');
 			if ($pos !== false) {
 				$memberUrlFilter = substr($memberURLs[0], $pos);
-				$foundMembers = $this->access->searchUsers($memberUrlFilter, 'dn');
+				$foundMembers = $this->access->searchUsers($memberUrlFilter, ['dn']);
 				$dynamicMembers = [];
 				foreach ($foundMembers as $value) {
 					$dynamicMembers[$value['dn'][0]] = 1;
@@ -915,6 +915,7 @@ public function usersInGroup($gid, $search = '', $limit = -1, $offset = 0) {
 		$attrs = $this->access->userManager->getAttributes(true);
 		foreach ($members as $member) {
 			switch ($this->ldapGroupMemberAssocAttr) {
+				/** @noinspection PhpMissingBreakStatementInspection */
 				case 'zimbramailforwardingaddress':
 					//we get email addresses and need to convert them to uids
 					$parts = explode('@', $member);
@@ -1088,6 +1089,7 @@ public function getGroups($search = '', $limit = -1, $offset = 0) {
 		if (!$this->enabled) {
 			return [];
 		}
+		$search = $this->access->escapeFilterPart($search, true);
 		$cacheKey = 'getGroups-' . $search . '-' . $limit . '-' . $offset;
 
 		//Check cache before driving unnecessary searches