-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Duplicated correlation ID in both query parameter and form body #8193
Copy link
Copy link
Open
Labels
Needs: Attention 👋Awaiting response from the MSAL.js teamAwaiting response from the MSAL.js teambug-unconfirmedA reported bug that needs to be investigated and confirmedA reported bug that needs to be investigated and confirmedmsal-nodeRelated to msal-node packageRelated to msal-node packagepublic-clientIssues regarding PublicClientApplicationsIssues regarding PublicClientApplicationsquestionCustomer is asking for a clarification, use case or information.Customer is asking for a clarification, use case or information.
Metadata
Metadata
Assignees
Labels
Needs: Attention 👋Awaiting response from the MSAL.js teamAwaiting response from the MSAL.js teambug-unconfirmedA reported bug that needs to be investigated and confirmedA reported bug that needs to be investigated and confirmedmsal-nodeRelated to msal-node packageRelated to msal-node packagepublic-clientIssues regarding PublicClientApplicationsIssues regarding PublicClientApplicationsquestionCustomer is asking for a clarification, use case or information.Customer is asking for a clarification, use case or information.
Type
Fields
Give feedbackNo fields configured for issues without a type.
Core Library
MSAL Node (@azure/msal-node)
Core Library Version
4.14.0-beta.1
Wrapper Library
Not Applicable
Wrapper Library Version
none
Public or Confidential Client?
Public
Description
When using
ClientCredentialClientto acquire a token, the request sent to ARM includes theclient-request-idin both the URL query parameters and the form body. This duplication creates ambiguity in the request.And other language SDKs (e.g., Go) do not set this field in both places, which suggests this might be an inconsistency in the implementation.
microsoft-authentication-library-for-js/lib/msal-node/src/client/ClientCredentialClient.ts
Lines 376 to 379 in 383a6c5
microsoft-authentication-library-for-js/lib/msal-common/src/client/BaseClient.ts
Lines 308 to 311 in 383a6c5
Error Message
No response
MSAL Logs
No response
Network Trace (Preferrably Fiddler)
MSAL Configuration
Relevant Code Snippets
Reproduction Steps
N/A
Expected Behavior
The
client-request-idshould only appear in one location to avoid confusion and ensure consistency.Identity Provider
Entra ID (formerly Azure AD) / MSA
Browsers Affected (Select all that apply)
None (Server)
Regression
No response