Diff in latest package release on NPM vs codebase, onRedirectNavigate not working

[bentrynning]

Core Library

MSAL.js (@azure/msal-browser)

Core Library Version

5.1.0

Wrapper Library

MSAL React (@azure/msal-react)

Wrapper Library Version

5.0.3

Public or Confidential Client?

Public

Description

onRedirectNavigate does not exist anymore in EndSessionRequest used for logoutRedirect

It is part for the latest tag version in repository: https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/msal-browser-v5.1.0/lib/msal-browser/src/request/EndSessionRequest.ts#L22

But not in the files in the released 5.1.0 npm version

So we can't intercept and stop the redirect with onRedirectNavigate: () => false when using logoutRedirect

Is this on purpose, what is the alternative, and the docs does not have any information about this.

Thanks.

Error Message

No response

MSAL Logs

No response

Network Trace (Preferrably Fiddler)

• Sent
• Pending

MSAL Configuration

{
  auth: {
    clientId: process.env.NEXT_PUBLIC_MSAL_CLIENT_ID,
    authority: process.env.NEXT_PUBLIC_MSAL_AUTHORITY,
    redirectUri: process.env.MSAL_REDIRECT_URI,
    postLogoutRedirectUri: process.env.MSAL_POST_LOGOUT_REDIRECT_URI ,
  },
  cache: {
    cacheLocation: "localStorage",
  },
};

Relevant Code Snippets

instance.logoutRedirect({
      account: instance.getActiveAccount(),
      onRedirectNavigate: () => false
    })

Reproduction Steps

Install latest version and try to use onRedirectNavigate with logoutRedirect

Expected Behavior

Should still support onRedirectNavigate for logoutRedirect

Identity Provider

Entra ID (formerly Azure AD) / MSA

Browsers Affected (Select all that apply)

Chrome

Regression

No response

[AdamConleyDSI]

I ran into something similar. In the PublicClientApplication, enableAccountStorageEvents and disableAccountStorageEvents are not in the npm version, but they are in the 5.1.0 code, between removePerformanceCallback and getAccount.

https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/msal-browser-v5.1.0/lib/msal-browser/src/app/PublicClientApplication.ts#L189-L212

npm code, also confirmed missing in node_modules when installed:

[tnorling]

Both of these changes are intentional and mentioned in the migration guide, looks like the tag is pointing to the wrong commit. We'll get that fixed.

[shylasummers]

This is fixed. The msal-browser-v5.1.0 tag should now point you to the correct commit

[bentrynning]

@tnorling Thanks for the reply. And sorry for not checking or finding the migration guide first.