Skip to content

Add support for WorkloadIdentityCredential - #4

Merged
jepio merged 1 commit into
mainfrom
danzatt/workload-identity
Sep 5, 2025
Merged

Add support for WorkloadIdentityCredential#4
jepio merged 1 commit into
mainfrom
danzatt/workload-identity

Conversation

@danzatt

@danzatt danzatt commented Sep 5, 2025

Copy link
Copy Markdown
Collaborator

@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

chewi commented Sep 5, 2025

Copy link
Copy Markdown
Collaborator

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

jepio commented Sep 5, 2025

Copy link
Copy Markdown
Owner

@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