Skip to content

Conversation

@jackofallops
Copy link
Member

Community Note

  • Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
  • Please do not leave comments along the lines of "+1", "me too" or "any updates", they generate extra noise for PR followers and do not help prioritize for review

Description

add capability of authorising at tenant for cli even if subscriptionhint is known.

Enhances tenantid validation

refactors newAzureCliConfig for more simple signature.

This is a (please select all that apply):

  • Bug Fix
  • New Feature
  • Enhancement
  • Breaking Change

Rollback Plan

If a change needs to be reverted, we will publish an updated version of the provider.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

Note

If this PR changes meaningfully during the course of review please update the title and description as required.

@jackofallops jackofallops requested a review from a team as a code owner October 16, 2025 11:39
@github-actions github-actions bot added the release-once-merged The SDK should be released once this PR is merged label Oct 16, 2025
@jackofallops jackofallops removed the release-once-merged The SDK should be released once this PR is merged label Oct 16, 2025
Copy link
Contributor

@sreallymatt sreallymatt left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@jackofallops jackofallops merged commit d325ef0 into main Oct 16, 2025
5 checks passed
@jackofallops jackofallops deleted the f/allow-cli-to-auth-at-tenant branch October 16, 2025 16: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