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

XML OpenID sample should POST to logout #8554

Closed
jzheaux opened this issue May 19, 2020 · 0 comments
Closed

XML OpenID sample should POST to logout #8554

jzheaux opened this issue May 19, 2020 · 0 comments
Assignees
Labels
in: docs An issue in Documentation or samples type: bug A general bug
Milestone

Comments

@jzheaux
Copy link
Contributor

jzheaux commented May 19, 2020

Describe the bug

Currently the XML OpenID sample uses GET /logout, which doesn't work since the DefaultLogoutPageGeneratingFilter is skipped for OpenID.

To Reproduce

Start up the OpenID sample, navigate to http://localhost:8080, specify an OpenID provider to log in with, and then click Logout. The result is a 404.

Expected behavior

Logout should take the user back to the entry point. Specifically, it should POST to the /logout endpoint and include the _csrf parameter.

@jzheaux jzheaux added in: docs An issue in Documentation or samples type: bug A general bug labels May 19, 2020
@jzheaux jzheaux added this to the 5.4.0-M2 milestone May 19, 2020
@jzheaux jzheaux self-assigned this May 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: docs An issue in Documentation or samples type: bug A general bug
Projects
None yet
Development

No branches or pull requests

1 participant