Skip to content

fix(coding-agent): use UserTenantId for credential resolution #7078

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Rajesh Kamal (rajeshkamal5050) merged 1 commit into from
Mar 12, 2026
Merged

fix(coding-agent): use UserTenantId for credential resolution #7078

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/azd/extensions/azure.coding-agent/internal/cmd/coding_agent_config.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ func runConfigCommand(cmd *cobra.Command, flagValues *flagValues) error {
return fmt.Errorf("failed getting a subscription from prompt. Try logging in manually with 'azd auth login' before running this command %w", err)
}

tenantID := subscriptionResponse.Subscription.TenantId
tenantID := subscriptionResponse.Subscription.UserTenantId
subscriptionID := subscriptionResponse.Subscription.Id
Comment on lines +202 to 203

@JeffreyCA JeffreyCA Mar 12, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we verify other extensions don't have the same issue too?

@vhvb1989 Victor Vazquez (vhvb1989) Mar 12, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'll turn the engines on it :)

@rajeshkamal5050 Rajesh Kamal (rajeshkamal5050) Mar 12, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

😂

@vhvb1989 Victor Vazquez (vhvb1989) Mar 12, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Audited all extensions. Found and fixed the same bug in 5 more:

Extension Files Status
azure.ai.models custom.go, init.go (2 spots) ❌ Fixed
azure.ai.agents init.go, init_from_code.go ❌ Fixed
azure.ai.finetune init.go (2 spots) ❌ Fixed
microsoft.azd.ai.builder start.go ❌ Fixed
microsoft.azd.demo prompt.go ❌ Fixed
azure.appservice swap.go ✅ Already correct (uses LookupTenant)
azure.ai.agents parser.go, service_target_agent.go ✅ Already correct (uses LookupTenant)

Pushed in b5c9ebf.

@richardpark-msft Richard Park (richardpark-msft) Mar 12, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It was me the entire time! :|


cred, err := azidentity.NewAzureDeveloperCLICredential(&azidentity.AzureDeveloperCLICredentialOptions{
Comment thread
vhvb1989 marked this conversation as resolved.
Expand Down
Loading