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

use SSO name for config-profiles block #871

Merged
merged 2 commits into from
Jun 12, 2024
Merged

Conversation

synfinatic
Copy link
Owner

Deal with users with multiple SSOConfig's and who use config-profiles command. We no longer will replace the previous config block in the ~/.aws/config file.

Copy link

codecov bot commented Jun 5, 2024

Codecov Report

Attention: Patch coverage is 74.13793% with 15 lines in your changes missing coverage. Please review.

Project coverage is 82.93%. Comparing base (0d9616d) to head (4b100c7).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #871      +/-   ##
==========================================
+ Coverage   81.56%   82.93%   +1.36%     
==========================================
  Files          38       38              
  Lines        2723     2741      +18     
==========================================
+ Hits         2221     2273      +52     
+ Misses        427      383      -44     
- Partials       75       85      +10     
Flag Coverage Δ
unittests 82.93% <74.14%> (+1.36%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
internal/awsconfig/config.go 80.65% <100.00%> (+0.65%) ⬆️
sso/awssso_auth.go 85.29% <100.00%> (+0.26%) ⬆️
internal/predictor/predictor.go 84.78% <50.00%> (+2.76%) ⬆️
internal/utils/fileedit.go 70.48% <79.17%> (+1.48%) ⬆️
internal/helper/helper.go 69.00% <61.90%> (+40.28%) ⬆️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

@synfinatic synfinatic force-pushed the config-profile-sso-name branch from a9742cc to fdb9b48 Compare June 5, 2024 03:33
@synfinatic
Copy link
Owner Author

This is still not working, correctly because we will dump multiple SSO instances in each config block.

@synfinatic synfinatic modified the milestone: 1.16.0 Jun 5, 2024
@synfinatic synfinatic force-pushed the config-profile-sso-name branch from 1ce65ca to 19f8157 Compare June 8, 2024 23:48
Deal with users with multiple SSOConfig's and who use `config-profiles`
command.  We no longer will replace the previous config block in the
~/.aws/config file.
@synfinatic synfinatic force-pushed the config-profile-sso-name branch 6 times, most recently from bdd3a00 to 6397f79 Compare June 12, 2024 04:34
- Generate multiple SSO config blocks, one for each instances
- fix fish autocomplete for profile names
- improve unit tests

Fixes: #740, #696
@synfinatic synfinatic force-pushed the config-profile-sso-name branch from 6397f79 to 4b100c7 Compare June 12, 2024 04:56
@synfinatic synfinatic merged commit 471d35d into main Jun 12, 2024
6 of 7 checks passed
@synfinatic synfinatic deleted the config-profile-sso-name branch June 12, 2024 05:03
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.

1 participant