microsoft-authentication-library-for-dotnet icon indicating copy to clipboard operation
microsoft-authentication-library-for-dotnet copied to clipboard

Published 20 hours ago • verified publisher icon AzureAD

[Feature Request] Include the full IDP response in AuthenticationResult

Open bgavrilMS opened this issue 3 years ago • 5 comments

Is your feature request related to a problem? Please describe. E&D customers would like to parse some of the IdP headers associated with a token request

Describe the solution you'd like MSAL can include the entire HttpResponseMessage along with MSAL's metrics, in AuthenticationResultMetadata

Describe alternatives you've considered Feature could be opt-in, .WithRecordFullIdPResponse(true)

bgavrilMS avatar Mar 10 '22 17:03 bgavrilMS

CC @jmprieur @brentschmaltz @TimHannMSFT for thoughts on devex.

bgavrilMS avatar Mar 10 '22 17:03 bgavrilMS

The scenario is for folks wanting to do negative testing.

Alternative: inject your own HttpClient and intercept the response from the /token endpoint.

bgavrilMS avatar Mar 10 '22 17:03 bgavrilMS

@bgavrilMS I think this feature ask should be in SAL.

brentschmaltz avatar Mar 10 '22 21:03 brentschmaltz

@bgavrilMS I think this feature ask should be in SAL.

We have an item tracking: 1766308. @brentschmaltz E&D asked for MSAL to have a solution as well, they want for both. This is to get response for MSAL calls, SAL may be completely out of the loop, no?

TimHannMSFT avatar Mar 10 '22 21:03 TimHannMSFT

@TimHannMSFT correct, i didn't notice the repo was MSAL.

brentschmaltz avatar Mar 11 '22 19:03 brentschmaltz