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

Standard wordpress logout url returns wrong logout url to delete Auth0 session and does not respect return url #8

Open
philipdalen opened this issue Apr 26, 2021 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@philipdalen
Copy link
Contributor

Reproduction steps

  1. Navigate to https://dcg-deutschland.de/ and click on "LOGIN"
  2. Enter your credentials and click on "Log In"
  3. In the top right corner you will notice the appearance of the "LOGOUT" button. Please note that the logout link is currently set to (https://dcg-deutschland.de/wp-login.php?action=logout&redirect_to=https%3A%2F%2Fdcg-deutschland.de&_wpnonce=77d0344246)
  4. Click on "LOGOUT"
  5. In the network tab you will notice that the response of the logout url is as follows. "https://login.bcc.no/v2/logout?id_token_hint=TOKEN&post_logout_redirect_uri: https://dcg-deutschland.de". This is incorrect and not according to the Auth0 docs
  6. You are redirected to https://bcc.no

Expected behavior

  1. Logout url going to Auth0 should be formed according to their documentation here "https://auth0.com/docs/api/authentication#logout"
  2. The returnTo query string paramater of the Auth0 logout url should be configurable from Wordpress BCC login plugin and thus re-direct the user to that url after logout
@philipdalen philipdalen added the bug Something isn't working label Apr 26, 2021
@philipdalen philipdalen changed the title Standard wordpress logout url returns wrong logout url to delete Auth0 Session and does not respect return url Standard wordpress logout url returns wrong logout url to delete Auth0 session and does not respect return url Apr 26, 2021
@rvanoord
Copy link
Member

@philipdalen Would it be feasible to create a rule which rewrites the post_logut_redirect_uri parameter, or handles the redirect?
This OIDC draft for RP-initiated logout uses the same parameter as currently implemented in the sign-on client:
https://openid.net/specs/openid-connect-rpinitiated-1_0.html

@philipdalen
Copy link
Contributor Author

@rvanoord sounds like a good option. I will investigate and come back to you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants