Skip to content

[Security] pass the current token to the checkPostAuth() method of user checkers #57773

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 25, 2024
Merged

[Security] pass the current token to the checkPostAuth() method of user checkers #57773

merged 1 commit into from
Jul 25, 2024

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Jul 19, 2024

Q A
Branch? 7.2
Bug fix? no
New feature? yes
Deprecations? no
Issues Fix #50650
License MIT

@xabbuh xabbuh requested a review from chalasr as a code owner July 19, 2024 07:47
@carsonbot carsonbot added this to the 7.2 milestone Jul 19, 2024
@xabbuh xabbuh mentioned this pull request Jul 19, 2024
Closed
@chalasr
Copy link
Member

chalasr commented Jul 19, 2024

👍 some testcase would be nice to have though

@xabbuh
Copy link
Member Author

xabbuh commented Jul 19, 2024

tests added

@OskarStark OskarStark changed the title [Security] pass the current token to the checkPostAuth() method of user checkers [Security] pass the current token to the checkPostAuth() method of user checkers Jul 22, 2024
@fabpot
Copy link
Member

fabpot commented Jul 25, 2024

Thank you @xabbuh.

@fabpot fabpot merged commit 6c7a6ff into symfony:7.2 Jul 25, 2024
18 checks passed
@fabpot fabpot mentioned this pull request Jul 25, 2024
Closed
@xabbuh xabbuh deleted the issue-50650 branch July 25, 2024 07:02
@fabpot fabpot mentioned this pull request Oct 27, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexandre-daubois alexandre-daubois alexandre-daubois left review comments

@AlexisRossat AlexisRossat AlexisRossat left review comments

@fabpot fabpot fabpot approved these changes

@chalasr chalasr chalasr approved these changes

Assignees
No one assigned
Labels
Feature Security Status: Reviewed
Projects
None yet
Milestone
7.2
Development

Successfully merging this pull request may close these issues.

[security] Provide authentification token when using an user checker
6 participants
@xabbuh @chalasr @fabpot @alexandre-daubois @AlexisRossat @carsonbot