-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
UserLoader Not Called #50511
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
Comments
Would you mind sending a PR? |
This was referenced Jul 25, 2023
guillaumesmo
added a commit
to guillaumesmo/symfony
that referenced
this issue
Jul 25, 2023
guillaumesmo
added a commit
to guillaumesmo/symfony
that referenced
this issue
Jul 25, 2023
guillaumesmo
added a commit
to guillaumesmo/symfony
that referenced
this issue
Jul 26, 2023
guillaumesmo
added a commit
to guillaumesmo/symfony
that referenced
this issue
Jul 28, 2023
fabpot
added a commit
that referenced
this issue
Aug 25, 2023
This PR was merged into the 6.3 branch. Discussion ---------- [Security] Fix loading user from UserBadge | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #50511 | License | MIT | Doc PR | none Fixed a breaking change from https://github.com/symfony/symfony/pull/48272/files#diff-de9707bb338188f62878f2ebd42e7a7bf9547f6d0bf07a4fcd9c386c263c601b Commits ------- 21532cb Fix breaking change in AccessTokenAuthenticator
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Symfony version(s) affected
6.3
Description
Always using the default userLoader.
AccessTokenHandler:
How to reproduce
Possible Solution
Before:
if (null === $userBadge->getUserLoader() && $this->userProvider) {
New Code:
https://github.com/vincentchalamon/symfony/blob/99a35f0fc32a7b5250aab5530129bae318c95209/src/Symfony/Component/Security/Http/Authenticator/AccessTokenAuthenticator.php#LL62C35-L62C35
Feature:
#48272
Additional Context
No response
The text was updated successfully, but these errors were encountered: