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

client: Whitelist logout redirect URL per client #1004

Closed
konstlepa opened this issue Aug 23, 2018 · 2 comments
Closed

client: Whitelist logout redirect URL per client #1004

konstlepa opened this issue Aug 23, 2018 · 2 comments
Labels
Milestone

Comments

@konstlepa
Copy link
Contributor

Hello,

Our team has many apps that use OAuth2. Each app requires own logout redirect url. In #984 OAUTH_LOGOUT_REDIRECT_URL was added but for all. May we have the ability to set it for one client only?

Thanks

@aeneasr aeneasr added feat New feature or request. package/consent labels Aug 23, 2018
@aeneasr aeneasr added this to the unplanned milestone Aug 23, 2018
@aeneasr
Copy link
Member

aeneasr commented Aug 23, 2018

Tracked as enhancement but currently not on our priority list for 1.0.0 stable.

@aeneasr aeneasr changed the title Support setting OAUTH_LOGOUT_REDIRECT_URL for a client. client: Whitelist logout redirect URL per client Aug 25, 2018
@singlewind
Copy link

@k-lepa You actually can implement your own logic.

  1. Your app's logout redirect to OP with return_url as query params, eg /signout where you record the return_url from query into session
  2. /signout redirect the user the /oauth2/sessions/login/revoke
  3. Then OP will redirect to OAUTH_LOGOUT_REDIRECT_URL which can be, eg /signout-redirect fetch the return_url from the session. And redirect the user back where they come from.

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

No branches or pull requests

3 participants