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

The assume role policy documentation on the README is confusing #334

Open
lazarillo opened this issue Jan 20, 2022 · 0 comments
Open

The assume role policy documentation on the README is confusing #334

lazarillo opened this issue Jan 20, 2022 · 0 comments

Comments

@lazarillo
Copy link

The README file says that an IAM role should be created with the following assume role policy (assume_role_policy attribute if you are using Terraform):

{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Action": [
        "sts:AssumeRole"
      ],
      "Effect": "Allow",
      "Resource": "*"
    }
  ]
}

However, assume role policies can only have the AssumeRole action associated with them, and these actions do not depend upon resources -- it is instead the role that is assumed which will have an associated policy that can impact resources. At least this is how I understand it looking from explanations like this one.

I find this all overly confusing, so I might be wrong. However, I think the README should instead say

{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Action": [
        "sts:AssumeRole"
      ],
      "Effect": "Allow",
    }
  ]
}

(Note that the "Resource" is gone.)

Am I correct? Would it be possible for someone who understands this clearly to make the example more elaborate? This is quite confusing to get my head around, and I doubt I am the only one.

Thanks!

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

No branches or pull requests

1 participant