Skip to content

add support for aws sso credentials provider #7527

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

efiShtain
Copy link

Why the changes in this PR are needed?

The PR adds the ability to work with aws sso credentials to load bundles from s3
It allows working locally with sso tokens, fetching credentials tokens when needed

I needed it to be able to work the same without changing the configuration on local (with sso credentials) and cloud (ec2/ecs)

Copy link
Contributor

@charlieegan3 charlieegan3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @efiShtain, thanks for this PR. It looks great to me.

Just so that I have this straight, this is to allow human users to pull bundles from s3 which are typically loaded by OPAs authenticated via some other means (e.g. instance identity document etc.). This would mainly be used for local developer testing?

Would it be possible to add some documentation around this new feature pointing to the relevant documentation e.g. https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-sso.html.

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.

3 participants