-
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
[msal-browser] Fix state mismatch error due to encoding #2049
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm. Please ensure follow up with the server to align on encoding mechanisms.
Hi all, it appears that this PR will fix an issue my team is currently having with MSAL. We're not sure what sort of release cadence you are planning for with 2.0. What sort of timeline should we expect for when this fix will be available through NPM? |
…tion-library-for-js into browser-state-mismatch-fix
It appears that the state that is returned from the server is being mutated by their base64 encoding mechanism, which is different to our encoding mechanism, causing a state mismatch error. This PR attempts to provide a temporary solution by decoding the state strings before comparing the response state and cached state. More investigation and communication with the server team may be necessary to figure out why this mutation occurs.
This PR addresses issue #2019 .