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

[WIP] Add optional external_id flag when using iam_role #3025

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lebenitza
Copy link

When assuming another IAM Role.
More here: https://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles_create_for-user_externalid.html

I marked this as WIP because I need help updating the tests. Any pointers or actual code is appreciated, I do not usually write much GoLang.

Description

Fixes #2036.

This was manually tested:

  • no IAM role to assume
  • just the IAM role
  • both the IAM role and ExternalId

TODOs

Read the Gruntwork contribution guidelines.

  • Update the docs.
  • Run the relevant tests successfully, including pre-commit checks.
  • Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added the ability to specify ExternalId when assuming another IAM Role through STS API.

…r IAM Role

Signed-off-by: Mihai Anei <mihai.anei@gmail.com>
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.

How to set iam-role external ID on multi-account-scenario for terragrunt use?
1 participant