Skip to content

Commit 2f2689b

Browse files
committed
sec(iac): remove SecretsManagerDescribe wildcard block (closes #430)
secretsmanager:ListSecrets on Resource="*" in SecretsManagerDescribe was a duplicate of the ListSecrets already in the SecretsManager block scoped to cudly-* ARNs. The wildcard grant let the CI/CD role enumerate all secret names in the account. Remove the over-privileged statement.
1 parent 8f9347e commit 2f2689b

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

‎terraform/environments/aws/ci-cd-permissions/policy_data.tf‎

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -189,12 +189,6 @@ resource "aws_iam_policy" "data" {
189189
]
190190
Resource = "arn:aws:secretsmanager:*:*:secret:cudly-*"
191191
},
192-
{
193-
Sid = "SecretsManagerDescribe"
194-
Effect = "Allow"
195-
Action = ["secretsmanager:ListSecrets"]
196-
Resource = "*"
197-
},
198192
{
199193
Sid = "S3TerraformState"
200194
Effect = "Allow"

0 commit comments

Comments
 (0)