Modify rule S6249: fix Terraform code examples #4502
Conversation
GabinL21
changed the title
| Principal = [ | ||
| "arn:aws:iam::123456789123:root" | ||
| ] # secondary location: only one principal is forced to use https | ||
| Principal = { |
There was a problem hiding this comment.
The previous code example was "right" too, but we're not currently detecting it with our rule implementation.
This change ensures that we're documenting a case that is working with our current implementation, while the implementation itself will be updated in other tickets.
There was a problem hiding this comment.
Pasting the noncompliant example does not raise an issue in the current implementation (if I didn't do any mistake)
There was a problem hiding this comment.
Nevermind, was a mistake on my setup.
|
|
| Principal = [ | ||
| "arn:aws:iam::123456789123:root" | ||
| ] # secondary location: only one principal is forced to use https | ||
| Principal = { |
There was a problem hiding this comment.
Pasting the noncompliant example does not raise an issue in the current implementation (if I didn't do any mistake)
| Principal = [ | ||
| "arn:aws:iam::123456789123:root" | ||
| ] # secondary location: only one principal is forced to use https | ||
| Principal = { |
There was a problem hiding this comment.
Nevermind, was a mistake on my setup.
GabinL21
changed the title
GabinL21
changed the title
Review
A dedicated reviewer checked the rule description successfully for: