Skip to content

fix: replace DefaultAzureCredential with ManagedIdentityCredential#431

Merged
Prajwal-Microsoft merged 1 commit intomainfrom
fix/managed-identity-upgrade
Jul 31, 2025
Merged

fix: replace DefaultAzureCredential with ManagedIdentityCredential#431
Prajwal-Microsoft merged 1 commit intomainfrom
fix/managed-identity-upgrade

Conversation

@Priyanka-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

  • ...
    This pull request introduces a new helper class to centralize Azure credential management and updates the configuration logic to use this helper. The changes improve maintainability and make the credential selection process more flexible based on the environment.

Azure Credential Management:

  • New helper class: Added AzureCredentialHelper in App/backend-api/Microsoft.GS.DPS.Host/Helpers/AzureCredentialHelper.cs to encapsulate logic for obtaining Azure credentials. It supports both development and production environments, with an optional client ID for user-assigned managed identities.

Configuration Updates:

  • Refactored App Configuration setup: Updated the Config method in AppConfiguration.cs to use AzureCredentialHelper.GetAzureCredential() instead of directly instantiating DefaultAzureCredential. This improves flexibility and aligns with the new helper class.
  • Namespace import: Added a reference to Microsoft.GS.DPSHost.Helpers in AppConfiguration.cs to enable the use of the new helper class.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

@Prajwal-Microsoft Prajwal-Microsoft merged commit 5516f95 into main Jul 31, 2025
5 checks passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Priyanka-Microsoft Priyanka-Microsoft deleted the fix/managed-identity-upgrade branch December 25, 2025 09:21
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.

2 participants