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

Support global & regional endpoints #30

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

Conversation

hvieira
Copy link

@hvieira hvieira commented Jul 15, 2022

When running in AWS and we want to use IAM roles for service accounts, we can use the ExAws.STS.AuthCache.AssumeRoleWebIdentityAdapter adapter, which relies on AWS_WEB_IDENTITY_TOKEN_FILE and AWS_ROLE_ARN to be present. These env vars are, as far as I know, are injected via the identity webhook - https://github.com/aws/amazon-eks-pod-identity-webhook#usage.
There are additional settings that can also be configured here, including whether or not to use regional endpoints.

As far as I understand from the existing code, as long as ExAWS is configured with a region, then it will use a regional endpoint - defaulting to us-east-1. However some regions do not support regional endpoints -
https://docs.aws.amazon.com/IAM/latest/UserGuide/id_credentials_temp_enable-regions.html#id_credentials_region-endpoints - and it is necessary to understand when to use (or not use) the regional endpoints in favor of the global endpoint.

Containers (or any process that wants to use assume role with web identity) can be configured to have an env variable - AWS_STS_REGIONAL_ENDPOINTS (see pod identity webhook docs) to depict whether or not to use the regional endpoints.

This PR is intended to accomplish 2 items:

  • Fix unit tests that were failing on xml parsing
  • add support for configuring whether or not to use regional endpoints

Please be aware that I don't have much experience with Elixir nor with ExAWS so there might be scenarios/behaviours that I'm not aware of that need to be addressed in the PR

@chrono
Copy link

chrono commented Nov 6, 2023

@vanetix @koozdra Is there any help required to get this merged?

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.

2 participants