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

BAU: Adds logout endpoint #189

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

BAU: Adds logout endpoint #189

wants to merge 5 commits into from

Conversation

Ryan-Andrews99
Copy link
Contributor

@Ryan-Andrews99 Ryan-Andrews99 commented Nov 14, 2024

What:

  • Adds a new /logout endpoint to the simulator designed to emulate the real /logout endpoint
  • Makes post_logout_redirect_uris configurable on the client config
  • Updates README client configuration
  • Moves some integration tests into the /tests dir for consistency

How to review:

  • Code review commit by commit
  • Look at tests in tests/integration/logout.test.ts to see behaviour aligns with real endpoint

@Ryan-Andrews99 Ryan-Andrews99 requested review from a team as code owners November 14, 2024 17:09
@Ryan-Andrews99 Ryan-Andrews99 changed the title BAU logout endpoint BAU: Adds logout endpoint Nov 14, 2024
We need to use this to get the public JWK to validate the id_token
hint if included in a logout request
Adds a new endpoint for simulating the logout endpoint, which
follows the real logout endpoint in validation and response
Copy link

sonarcloud bot commented Nov 14, 2024

src/components/logout/logout-controller.ts Dismissed Show dismissed Hide dismissed
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.

1 participant