Skip to content
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

[Security] Configuration logout target indicates route name instead of path #20251

Closed
wants to merge 1 commit into from

Conversation

remy-theroux
Copy link

Doc indicates a route name instead of a path name. Route names did not success for me so i think this is a choice.

Doc indicates a route name instead of a path name.
Route names did not success for me so i think this is a choice.
@remy-theroux remy-theroux changed the title fix: Configuration logout target path confusing example [Security] Configuration logout target indicates route name instead of path Sep 24, 2024
@xabbuh
Copy link
Member

xabbuh commented Sep 25, 2024

A route name should work the same as a path.

@remy-theroux
Copy link
Author

It doesn't work for me with route name, i don't have time to up an empty project to test outside my use case so i close the mege request. Thanks

@wouterj
Copy link
Member

wouterj commented Sep 26, 2024

This feature was introduced in 6.4: https://symfony.com/blog/new-in-symfony-6-4-simpler-logout

Maybe you are on 5.4? If this is the case, I would recommend setting the doc version to 5.x when reading the documentation.

@remy-theroux
Copy link
Author

Nope i was on symfony 7.1

@remy-theroux
Copy link
Author

I will make a try on a clean symfony install and will open bug issue if still a problem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants