-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
User is not redirected to redirect uri on mobile #5111
Comments
@1hw7 is the Here's the three requirements for successful redirection after logout:
Check out the MSAL Logout docs for more details. |
Thanks for your help Hector! Yes, it is and this works for us in desktop. The issue only arises on a mobile device |
We confirmed that our implementation follows the docs you linked Hector (thank you!) so we are awaiting further context on what the error is and how we can solve this problem |
To update: user is not actually signed out and the user is sso'd back into the app. It seems the server session is not ended? |
Can we please get the msal and network logs @1hw7? |
@1hw7 This issue has been automatically marked as stale because it is marked as requiring author feedback but has not had any activity for 5 days. If your issue has been resolved please let us know by closing the issue. If your issue has not been resolved please leave a comment to keep this open. It will be closed automatically in 7 days if it remains stale. |
Keeping this active. The team reached out to me offline and provided the needed information. |
Same problem for me, please reopen |
For me, the bug is happening only in Chrome on mobile. FF is ok. |
Hi @sameerag could we please reopen this issue(edit: i didn't realize my comment would reopen sorry!)? i do not think the issue has been resolved on our end |
@1hw7 Please share logs on the email mentioned on my profile if possible , looking forward to chat on Monday when you are next available. |
@1hw7 This issue has been automatically marked as stale because it is marked as requiring author feedback but has not had any activity for 5 days. If your issue has been resolved please let us know by closing the issue. If your issue has not been resolved please leave a comment to keep this open. It will be closed automatically in 7 days if it remains stale. |
Commenting to leave open, as the issue continues to persist |
This issue requires attention from the MSAL.js team and has not seen activity in 5 days. @bmahall please follow up. |
1 similar comment
This issue requires attention from the MSAL.js team and has not seen activity in 5 days. @bmahall please follow up. |
This issue requires attention from the MSAL.js team and has not seen activity in 5 days. @tnorling please follow up. |
@1hw7 Are we still stuck on this? |
@1hw7 This issue has been automatically marked as stale because it is marked as requiring author feedback but has not had any activity for 5 days. If your issue has been resolved please let us know by closing the issue. If your issue has not been resolved please leave a comment to keep this open. It will be closed automatically in 7 days if it remains stale. |
@sameerag sorry i missed your comment! Thomas was able to help us look into it (thank you Thomas!). He gave us some insight on what might be going wrong on our end, we really appreciate the help! |
can u describe what is causing the error and how u solved? @1hw7 |
Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.24.0
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
1.4.0
Public or Confidential Client?
Public
Description
On mobile (safari and edge) when a user logs out, it hangs on the "you have logged out page" and the redirect uri does not seem to be triggered (user is never redirected to redirect uri)
Error Message
No response
Msal Logs
No response
MSAL Configuration
Relevant Code Snippets
Reproduction Steps
Expected Behavior
User should be redirected to redirect uri
Identity Provider
Azure AD / MSA
Browsers Affected (Select all that apply)
Edge, Safari
Regression
No response
Source
Internal (Microsoft)
The text was updated successfully, but these errors were encountered: