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.

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

auth: Unverified fixes #1626

Closed
wants to merge 4 commits into from
Closed

auth: Unverified fixes #1626

wants to merge 4 commits into from

Conversation

alexweininger
Copy link
Member

A set of attempts/fixes for Brandon to use

@bwateratmsft
Copy link
Contributor

bwateratmsft commented Nov 20, 2023

I think I know what's remaining that isn't right--we never set the endpoint property on the subscription client, so it would only ever list subscriptions from the public cloud--but we're getting a token for the sovereign cloud, so it throws an error that the audience of the token is wrong.

We need to configure the subscription client's endpoint property.

That's what I get for trying to not depend on @microsoft/vscode-azext-azureutils...

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