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

console command now supports $AWS_PROFILE #239

Merged
merged 1 commit into from
Jan 7, 2022
Merged

Conversation

synfinatic
Copy link
Owner

After all other existing checks, look to see if $AWS_PROFILE
is set and if so, use those credentials to open the console with.

Since not all profiles are necesssarily AWS SSO profiles, we will
try using the Go SDK if necessary to load our credentials instead
of the creds we have cached.

Fixes: #234

@codecov
Copy link

codecov bot commented Jan 7, 2022

Codecov Report

Merging #239 (68d4e0d) into main (373c99d) will increase coverage by 0.39%.
The diff coverage is 45.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
+ Coverage   43.47%   43.86%   +0.39%     
==========================================
  Files          11       11              
  Lines        1702     1721      +19     
==========================================
+ Hits          740      755      +15     
- Misses        898      902       +4     
  Partials       64       64              
Impacted Files Coverage Δ
sso/cache.go 38.86% <0.00%> (-0.42%) ⬇️
sso/cache_roles.go 67.75% <56.25%> (+2.10%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 373c99d...68d4e0d. Read the comment docs.

@synfinatic synfinatic force-pushed the console-profile branch 3 times, most recently from 37d2ea4 to 76228fa Compare January 7, 2022 05:23
After all other existing checks, look to see if $AWS_PROFILE
is set and if so, use those credentials to open the console with.

Since not all profiles are necesssarily AWS SSO profiles, we will
try using the Go SDK if necessary to load our credentials instead
of the creds we have cached.

Fixes: #234
@synfinatic synfinatic merged commit 7ff3712 into main Jan 7, 2022
@synfinatic synfinatic deleted the console-profile branch January 7, 2022 05:29
@synfinatic
Copy link
Owner Author

I even tested it :)

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.

console command should honor $AWS_PROFILE
1 participant