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

[DX-2713] feat: ability to skip logging out of auth0 in the browser #175

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

nattb8
Copy link
Collaborator

@nattb8 nattb8 commented Feb 29, 2024

Summary

The SDK's current logout function performs a local logout and also opens the browser to log them out of the browser session. However, some customers might find the browser popup undesirable. To address this issue, a change has been made to allow users to skip logging out of the browser step and only perform a local logout:

  • To skip the Auth0 logout and only perform a local logout, set hardLogout to false when calling Logout() or LogoutPKCE().
  • To perform local and Auth0 logout, set the hardLogout parameter to true when calling the Logout() or LogoutPKCE() function. This is the default.

Customer Impact

Developers can now choose not to display the browser pop-up when logging out. However, if they choose this option, users will remain logged into Passport in the browser until the browser session expires.

e.g.

  • User logs into Passport with Google Account A
  • User logs out without the browser popup (hardLogout is set to false)
  • User tries to log into Passport again
  • If the Passport session in the browser is:
    • still valid, the user will be automatically logged in as Google Account A
    • no longer valid, the user will go through the full Passport login flow again (choose Google/Email/etc.)

Other things to consider:

@nattb8 nattb8 requested a review from a team as a code owner February 29, 2024 01:36
@nattb8 nattb8 merged commit afffe37 into main Feb 29, 2024
13 checks passed
@nattb8 nattb8 deleted the feat/hard-logout branch February 29, 2024 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants