Skip to content
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

Jump to bottom

{Profile} az account list: Fix: --refresh returns user as null #20430

Merged
merged 1 commit into from
Nov 23, 2021
Merged

{Profile} az account list: Fix: --refresh returns user as null #20430

merged 1 commit into from
Nov 23, 2021

Conversation

jiasli
Copy link
Member

@jiasli jiasli commented Nov 19, 2021 •
edited
Loading

Description

After MSAL migration (#19853), SubscriptionFinder no longer maintains user_id, so refresh_accounts should use existing user_id instead of reading from SubscriptionFinder.

Testing Guide

# PowerShell
# Use a clean config dir if you want
$env:AZURE_CONFIG_DIR='D:\cli\azure2'
az login
az account list --refresh

Also see

@yonzhan
Copy link
Collaborator

yonzhan commented Nov 19, 2021

Profile

@yonzhan yonzhan added this to the Nov 2021 (2021-12-07) milestone Nov 19, 2021
@yonzhan yonzhan requested a review from calvinhzy November 19, 2021 03:26
@jiasli
Copy link
Member Author

jiasli commented Nov 22, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@jiasli jiasli merged commit 6208421 into Azure:dev Nov 23, 2021
@jiasli jiasli deleted the refresh branch November 23, 2021 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@evelyn-ys evelyn-ys evelyn-ys approved these changes

@calvinhzy calvinhzy calvinhzy approved these changes

@jsntcy jsntcy Awaiting requested review from jsntcy jsntcy is a code owner

@kairu-ms kairu-ms Awaiting requested review from kairu-ms kairu-ms is a code owner

@zhoxing-ms zhoxing-ms Awaiting requested review from zhoxing-ms zhoxing-ms is a code owner

Assignees

@jiasli jiasli

Labels
None yet
Projects
None yet
Milestone
Nov 2021 (2021-12-07)
Development

Successfully merging this pull request may close these issues.

Service principal context left in inconsistent state after az account list --refresh
4 participants
@jiasli @yonzhan @evelyn-ys @calvinhzy