Use AzureRegion while calculating key

[SaurabhMSFT]

Use AzureRegion while calculating key

Description

AzureRegion was used while calculating key in DefaultTokenAcquirerFactoryImplementation but not in TokenAcquisition.cs file. Now the GetKey method in TokenAcquisition calls the method with same name in DefaultTokenAcquirerFactoryImplementation . This will ensure that both methods rely on the same set of attributes while creating key.

Fixes #{3001194} (https://identitydivision.visualstudio.com/Engineering/_workitems/edit/3001194)

[bgavrilMS]

We should have a bug for this @SaurabhMSFT .

LGTM

[bgavrilMS]

Use AzureRegion while calculating key

Description

AzureRegion was used while calculating key in DefaultTokenAcquirerFactoryImplementation but not in TokenAcquisition.cs file. Now the GetKey method in TokenAcquisition calls the method with same name in DefaultTokenAcquirerFactoryImplementation . This will ensure that both methods rely on the same set of attributes while creating key.

Fixes #{3001194} (https://identitydivision.visualstudio.com/Engineering/_workitems/edit/3001194)

I'd be surprised if this is sufficient to fix the bug you referenced, but no objection to trying it out.

[SaurabhMSFT]

Use AzureRegion while calculating key

Description

AzureRegion was used while calculating key in DefaultTokenAcquirerFactoryImplementation but not in TokenAcquisition.cs file. Now the GetKey method in TokenAcquisition calls the method with same name in DefaultTokenAcquirerFactoryImplementation . This will ensure that both methods rely on the same set of attributes while creating key.
Fixes #{3001194} (https://identitydivision.visualstudio.com/Engineering/_workitems/edit/3001194)

I'd be surprised if this is sufficient to fix the bug you referenced, but no objection to trying it out.

This bug required two fixes. One in this branch and the second in https://identitydivision.visualstudio.com/DevEx/_git/MISE/pullrequest/14342