Skip to content

Commit dc1430f

Browse files
committed
eli-384 further permissions changes
1 parent fde341e commit dc1430f

File tree

3 files changed

+24
-17
lines changed

3 files changed

+24
-17
lines changed

infrastructure/stacks/api-layer/waf.tf

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -196,6 +196,7 @@ resource "aws_cloudwatch_log_resource_policy" "waf" {
196196
Version = "2012-10-17"
197197
Statement = [
198198
{
199+
Sid = "AWSLogDeliveryWrite"
199200
Effect = "Allow"
200201
Principal = {
201202
Service = "delivery.logs.amazonaws.com"
@@ -205,6 +206,23 @@ resource "aws_cloudwatch_log_resource_policy" "waf" {
205206
"logs:PutLogEvents"
206207
]
207208
Resource = "${aws_cloudwatch_log_group.waf[0].arn}:*"
209+
Condition = {
210+
StringEquals = {
211+
"aws:SourceAccount" = data.aws_caller_identity.current.account_id
212+
}
213+
ArnLike = {
214+
"aws:SourceArn" = "arn:aws:logs:${var.default_aws_region}:${data.aws_caller_identity.current.account_id}:*"
215+
}
216+
}
217+
},
218+
{
219+
Sid = "AWSLogDeliveryAclCheck"
220+
Effect = "Allow"
221+
Principal = {
222+
Service = "delivery.logs.amazonaws.com"
223+
}
224+
Action = "logs:GetLogDelivery"
225+
Resource = "*"
208226
}
209227
]
210228
})

infrastructure/stacks/iams-developer-roles/github_actions_policies.tf

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,11 @@ resource "aws_iam_policy" "api_infrastructure" {
240240
"acm:ListCertificates",
241241
# WAF v2 list operations
242242
"wafv2:ListWebACLs",
243-
"wafv2:ListTagsForResource"
243+
"wafv2:ListTagsForResource",
244+
# CloudWatch Logs resource policies (require wildcard)
245+
"logs:PutResourcePolicy",
246+
"logs:DeleteResourcePolicy",
247+
"logs:DescribeResourcePolicies"
244248

245249
],
246250
Resource = "*"

infrastructure/stacks/iams-developer-roles/iams_permissions_boundary.tf

Lines changed: 1 addition & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -170,22 +170,7 @@ data "aws_iam_policy_document" "permissions_boundary" {
170170
"lambda:ListProvisionedConcurrencyConfigs",
171171

172172
# CloudWatch Logs - log management
173-
"logs:CreateLogGroup",
174-
"logs:DeleteLogGroup",
175-
"logs:CreateLogStream",
176-
"logs:PutLogEvents",
177-
"logs:DescribeLogGroups",
178-
"logs:DescribeLogStreams",
179-
"logs:Describe*",
180-
"logs:ListTagsForResource",
181-
"logs:TagResource",
182-
"logs:UntagResource",
183-
"logs:PutRetentionPolicy",
184-
"logs:AssociateKmsKey",
185-
"logs:PutMetricFilter",
186-
"logs:PutSubscriptionFilter",
187-
"logs:DeleteSubscriptionFilter",
188-
"logs:DescribeSubscriptionFilters",
173+
"logs:*",
189174

190175
# S3 - bucket and object management
191176
"s3:GetLifecycleConfiguration",

0 commit comments

Comments
 (0)