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

Expose idTokenClaims on AccountInfo #2554

Merged
merged 14 commits into from Dec 2, 2020
Merged

Conversation

jasonnutter
Copy link
Contributor

@jasonnutter jasonnutter commented Nov 6, 2020

Since AccountEntity objects and ID tokens are cached separately, when building AccountInfo objects we will need to look up the ID token for the given account, decode it, and attach it to the returned object. Should work for browser and Node since all the changes are done in common.

@github-actions github-actions bot added msal-browser Related to msal-browser package msal-common Related to msal-common package labels Nov 6, 2020
@jasonnutter jasonnutter marked this pull request as draft November 6, 2020 01:32
@jasonnutter jasonnutter linked an issue Nov 6, 2020 that may be closed by this pull request
5 tasks
@github-actions github-actions bot added the msal-node Related to msal-node package label Nov 6, 2020
@github-actions
Copy link
Contributor

This PR has not seen activity in 14 days. It may be closed if it remains stale.

@github-actions github-actions bot added the no-pr-activity PR has been inactive for 14 days label Nov 26, 2020
@dipuoec
Copy link

dipuoec commented Nov 26, 2020

When is this PR going to be completed?

@github-actions github-actions bot removed the no-pr-activity PR has been inactive for 14 days label Nov 27, 2020
@jasonnutter jasonnutter marked this pull request as ready for review December 2, 2020 01:43
@coveralls
Copy link

coveralls commented Dec 2, 2020

Coverage Status

Coverage increased (+0.1%) to 81.513% when pulling 556d217 on browser-idtokenclaims-accountinfo into bdadc82 on dev.

@jasonnutter
Copy link
Contributor Author

When is this PR going to be completed?

Looking to have it merged soon and included in @azure/msal-browser@2.8.0

jasonnutter and others added 3 commits December 2, 2020 11:24
…claims-accountinfo.json

Co-authored-by: Jo Arroyo <joarroyo@microsoft.com>
Co-authored-by: Jo Arroyo <joarroyo@microsoft.com>
@jasonnutter jasonnutter merged commit 7f3a24f into dev Dec 2, 2020
@jasonnutter jasonnutter deleted the browser-idtokenclaims-accountinfo branch December 2, 2020 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
msal-browser Related to msal-browser package msal-common Related to msal-common package msal-node Related to msal-node package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Retrieve idTokenClaims for logged in accounts using msal-browser 2.x
8 participants