Skip to content
This repository has been archived by the owner on Dec 30, 2024. It is now read-only.

Commit

Permalink
Fix cfn-nag violations #204
Browse files Browse the repository at this point in the history
  • Loading branch information
JimTharioAmazon committed Mar 4, 2021
1 parent 1d15fe7 commit d373b28
Showing 1 changed file with 32 additions and 9 deletions.
41 changes: 32 additions & 9 deletions source/msam/build/msam-iam-roles-release.template
Original file line number Diff line number Diff line change
Expand Up @@ -219,9 +219,14 @@
"Metadata": {
"cfn_nag": {
"rules_to_suppress": [{
"id": "W11",
"reason": "This role is used by a scanner requiring access to all resources within these services."
}]
"id": "W11",
"reason": "This role is used by a scanner requiring access to all resources within these services."
},
{
"id": "W76",
"reason": "This role is used by a scanner requiring access to all resources within these services."
}
]
}
}
},
Expand Down Expand Up @@ -305,9 +310,18 @@
"Metadata": {
"cfn_nag": {
"rules_to_suppress": [{
"id": "W12",
"reason": "Resource ARNs are not known in advance."
}]
"id": "W12",
"reason": "Resource ARNs are not known in advance."
},
{
"id": "F39",
"reason": "Resource ARNs are not known in advance."
},
{
"id": "F4",
"reason": "This policy is used by compartmentalized teams to install the solution."
}
]
}
}
},
Expand Down Expand Up @@ -340,9 +354,18 @@
"Metadata": {
"cfn_nag": {
"rules_to_suppress": [{
"id": "W13",
"reason": "Resource ARNs are not known in advance."
}]
"id": "W13",
"reason": "Resource ARNs are not known in advance."
},
{
"id": "F5",
"reason": "This policy is used by compartmentalized teams to install the solution."
},
{
"id": "F39",
"reason": "This policy is used by compartmentalized teams to install the solution."
}
]
}
}
}
Expand Down

0 comments on commit d373b28

Please sign in to comment.