Skip to content

Conversation

@markbrockhoff
Copy link

Closes #233

PR Checklist

  • I have run npm test locally and all tests are passing.
  • I have added/updated tests for any new behavior.
  • If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description

Instead of throwing an error if the post logout redirect url isn't present no redirect will happen but instead a simple page with the text "Logout successful" will be returned. It can be used to verify the logout e.g. during integration tests.

Instead of throwing an error if the post logout redirect url isn't present no redirect will happen but instead a simple page with the text "Logout successful" will be returned. It can be used to verify the logout e.g. during integration tests.
@nulltoken nulltoken requested a review from poveden December 5, 2024 19:47
@nulltoken
Copy link
Contributor

nulltoken commented Dec 5, 2024

@poveden I'm not really sure about the proposal from a pure spec standpoint.

You know better than me. What's your opinion about this?

More context available in #233

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't require post_logout_redirect_uri

2 participants