Skip to content

Commit

Permalink
improve dx and document auth exception
Browse files Browse the repository at this point in the history
  • Loading branch information
jrushlow committed Apr 1, 2021
1 parent a4b3d12 commit b9b14e0
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 0 deletions.
4 changes: 4 additions & 0 deletions Authenticator/Passport/Badge/UserBadge.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@

namespace Symfony\Component\Security\Http\Authenticator\Passport\Badge;

use Symfony\Component\Security\Core\Exception\AuthenticationException;
use Symfony\Component\Security\Core\Exception\UsernameNotFoundException;
use Symfony\Component\Security\Core\User\UserInterface;
use Symfony\Component\Security\Http\EventListener\UserProviderListener;
Expand Down Expand Up @@ -55,6 +56,9 @@ public function getUserIdentifier(): string
return $this->userIdentifier;
}

/**
* @throws AuthenticationException when the user cannot be found
*/
public function getUser(): UserInterface
{
if (null === $this->user) {
Expand Down
3 changes: 3 additions & 0 deletions Authenticator/Passport/Passport.php
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,9 @@ public function __construct($userBadge, CredentialsInterface $credentials, array
}
}

/**
* {@inheritdoc}
*/
public function getUser(): UserInterface
{
if (null === $this->user) {
Expand Down
4 changes: 4 additions & 0 deletions Authenticator/Passport/UserPassportInterface.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@

namespace Symfony\Component\Security\Http\Authenticator\Passport;

use Symfony\Component\Security\Core\Exception\AuthenticationException;
use Symfony\Component\Security\Core\User\UserInterface;

/**
Expand All @@ -22,5 +23,8 @@
*/
interface UserPassportInterface extends PassportInterface
{
/**
* @throws AuthenticationException when the user cannot be found
*/
public function getUser(): UserInterface;
}

0 comments on commit b9b14e0

Please sign in to comment.