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

[Bug] Fix Date-Time Parsing in Token Expiration Calculation for Multiple Formats #873

Open
gladjohn opened this issue Oct 22, 2024 · 0 comments
Assignees
Labels
Bug Something isn't working, needs an investigation and a fix confidential-client For issues related to confidential client apps P3 scenario:ManagedIdentity
Milestone

Comments

@gladjohn
Copy link

Library version used

latest

Java version

latest

Scenario

ManagedIdentityClient - managed identity

Is this a new or an existing app?

None

Issue description and reproduction steps

The method responsible for calculating token expiration (expires_on) in MSAL (Managed Identity) is not handling various date-time formats correctly. This leads to errors or incorrect results when processing expiration times in different formats provided by the Managed Identity service.

more info : AzureAD/microsoft-authentication-library-for-dotnet#4963

Relevant code snippets

No response

Expected behavior

No response

Identity provider

Microsoft Entra ID (Work and School accounts and Personal Microsoft accounts)

Regression

No response

Solution and workarounds

No response

@gladjohn gladjohn added needs attention Automatically used when an issue is created through an issue template untriaged Automatically used when an issue is created through an issue template scenario:ManagedIdentity and removed needs attention Automatically used when an issue is created through an issue template untriaged Automatically used when an issue is created through an issue template labels Oct 22, 2024
@bgavrilMS bgavrilMS added Bug Something isn't working, needs an investigation and a fix P2 Normal priority items, should be done after P1 labels Nov 19, 2024
@bgavrilMS bgavrilMS added the confidential-client For issues related to confidential client apps label Jan 8, 2025
@bgavrilMS bgavrilMS added this to the 1.18.1 milestone Jan 8, 2025
@bgavrilMS bgavrilMS added P3 and removed P2 Normal priority items, should be done after P1 labels Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working, needs an investigation and a fix confidential-client For issues related to confidential client apps P3 scenario:ManagedIdentity
Projects
None yet
Development

No branches or pull requests

3 participants