Update Identity version in attestation package to 2.0.0-beta.4

[KarishmaGhiya]

Update identity to 2.0.0-beta.4 for @azure/attestation

Related epic - #14581

Instructions:

• Replace tenant id. tenantId now can't be an invalid tenantId, so use the recorder's replaceableVariables to set one like 12345678-1234-1234-1234-123456789012.
• Re-record the NodeJS tests, since MSAL does some more recordings (one of them being that they verify that the Identity endpoint is available, which is something our previous Identity client didn't do).
• If you find any issues while upgrading, please report them in this issue.
• Reference PR - [Synapse Artifacts] Update identity version ad re do recordings #15704 and Updating identity v2 beta versions for identity tests and reverting to GA version v1 for samples #15654

[KarishmaGhiya]

You can use this PR as reference for attestation package - #15717

[LarryOsterman]

@KarishmaGhiya Playback is failing after updating recordings:

 CredentialUnavailableError: endpoints_resolution_error: Error: could not resolve endpoints. Please check network and try again. Detail: ClientAuthError: openid_config_error: Could not retrieve endpoints. Check your authority and verify the .well-known/openid-configuration endpoint returns the required endpoints. Attempted to retrieve endpoints from: https://login.microsoftonline.com/12345678-1234-1234-1234-1234567890ab/v2.0/.well-known/openid-configuration

Your instructions said "one like 1234...123456789012". I suspect that it must be 1234....123456789012, because 1234567890AB failed.

[HarshaNalluru]

Your instructions said "one like 1234...123456789012". I suspect that it must be 1234....123456789012, because 1234567890AB failed.

More info: msal library checks for the ids and would fail if it doesn't like the syntax.

[KarishmaGhiya]

@LarryOsterman msal library needs those to be just numbers not characters in the above format.

[LarryOsterman]

@LarryOsterman msal library needs those to be just numbers not characters in the above format.

This is REALLY confusing then. AZURE_TENANT_ID is a GUID, not an array of numbers. But the replacement string needs to be a set of numbers, not a GUID?

Is that intentional? If so, why??

[HarshaNalluru]

I believe msal expects tenant-id to be in that format. Do you have any specifics, @sadasant?