Skip to content

Commit 5060d64

Browse files
authored
Merge pull request #56085 from nextcloud/fix/55613/drop-hard-base-checks
fix(LDAP): drop hard base checks for now
2 parents 99d6276 + 57cb905 commit 5060d64

File tree

1 file changed

+0
-14
lines changed

1 file changed

+0
-14
lines changed

apps/user_ldap/lib/Connection.php

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -520,20 +520,6 @@ private function doCriticalValidation(): void {
520520
);
521521
}
522522

523-
if (!empty($baseUsers) && !$this->checkBasesAreValid($baseUsers, $base)) {
524-
throw new ConfigurationIssueException(
525-
'User base is not in root base',
526-
$this->l10n->t('User base DN is not a subnode of global base DN'),
527-
);
528-
}
529-
530-
if (!empty($baseGroups) && !$this->checkBasesAreValid($baseGroups, $base)) {
531-
throw new ConfigurationIssueException(
532-
'Group base is not in root base',
533-
$this->l10n->t('Group base DN is not a subnode of global base DN'),
534-
);
535-
}
536-
537523
if (mb_strpos((string)$this->configuration->ldapLoginFilter, '%uid', 0, 'UTF-8') === false) {
538524
throw new ConfigurationIssueException(
539525
'Login filter does not contain %uid placeholder.',

0 commit comments

Comments
 (0)