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

Update docs on AWS SSO #236

Merged
merged 1 commit into from May 7, 2022
Merged

Update docs on AWS SSO #236

merged 1 commit into from May 7, 2022

Conversation

ghost
Copy link

@ghost ghost commented May 6, 2022

My org is adopting AWS SSO and we use this tool, so I thought I’d check
whether the info here in the README was still accurate.

My SSO profile does include the property credential_process but for
the purposes of this test I commented it out, because I wanted to see if
the version of the AWS SDK that this program is currently using would
support SSO sessions directly, rather than via credential_process.

I made sure I had a valid and working SSO profile and session via
aws sso login and aws s3 ls which both succeeded.

Then I ran cw ls groups --profile <my-profile> and it succeeded — the
program printed a list of groups.

This is with cw 4.1.1, installed via Homebrew from the tap, as per the
docs, on MacOS 12.3.1 on an Apple Silicon (ARM64/aarch64) Mac.

My org is adopting AWS SSO and we use this tool, so I thought I’d check
whether the info here in the README was still accurate.

My SSO profile does include the property `credential_process` but for
the purposes of this test I commented it out, because I wanted to see if
the version of the AWS SDK that this program is currently using would
support SSO sessions directly, rather than via `credential_process`.

I made sure I had a valid and working SSO profile and session via
`aws sso login` and `aws s3 ls` which both succeeded.

Then I ran `cw ls groups --profile <my-profile>` and it succeeded — the
program printed a list of groups.

This is with cw 4.1.1, installed via Homebrew from the tap, as per the
docs, on MacOS 12.3.1 on an Apple Silicon (ARM64/aarch64) Mac.
@lucagrulla lucagrulla merged commit 138fcd0 into lucagrulla:master May 7, 2022
@lucagrulla
Copy link
Owner

Thanks for testing this out. Great to hear the SDK has been fixed.

@ghost
Copy link
Author

ghost commented May 8, 2022

My pleasure!

@ghost ghost deleted the patch-1 branch May 8, 2022 16:12
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