Skip to content

Commit

Permalink
bug #905 Changed JWTAuthenticator::start method return type to more…
Browse files Browse the repository at this point in the history
… generic `Response` type (aurimasniekis)

This PR was merged into the 2.x branch.

Discussion
----------

Changed `JWTAuthenticator::start` method return type to more generic `Response` type

Not a big change just fixes an issue where the old code is upgraded to use the new JWTAuthenticator.

This fixes #904

Commits
-------

5e76203 Changed `JWTAuthenticator::start`` method to more generic `Response` type
  • Loading branch information
chalasr committed Aug 23, 2021
2 parents 0161668 + 5e76203 commit adfcb5b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Security/Authenticator/JWTAuthenticator.php
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ public function __construct(
/**
* {@inheritdoc}
*/
public function start(Request $request, AuthenticationException $authException = null): JWTAuthenticationFailureResponse
public function start(Request $request, AuthenticationException $authException = null): Response
{
$exception = new MissingTokenException('JWT Token not found', 0, $authException);
$event = new JWTNotFoundEvent($exception, new JWTAuthenticationFailureResponse($exception->getMessageKey()));
Expand Down

0 comments on commit adfcb5b

Please sign in to comment.