Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Cross account assume roles won't work #17

Closed
jufemaiz opened this issue Aug 9, 2019 · 0 comments · Fixed by #18
Closed

Cross account assume roles won't work #17

jufemaiz opened this issue Aug 9, 2019 · 0 comments · Fixed by #18

Comments

@jufemaiz
Copy link
Member

jufemaiz commented Aug 9, 2019

authorised_account_ids = (length(var.authorised_account_ids) == 0 ?
var.authorised_account_ids :
[data.aws_caller_identity.current.account_id])
authorised_account_root_arns = formatlist("arn:aws:iam::%s:root", local.authorised_account_ids)

Should be:

  authorised_account_ids = (length(var.authorised_account_ids) == 0 ?
    [data.aws_caller_identity.current.account_id] :
    var.authorised_account_ids)
  authorised_account_root_arns = formatlist("arn:aws:iam::%s:root", local.authorised_account_ids)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant