Skip to content

Commit 7583889

Browse files
committed
eli-510 adding additional permissions to allow assume role
1 parent 0251c52 commit 7583889

File tree

3 files changed

+14
-7
lines changed

3 files changed

+14
-7
lines changed

infrastructure/stacks/api-layer/csoc_log_forwarding.tf

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,7 @@ data "aws_iam_policy_document" "cwl_subscription_assume_role" {
1414

1515
principals {
1616
type = "Service"
17-
identifiers = ["logs.${var.default_aws_region}.amazonaws.com"]
18-
}
19-
20-
condition {
21-
test = "StringLike"
22-
variable = "aws:SourceArn"
23-
values = ["${module.eligibility_signposting_api_gateway.cloudwatch_destination_arn}:*"]
17+
identifiers = ["logs.amazonaws.com"]
2418
}
2519

2620
condition {

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

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -266,6 +266,17 @@ resource "aws_iam_policy" "api_infrastructure" {
266266
"arn:aws:logs:${var.default_aws_region}:${data.aws_caller_identity.current.account_id}:log-group:/aws/kinesisfirehose/*"
267267
]
268268
},
269+
{
270+
Effect = "Allow",
271+
Action = [
272+
# CloudWatch Logs subscription to CSOC cross-account destination
273+
"logs:PutSubscriptionFilter"
274+
],
275+
Resource = [
276+
# CSOC cross-account destination for API Gateway logs
277+
"arn:aws:logs:${var.default_aws_region}:693466633220:destination:api_gateway_log_destination"
278+
]
279+
},
269280
{
270281
Effect = "Allow",
271282
Action = [
@@ -464,6 +475,7 @@ resource "aws_iam_policy" "iam_management" {
464475
"iam:CreateRole",
465476
"iam:DeleteRole",
466477
"iam:UpdateRole",
478+
"iam:UpdateAssumeRolePolicy",
467479
"iam:PutRolePolicy",
468480
"iam:PutRolePermissionsBoundary",
469481
"iam:AttachRolePolicy",

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ data "aws_iam_policy_document" "permissions_boundary" {
101101
"iam:CreateRole",
102102
"iam:DeleteRole",
103103
"iam:UpdateRole",
104+
"iam:UpdateAssumeRolePolicy",
104105
"iam:PutRolePolicy",
105106
"iam:PutRolePermissionsBoundary",
106107
"iam:AttachRolePolicy",

0 commit comments

Comments
 (0)