Added Support for Azure Machine Learning in Managed Identity

[Robbie-Microsoft]

Fixes #7489

Added Machine Learning Managed Identity Source

Tested with unit tests.

[Copilot]

Copilot reviewed 5 out of 9 changed files in this pull request and generated 1 comment.

Files not reviewed (4)

• change/@azure-msal-node-ca262c56-49c0-4e6c-b402-b296b4a10d52.json: Language not supported
• lib/msal-node/src/client/ManagedIdentitySources/ServiceFabric.ts: Evaluated as low risk
• lib/msal-node/test/test_kit/StringConstants.ts: Evaluated as low risk
• lib/msal-node/docs/managed-identity.md: Evaluated as low risk

Comments suppressed due to low confidence (1)

lib/msal-node/src/utils/Constants.ts:12

• [nitpick] The constant name 'MACHINE_LEARNING_AND_SERVICE_FABRIC_SECRET_HEADER_NAME' is too long and unclear. Consider renaming it to 'ML_AND_SF_SECRET_HEADER_NAME'.

export const MACHINE_LEARNING_AND_SERVICE_FABRIC_SECRET_HEADER_NAME: string = "secret";

[gladjohn]

The environment variables we need to check are MSI_ENDPOINT and MSI_SECRET

Please also add a test where App Service 2019 and App Service 2017 (aka ML Source) are present. If so, App Service 2019 must be the detected source.

Copilot reviewed 5 out of 9 changed files in this pull request and generated no comments.

Files not reviewed (4)

• change/@azure-msal-node-ca262c56-49c0-4e6c-b402-b296b4a10d52.json: Language not supported
• lib/msal-node/src/client/ManagedIdentitySources/ServiceFabric.ts: Evaluated as low risk
• lib/msal-node/src/utils/Constants.ts: Evaluated as low risk
• lib/msal-node/test/client/ManagedIdentitySources/MachineLearning.spec.ts: Evaluated as low risk

Comments suppressed due to low confidence (1)

lib/msal-node/src/client/ManagedIdentitySources/MachineLearning.ts:16

• [nitpick] The constant name ML_AND_SF_SECRET_HEADER_NAME is ambiguous. It should be renamed to MACHINE_LEARNING_SECRET_HEADER_NAME to be more specific.

ML_AND_SF_SECRET_HEADER_NAME