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

feat: Add AWS Web Idenity compability to PROPAGATE_AWS_AUTH_TOKENS #249

Merged
merged 2 commits into from
May 30, 2023

Conversation

zopanix
Copy link

@zopanix zopanix commented May 23, 2023

This adds compatibilty with AWS authentication using delegated authentication using web identity token files (useful with AWS EKS for example). It adds an environment variable and a volume if the AWS_WEB_IDENTITY_TOKEN_FILE is present to be able to use them from the specified container.

In addition, we have added the AWS_STS_REGION_ENDPOINTS variable compatibility which allows you AWS IAM calls to be directed towards the region you are in instead of redirecting automatically to us-east-1 by default.

If you need more explanation or have concerns with the code, I'll be happy to address and/or correct them if needed :)

This adds compatibilty with AWS authentication using delegated
authentication using web identity token files (useful with AWS EKS for
example). It adds an environment variable and a volume if the
AWS_WEB_IDENTITY_TOKEN_FILE is present to be able to use them from the
specified container.

In addition, we have added the AWS_STS_REGION_ENDPOINTS variable
compatibility which allows you AWS IAM calls to be directed towards
the region you are in instead of redirecting automatically to us-east-1
by default.
@toote
Copy link
Contributor

toote commented May 30, 2023

Thanks a lot for the contribution!

@toote toote merged commit 4b82bf5 into buildkite-plugins:master May 30, 2023
@toote toote mentioned this pull request May 30, 2023
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