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] Add pattern for login on page-load #1543
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.
Thanks for addressing feedback!
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
@@ -106,6 +107,7 @@ export class PublicClientApplication { | |||
* containing data from the server (returned with a null or non-blocking error). | |||
*/ | |||
async handleRedirectCallback(authCallback: AuthCallback): Promise<void> { | |||
console.warn("handleRedirectCallback will be deprecated upon release of msal-browser@v2.0.0. Please transition to using onRedirectAppLoad()."); |
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.
@pkanher617 Forgot to update the warning with the new API name :(
This PR fixes an issue raised in #1497 where calling loginRedirect on page load will cause the application to go into a redirect loop.
This PR adds an API which allows applications using the redirect flow to utilize and notify their application that MSAL is done loading.