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

Permit suppression of the aws configure sso example command #9132

Open
2 tasks
richardkmichael opened this issue Dec 11, 2024 · 2 comments
Open
2 tasks

Permit suppression of the aws configure sso example command #9132

richardkmichael opened this issue Dec 11, 2024 · 2 comments
Labels
configure feature-request A feature should be added or improved. p3 This is a minor priority issue sso

Comments

@richardkmichael
Copy link

richardkmichael commented Dec 11, 2024

Describe the feature

Concerning the example printed at the end of aws configure sso:

To use this profile, specify the profile name using --profile, as shown:

aws s3 ls --profile myProfile

Either:

  • Add a flag to suppress it; or,
  • Simply remove it, as it seems noisy to use a flag, and a suitably terse flag name is not obvious.

Use Case

In our AWS configuration, ListBuckets is prohibited, so a user receives AccessDenied when running this command.

This confuses our staff (especially newcomers to the AWS CLI), who follow our setup instructions, run the suggested command (believing it to be a test), which then fails and they have the impression they made a mistake in the setup.

Proposed Solution

No response

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CLI version used

aws-cli/2.22.4 Python/3.12.7 Darwin/23.6.0 source/arm64

Environment details (OS name and version, etc.)

macOS 14

@richardkmichael richardkmichael added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Dec 11, 2024
@ashovlin ashovlin added the sso label Dec 12, 2024
@tim-finnigan tim-finnigan self-assigned this Dec 12, 2024
@tim-finnigan
Copy link
Contributor

Thanks for the feature request — I think the example command is helpful to show how users can reference the new profile. That being said, maybe there's a better example to use there than S3 — maybe aws sts get-caller-identity. Will leave this open for further tracking/discussion.

@tim-finnigan tim-finnigan added configure p3 This is a minor priority issue labels Dec 12, 2024
@tim-finnigan tim-finnigan removed their assignment Dec 12, 2024
@tim-finnigan tim-finnigan removed the needs-triage This issue or PR still needs to be triaged. label Dec 12, 2024
@richardkmichael
Copy link
Author

@tim-finnigan I agree an example is helpful, and I considered suggesting a replacement but I couldn't think of one off-hand that would be almost certain to succeed. Yours is a good one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
configure feature-request A feature should be added or improved. p3 This is a minor priority issue sso
Projects
None yet
Development

No branches or pull requests

3 participants