Skip to content

Adding fix for still persistent #40 which still fired when user didn't exist - #70

Closed
rossdrew wants to merge 2 commits into
Broadcom:masterfrom
rossdrew:master
Closed

Adding fix for still persistent #40 which still fired when user didn't exist#70
rossdrew wants to merge 2 commits into
Broadcom:masterfrom
rossdrew:master

Conversation

@rossdrew

Copy link
Copy Markdown

No description provided.

@klehman-rally

Copy link
Copy Markdown
Contributor

Incorporated the line of code with the else "Unknown" in 1.2.0. The root cause of the condition in which this happens in that the user credentials for the connection aren't a SubscriptionAdmin, so that issue is taken care of in code preceding this. Also the warning is commented out as if the user is not a SubscriptionAdmin the warning output is just lengthy without offering any real value.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@rossdrew@klehman-rally