Skip to content
This repository was archived by the owner on Feb 12, 2025. It is now read-only.

GetTokenFromCLIWithParams fixed empty error message if command invoke failed. #653

Merged
jhendrixMSFT merged 4 commits into from
Nov 5, 2021
Merged

GetTokenFromCLIWithParams fixed empty error message if command invoke failed. #653

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
d287e78
Fixed empty error message if cli invoke failed
StiviiK Sep 3, 2021
ccff193
Addendum to last commit
StiviiK Sep 3, 2021
c0e4158
Merge branch 'master' into master
jhendrixMSFT Sep 10, 2021
e84d947
Check stderr for empty string
StiviiK Sep 21, 2021
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
6 changes: 5 additions & 1 deletion autorest/azure/cli/token.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,11 @@ func GetTokenFromCLIWithParams(params GetAccessTokenParams) (*Token, error) {

output, err := cliCmd.Output()
if err != nil {
return nil, fmt.Errorf("Invoking Azure CLI failed with the following error: %s", stderr.String())
if _, ok := err.(*exec.ExitError); ok {
return nil, fmt.Errorf("Invoking Azure CLI failed with the following error: %s", stderr.String())
}

return nil, fmt.Errorf("Invoking Azure CLI failed with the following error: %s", err.Error())
}

tokenResponse := Token{}
Expand Down