Skip to content

Commit 5b21ee9

Browse files
committed
Fix bad auth message
1 parent 800cbf2 commit 5b21ee9

File tree

2 files changed

+20
-1
lines changed

2 files changed

+20
-1
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
<?php
2+
3+
namespace Ggbb\SymfonyUserPermission\Exception;
4+
5+
use Symfony\Component\Security\Core\Exception\AuthenticationException;
6+
7+
class AuthenticationUserNotFoundException extends AuthenticationException
8+
{
9+
public function getMessageKey(): string
10+
{
11+
return 'User not found';
12+
}
13+
}

src/Security/Provider/UserProvider.php

+7-1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
namespace Ggbb\SymfonyUserPermission\Security\Provider;
44

55
use Doctrine\ORM\EntityManagerInterface;
6+
use Ggbb\SymfonyUserPermission\Exception\AuthenticationUserNotFoundException;
67
use Ggbb\SymfonyUserPermission\GgbbUserPermissionBundle;
78
use Symfony\Bridge\Doctrine\Security\User\UserLoaderInterface;
89
use Symfony\Component\DependencyInjection\ParameterBag\ContainerBagInterface;
@@ -24,7 +25,12 @@ public function loadUserByIdentifier(string $identifier): UserInterface
2425
/** @var UserLoaderInterface $userRepository */
2526
$userRepository = $this->entityManager->getRepository($userName);
2627

27-
return $userRepository->loadUserByIdentifier($identifier);
28+
$user = $userRepository->loadUserByIdentifier($identifier);
29+
if (!$user) {
30+
throw new AuthenticationUserNotFoundException();
31+
}
32+
33+
return $user;
2834
}
2935

3036
public function refreshUser(UserInterface $user): UserInterface

0 commit comments

Comments
 (0)