fix(ldap): if ldap server not defined (dn == '') returns a UserNotFou…

…ndException (not a Symfony\Component\Ldap\Adapter\ExtLdap\ConnectionException)
theus77 · Dec 26, 2023 · 1898374 · 1898374
1 parent 0e83cfc
commit 1898374
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/EMS/core-bundle/src/Security/Ldap/LdapUserProvider.php b/EMS/core-bundle/src/Security/Ldap/LdapUserProvider.php
@@ -11,6 +11,7 @@
 use Symfony\Component\Ldap\Security\LdapUser;
 use Symfony\Component\Ldap\Security\LdapUserProvider as SymfonyLdapUserProvider;
 use Symfony\Component\Security\Core\Exception\UserNotFoundException;
+use Symfony\Component\Security\Core\User\UserInterface;
 
 class LdapUserProvider extends SymfonyLdapUserProvider
 {
@@ -41,4 +42,13 @@ protected function loadUser(string $identifier, Entry $entry): User
 
         return $user ?: LdapCoreUserFactory::create($ldapUser, $this->ldapConfig);
     }
+
+    public function loadUserByIdentifier(string $identifier): UserInterface
+    {
+        if ('' === $this->ldapConfig->baseDn) {
+            throw new UserNotFoundException(\sprintf('Ldap server not configured'));
+        }
+
+        return parent::loadUserByIdentifier($identifier);
+    }
 }