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.

Sign up for GitHub

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

Jump to bottom

Set config "profile" prefix as optional #61

Merged
merged 2 commits into from
Jul 29, 2022
Merged

Set config "profile" prefix as optional #61

merged 2 commits into from
Jul 29, 2022

Conversation

nitrocode
Copy link
Contributor

@nitrocode nitrocode commented Jul 16, 2022 •
edited
Loading

what

  • Set config "profile" prefix as optional

why

  • AWS allows profile prefix and without that prefix

references

✗ echo '[profile default]' > ~/.aws/config
✗ aws2-wrap
'sso_start_url' not found in profile: {'profile_name': 'default'}

✗ echo '[default]' > ~/.aws/config
✗ aws2-wrap
'sso_start_url' not found in profile: {'profile_name': 'default'}

@pcolmer pcolmer merged commit 892638a into linaro-its:master Jul 29, 2022
@pcolmer
Copy link
Contributor

pcolmer commented Jul 29, 2022

Thanks, @nitrocode

@nitrocode nitrocode deleted the with-or-without-profile branch July 31, 2022 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

cannot handle default profile
2 participants
@nitrocode @pcolmer