Skip to content

Impersonation exit logs out user #47356

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
Closed
simonsolutions opened this issue Aug 22, 2022 · 3 comments
Closed

Impersonation exit logs out user #47356

simonsolutions opened this issue Aug 22, 2022 · 3 comments
Labels
Bug Security Status: Needs Review Status: Waiting feedback

Comments

@simonsolutions
Copy link

Symfony version(s) affected

6.1.3

Description

Exact the same behaviour as described in PR #46748 is in the current 6.1.3.

The files changed in the PR seems not to be available in 6.1 branch, so the merging in this branch might have not fixed the issue.

How to reproduce

Same as in PR described

Possible Solution

No response

Additional Context

No response
@xabbuh
Copy link
Member

xabbuh commented Aug 22, 2022

Can you create a small example application that allows to reproduce your issue?

The UserAuthenticationProvider class that was fixed in #46748 does not exist in 6.0+ anymore. So the fix provided there cannot be the solution for your issue.

@chalasr
Copy link
Member

chalasr commented Aug 22, 2022

Also a similar piece of code meant to workaround the very same issue exists in https://github.com/symfony/symfony/blob/6.2/src/Symfony/Component/Security/Http/Firewall/ContextListener.php#L309 (where "deauthentication on user change" happens)

@xabbuh
Copy link
Member

xabbuh commented Sep 2, 2022

I am going to close here for now due to the lack of feedback. Please let us know when you have more information and we can consider to reopen.

@xabbuh xabbuh closed this as completed Sep 2, 2022
@simonsolutions simonsolutions mentioned this issue Oct 11, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Security Status: Needs Review Status: Waiting feedback
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@xabbuh @chalasr @carsonbot @simonsolutions