Skip to content

[SecurityBundle] Don't register deprecated listeners with authenticator manager enabled #41376

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
nicolas-grekas merged 1 commit into from
May 24, 2021
Merged

[SecurityBundle] Don't register deprecated listeners with authenticator manager enabled #41376

nicolas-grekas merged 1 commit into from
May 24, 2021

Conversation

@chalasr
Copy link
Member

@chalasr chalasr commented May 21, 2021

Q A
Branch? 5.3
Bug fix? yes
New feature? no
Deprecations? no
Tickets fixes #41373
License MIT
Doc PR -

@chalasr chalasr requested a review from wouterj as a code owner May 21, 2021 23:33
@carsonbot carsonbot added this to the 5.3 milestone May 21, 2021
@chalasr chalasr force-pushed the sec-deprec-load branch from 761883c to 5374bd3 Compare May 21, 2021 23:35
@chalasr chalasr force-pushed the sec-deprec-load branch from 5374bd3 to 66c7d10 Compare May 21, 2021 23:38
@chalasr chalasr mentioned this pull request May 22, 2021
Closed
@jmsche
Copy link
Contributor

jmsche commented May 22, 2021

Hi, I can confirm the fix works on my project :)

@chalasr
Copy link
Member Author

chalasr commented May 23, 2021

Thanks for confirming.
Build failures are unrelated.

@chalasr chalasr added the Ready label May 23, 2021
@nicolas-grekas
Copy link
Member

nicolas-grekas commented May 24, 2021

Thank you @chalasr.

@nicolas-grekas nicolas-grekas merged commit 24a1c28 into symfony:5.3 May 24, 2021
@chalasr chalasr deleted the sec-deprec-load branch May 24, 2021 11:05
@fabpot fabpot mentioned this pull request May 31, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wouterj wouterj Awaiting requested review from wouterj wouterj is a code owner

Assignees

No one assigned

Labels

Bug Ready SecurityBundle Status: Needs Review

Projects

None yet

Milestone

5.3

Development

Successfully merging this pull request may close these issues.

4 participants

@chalasr @jmsche @nicolas-grekas @carsonbot