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

[Identity] [External] MSAL requires localAccountId only in JS #14377

Closed
sadasant opened this issue Mar 18, 2021 · 2 comments
Closed

[Identity] [External] MSAL requires localAccountId only in JS #14377

sadasant opened this issue Mar 18, 2021 · 2 comments

Comments

@sadasant
Copy link
Contributor

sadasant commented Mar 18, 2021

MSAL currently requires localAccountId only in JS when one specifies an account through the configuration options. This value is returned from the account information we receive on the response objects. In .Net, this value is not returned on the response objects and not required.

As part of this PR #14064 I have made the workaround of re-using the homeAccountId to fulfill the localAccountId requirement. This does work, but the MSAL team tells me this is not correct and a real localAccountId is necessary for authentications such as ADSF.

I am not sure whether they should make this optional or require it in all languages. In any case, we probably shouldn't be using the homeAccountId as a replacement value anyway.

External issue: AzureAD/microsoft-authentication-library-for-js#3336

@sadasant
Copy link
Contributor Author

@sadasant
Copy link
Contributor Author

sadasant commented Jun 7, 2021

@sadasant sadasant closed this as completed Jun 7, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant