Skip to content

Conversation

@heaths
Copy link
Member

@heaths heaths commented May 5, 2025

No description provided.

Copilot AI review requested due to automatic review settings May 5, 2025 19:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for authenticating with the Azure Developer CLI (azd) by expanding the DefaultAzureCredential functionality.

  • Added a new field and accompanying method in the DefaultAzureCredentialBuilder to include/exclude the Azure Developer CLI credential.
  • Extended the included credential types and token retrieval logic to support the AzureDeveloperCliCredential, including corresponding tests and changelog updates.

Reviewed Changes

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

File Description
sdk/identity/azure_identity/src/credentials/default_azure_credentials.rs Added inclusion/exclusion of Azure Developer CLI credential and updated credential selection logic.
sdk/identity/azure_identity/src/azure_developer_cli_credential.rs Introduced conversion from TokenCredentialOptions to AzureDeveloperCliCredentialOptions.
sdk/identity/azure_identity/CHANGELOG.md Updated changelog to document the addition of the Azure Developer CLI credential.

@azure-sdk
Copy link
Collaborator

azure-sdk commented May 5, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

azure_identity

@github-project-automation github-project-automation bot moved this from Untriaged to In Progress in Azure Identity SDK Improvements May 5, 2025
@heaths heaths merged commit 449f358 into Azure:main May 5, 2025
26 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Azure Identity SDK Improvements May 5, 2025
@heaths heaths deleted the azd-in-dac branch May 5, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure.Identity The azure_identity crate

Projects

Development

Successfully merging this pull request may close these issues.

3 participants