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

[Feature Request] Expose AuthenticationResult.TokenType with MSAL.NET #2637

Closed
hkuadithya opened this issue May 17, 2021 · 2 comments · Fixed by #2638
Closed

[Feature Request] Expose AuthenticationResult.TokenType with MSAL.NET #2637

hkuadithya opened this issue May 17, 2021 · 2 comments · Fixed by #2638

Comments

@hkuadithya
Copy link

We are migrating some legacy code in our projects from ADAL to MSAL.

Both ADAL and MSAL return an object of type AuthenticationResult after acquiring an access token.

In our older code path, we rely on the type of ADAL access token i.e. AuthenticationResult.AccessTokenType to set the scheme in AuthenticationHeaderValue for making another request.

new AuthenticationHeaderValue(
    scheme: firstPartyAuthResult.AccessTokenType,
    parameter: firstPartyAuthResult.AccessToken);

MSAL's AutenticationResult does have AccessToken but it doesn't seem to have AccessTokenType (correct me If I'm wrong).

How do I determine the AccessTokenType for a MSAL auth result? Please don't suggest hardcoding the scheme to Bearer. Thanks in advance.

SO thread mirroring this issue: https://stackoverflow.microsoft.com/questions/258737

@hkuadithya
Copy link
Author

hkuadithya commented May 25, 2021

@bgavrilMS could you provide a timeline for the release of MSAL.NET 4.32?
I can find the current releases here but I'm waiting for 4.32: https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/releases

Is there a readme doc that indicates the timeline for releases of MSAL.NET Nuget packages?

@jennyf19
Copy link
Collaborator

@hkuadithya should be in 1-2 weeks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants