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.

Sign up for GitHub

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

Jump to bottom

[Feature Request] Enable Azure.Identity to benefit from MSAL.NET token cache when used for Managed identity #3137

Closed
jmprieur opened this issue Feb 3, 2022 · 2 comments
Closed

[Feature Request] Enable Azure.Identity to benefit from MSAL.NET token cache when used for Managed identity #3137

jmprieur opened this issue Feb 3, 2022 · 2 comments
Assignees
@trwalke
Labels
Feature Request
Milestone
4.44.0

Comments

@jmprieur
Copy link
Contributor

jmprieur commented Feb 3, 2022

Is your feature request related to a problem? Please describe.
Azure.Identity (Azure SDK) is using MSAL.NET for some flows, and in that case it benefits from MSAL.NET cache. But in other cases, it goes directly to the MSI endpoint, and does not cache the token.
The ask is to have a way for the Azure SDK to benefit from MSAL.NET cache

Describe the solution you'd like
The spec is here

Describe alternatives you've considered
See the spec

Additional context
This will be a consistency item,
@jmprieur
Copy link
Contributor Author

jmprieur commented Feb 3, 2022

See also the previous spike: ffb0592

@jmprieur jmprieur changed the title [Feature Request] Enable Azure.Identity to benefit from MSAL.NET token cache [Feature Request] Enable Azure.Identity to benefit from MSAL.NET token cache in Managed identity case Feb 3, 2022
@jmprieur jmprieur changed the title [Feature Request] Enable Azure.Identity to benefit from MSAL.NET token cache in Managed identity case [Feature Request] Enable Azure.Identity to benefit from MSAL.NET token cache when used for Managed identity Feb 3, 2022
@bgavrilMS bgavrilMS added this to the 4.42.0 milestone Feb 23, 2022
@bgavrilMS
Copy link
Member

@trwalke - please sync with @schaabs on this first and also read the spec.

@bgavrilMS bgavrilMS modified the milestones: 4.42.0, 4.43 Feb 28, 2022
@trwalke trwalke mentioned this issue Mar 3, 2022
Merged
@chlowell chlowell mentioned this issue Apr 4, 2022
Closed
@pmaytak pmaytak modified the milestones: 4.43.0, 4.44.0 Apr 4, 2022
@pmaytak pmaytak closed this as completed May 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trwalke trwalke

Labels
Feature Request
Projects
None yet
Milestone
4.44.0
Development

No branches or pull requests
4 participants
@bgavrilMS @jmprieur @trwalke @pmaytak