-
Notifications
You must be signed in to change notification settings - Fork 23
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
Feature OIDC logout #494
base: master
Are you sure you want to change the base?
Feature OIDC logout #494
Conversation
For what it's worth, this will need to be merged for OIE orgs. The old fromURI Okta logout doesn't behave the same between classic and OIE tenants. Of course, Noi's Terraform refactor needs to be merged first so I can update this branch with my terraform changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mdwallick We need to rebase or update the terraform
directory structure, since that has changed.
…the direct Okta tenant /logout
…the direct Okta tenant /logout
c574681
to
eaf505c
Compare
Kudos, SonarCloud Quality Gate passed!
|
Changes the logout behavior to use the OIDC v1/logout endpoint rather than the Okta logout URL directly. Also updates the HCL for each vertical to add post_logout_redirect_uris.
Resolves #351