Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
[Security] Allow configuring a target url when switching user#17024
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
xabbuh commentedJul 21, 2022
looking at the example configuration I wonder if being able to configure a route name wouldn't be more sensible in most cases than having to configure the URL (path) |
94noni commentedJul 22, 2022
I see, I've inspired myself with thelogin default_target_path iirc swapping to a route name, what changes will be necessary on the code side? injecting the router that's all? |
xabbuh commentedJul 22, 2022
injecting a |
94noni commentedAug 16, 2022
@xabbuh got it I’ll try to handle the code change in the next days, and then rewrite this doc entry Status: needs work |
06103c9 to26bbd9dCompare26bbd9d tofacbbb0Compare94noni commentedSep 13, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
its finally done@xabbuh :) many thanks |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
94noni commentedSep 18, 2022
I am afk so easier to accept suggested changes than using phone editor |
javiereguiluz commentedSep 20, 2022
Thanks Antoine. |
7440452 to90dc3c8Compare
Uh oh!
There was an error while loading.Please reload this page.
Documentssymfony/symfony#47343
Supersedsymfony/symfony#46338 (feature changed)
Close#17021