diff options
| author | Côme Chilliet <come.chilliet@nextcloud.com> | 2023-06-29 10:42:59 +0200 |
|---|---|---|
| committer | Côme Chilliet <come.chilliet@nextcloud.com> | 2023-06-29 10:42:59 +0200 |
| commit | 4a0351878c8800e76a12669e12b4e41fa9c13b2f (patch) | |
| tree | 09ab46f33d9e94693ed571bdb646f101b4099b60 | |
| parent | 4cfab4b838ed40dec200f7673992009896c69f16 (diff) | |
| download | nextcloud-server-4a0351878c8800e76a12669e12b4e41fa9c13b2f.tar.gz nextcloud-server-4a0351878c8800e76a12669e12b4e41fa9c13b2f.zip | |
Avoid adding (attribute=) part to filter which will confuse LDAP servers
Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
| -rw-r--r-- | apps/user_ldap/lib/Access.php | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/apps/user_ldap/lib/Access.php b/apps/user_ldap/lib/Access.php index 3fbb003ed51..0b115c42764 100644 --- a/apps/user_ldap/lib/Access.php +++ b/apps/user_ldap/lib/Access.php @@ -1545,12 +1545,16 @@ class Access extends LDAPUtility { return ''; } // wildcards don't work with some attributes - $filter[] = $fallbackAttribute . '=' . $originalSearch; + if ($originalSearch !== '') { + $filter[] = $fallbackAttribute . '=' . $originalSearch; + } $filter[] = $fallbackAttribute . '=' . $search; } else { foreach ($searchAttributes as $attribute) { // wildcards don't work with some attributes - $filter[] = $attribute . '=' . $originalSearch; + if ($originalSearch !== '') { + $filter[] = $attribute . '=' . $originalSearch; + } $filter[] = $attribute . '=' . $search; } } |