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

[Bug] Logout error when using Easy Auth v2 #1234

Closed
1 of 9 tasks
jwevansgam opened this issue Jun 1, 2021 · 5 comments · Fixed by #1246
Closed
1 of 9 tasks

[Bug] Logout error when using Easy Auth v2 #1234

jwevansgam opened this issue Jun 1, 2021 · 5 comments · Fixed by #1246

Comments

@jwevansgam
Copy link

Which version of Microsoft Identity Web are you using?
1.10

Where is the issue?

  • Web app
    • Sign-in users
    • Sign-in users and call web APIs
  • Web API
    • Protected web APIs (validating tokens)
    • Protected web APIs (validating scopes)
    • Protected web APIs call downstream web APIs
  • Token cache serialization
    • In-memory caches
    • Session caches
    • Distributed caches
  • Other (please describe)
    • Sign-out users

Is this a new or an existing app?
c. This is a new app or an experiment.

Repro

The Issue is described in this discussion #1222

Basic summary is that the the environment variable WEBSITE_AUTH_LOGOUT_PATH has been deprecated with App Service Authentication v2 (Easy Auth) so I get the below exception when logging out.

Expected behavior
The user should be logged out of the Easy Auth session.

Actual behavior
The following exception is thrown

Value cannot be null or empty: (Parameter: 'localUrl')

Possible solution

Additional context / logs / screenshots
Add any other context about the problem here, such as logs and screenshots.

@jmprieur
Copy link
Collaborator

jmprieur commented Jun 1, 2021

@navyasric what would be your recommendation?

@jmprieur
Copy link
Collaborator

jmprieur commented Jun 7, 2021

• EasyAuth took a breaking change by not adding the logout path environment variable in v2 where it would be added in v1.
• if auth is enabled on the app (the enabled env var is set to true), then we can assume no WEBSITE_AUTH_LOGOUT_PATH environment variable to mean that the logout path is at WEBSITE_AUTH_API_PREFIX/logout, and if now api prefix is specified, then the logout path is at the default "/.auth/logout"

@jennyf19
Copy link
Collaborator

Fixed in 1.13 release.

@jwevansgam
Copy link
Author

What a great turnaround, 13 days from reporting to being added to the code base. Thanks so much for this. I have just tested today and can confirm that this has resolved my issue.

@jmprieur
Copy link
Collaborator

Thanks for confirming, @jwevansgam

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

Successfully merging a pull request may close this issue.

3 participants