Skip to content

Conversation

danzatt
Copy link
Collaborator

@danzatt danzatt commented Sep 5, 2025

@jepio jepio merged commit c72d89b into main Sep 5, 2025
1 check passed
@jepio jepio deleted the danzatt/workload-identity branch September 5, 2025 07:38
@chewi
Copy link
Collaborator

chewi commented Sep 5, 2025

I was going to say we should probably move AzureCliCredential below ManagedIdentityCredential while we're at it to avoid the annoying errors and because that's what DefaultAzureCredential does, but you were too quick!

@jepio
Copy link
Owner

jepio commented Sep 5, 2025

@chewi The order was set this way because of this issue: Azure/azure-sdk-for-cpp#4952

This was particularly painful in WSL. If someone were to validate that this is no longer a problem (regardless of that specific issue being closed) then we could change the order.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants