Skip to content

Set AppConfigurationAudience based on Cloud configuration when creating App Configuration SDK clients#2279

Merged
alzimmermsft merged 3 commits intomicrosoft:mainfrom
alzimmermsft:AddMissingAppConfigurationAudience
Mar 30, 2026
Merged

Set AppConfigurationAudience based on Cloud configuration when creating App Configuration SDK clients#2279
alzimmermsft merged 3 commits intomicrosoft:mainfrom
alzimmermsft:AddMissingAppConfigurationAudience

Conversation

@alzimmermsft
Copy link
Copy Markdown
Contributor

@alzimmermsft alzimmermsft commented Mar 30, 2026

What does this PR do?

Fixes authentication issues with App Configuration tools where they weren't setting AppConfigurationAudience based on the cloud configuration of Azure MCP, resulting in a mismatch audience issue.

Now all App Configuration SDKs have AppConfigurationAudience configured based on which cloud is being interacted with.

GitHub issue number?

[Link to the GitHub issue this PR addresses]

Pre-merge Checklist

  • Required for All PRs
    • Read contribution guidelines
    • PR title clearly describes the change
    • Commit history is clean with descriptive messages (cleanup guide)
    • Added comprehensive tests for new/modified functionality
    • Created a changelog entry if the change falls among the following: new feature, bug fix, UI/UX update, breaking change, or updated dependencies. Follow the changelog entry guide
  • For MCP tool changes:
    • One tool per PR: This PR adds or modifies only one MCP tool for faster review cycles
    • Updated servers/Azure.Mcp.Server/README.md and/or servers/Fabric.Mcp.Server/README.md documentation
    • Validate README.md changes running the script ./eng/scripts/Process-PackageReadMe.ps1. See Package README
    • For new or modified tool descriptions, ran ToolDescriptionEvaluator and obtained a score of 0.4 or more and a top 3 ranking for all related test prompts
    • For tools with new names, including new tools or renamed tools, update consolidated-tools.json
    • For renamed tools, follow the Tool Rename Checklist and tag the PR with the breaking-change label
    • For new tools associated with Azure services or publicly available tools/APIs/products, add URL to documentation in the PR description
  • Extra steps for Azure MCP Server tool changes:
    • Updated command list in servers/Azure.Mcp.Server/docs/azmcp-commands.md
    • Ran ./eng/scripts/Update-AzCommandsMetadata.ps1 to update tool metadata in azmcp-commands.md (required for CI)
    • Updated test prompts in servers/Azure.Mcp.Server/docs/e2eTestPrompts.md
    • 👉 For Community (non-Microsoft team member) PRs:
      • Security review: Reviewed code for security vulnerabilities, malicious code, or suspicious activities before running tests (crypto mining, spam, data exfiltration, etc.)
      • Manual tests run: added comment /azp run mcp - pullrequest - live to run Live Test Pipeline

Copy link
Copy Markdown
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 fixes Azure App Configuration authentication issues in sovereign clouds by configuring ConfigurationClientOptions.Audience based on the active Azure cloud configuration when creating App Configuration SDK clients.

Changes:

  • Set ConfigurationClientOptions.Audience according to TenantService.CloudConfiguration.CloudType when creating ConfigurationClient.
  • Pass the tenant argument through to the App Configuration store resource-graph lookup.
  • Add an Azure MCP Server changelog entry for the bug fix.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
tools/Azure.Mcp.Tools.AppConfig/src/Services/AppConfigService.cs Configures App Configuration SDK client audience based on cloud; threads tenant through store lookup; adds helper for audience mapping.
servers/Azure.Mcp.Server/changelog-entries/1774886640626.yaml Documents the bug fix in the server changelog entries.

Comment thread tools/Azure.Mcp.Tools.AppConfig/src/Services/AppConfigService.cs Outdated
Comment thread tools/Azure.Mcp.Tools.AppConfig/src/Services/AppConfigService.cs Outdated
Comment thread tools/Azure.Mcp.Tools.AppConfig/src/Services/AppConfigService.cs Outdated
Comment thread servers/Azure.Mcp.Server/changelog-entries/1774886640626.yaml
@github-project-automation github-project-automation Bot moved this from Untriaged to In Progress in Azure MCP Server Mar 30, 2026
@alzimmermsft alzimmermsft merged commit 073c70b into microsoft:main Mar 30, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Azure MCP Server Mar 30, 2026
@alzimmermsft alzimmermsft deleted the AddMissingAppConfigurationAudience branch March 30, 2026 18:45
alzimmermsft added a commit to alzimmermsft/mcp that referenced this pull request Mar 30, 2026
…ng App Configuration SDK clients (microsoft#2279)

* Set AppConfigurationAudience based on Cloud configuration when creating App Configuration SDK clients

* Pass tenant to credential retrieval

* Fix method typo
alzimmermsft added a commit that referenced this pull request Mar 30, 2026
…ng App Configuration SDK clients (#2287)

Set AppConfigurationAudience based on Cloud configuration when creating App Configuration SDK clients (#2279)

* Set AppConfigurationAudience based on Cloud configuration when creating App Configuration SDK clients

* Pass tenant to credential retrieval

* Fix method typo
colbytimm pushed a commit to colbytimm/microsoft-mcp that referenced this pull request Apr 20, 2026
…ng App Configuration SDK clients (microsoft#2279)

* Set AppConfigurationAudience based on Cloud configuration when creating App Configuration SDK clients

* Pass tenant to credential retrieval

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

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants