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

cli: fix more login bugs #111

Merged
merged 3 commits into from
Oct 12, 2023
Merged

cli: fix more login bugs #111

merged 3 commits into from
Oct 12, 2023

Conversation

pgavlin
Copy link
Member

@pgavlin pgavlin commented Oct 12, 2023

  • When running login in an environment where there are no existing credentials, also set the current pulumi account if using shared creds
  • When logging out, check to see if we're actually logged in before attempting to log out.

Fixes #100.

- When running `login` in an environment where there are no existing
  credentials, also set the current `pulumi` account if using shared
  creds
- When logging out, check to see if we're actually logged in before
  attempting to log out.

Fixes #100.
@pgavlin pgavlin requested review from komalali and removed request for AaronFriel October 12, 2023 16:00
@pgavlin pgavlin enabled auto-merge (squash) October 12, 2023 17:11
@pgavlin pgavlin merged commit 1a26137 into main Oct 12, 2023
@pgavlin pgavlin deleted the pgavlin/gh100 branch October 12, 2023 17:29
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.

esc login not working
2 participants