Skip to content

Commit 78a448a

Browse files
committed
CCM-13611: Allow access to Core PDF pipeline bucket for getletterdata lambda
1 parent a420f27 commit 78a448a

File tree

4 files changed

+14
-14
lines changed

4 files changed

+14
-14
lines changed
Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
resource "aws_lambda_event_source_mapping" "letter_updates_transformer_kinesis" {
2-
event_source_arn = aws_kinesis_stream.letter_change_stream.arn
3-
function_name = module.letter_updates_transformer.function_arn
4-
starting_position = "LATEST"
5-
batch_size = 10
6-
maximum_batching_window_in_seconds = 1
2+
event_source_arn = aws_kinesis_stream.letter_change_stream.arn
3+
function_name = module.letter_updates_transformer.function_arn
4+
starting_position = "LATEST"
5+
batch_size = 10
6+
maximum_batching_window_in_seconds = 1
77

88
depends_on = [
9-
module.letter_updates_transformer # ensures updates transformer exists
9+
module.letter_updates_transformer # ensures updates transformer exists
1010
]
1111
}

infrastructure/terraform/components/api/locals.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@ locals {
2929
DOWNLOAD_URL_TTL_SECONDS = 60
3030
}
3131

32-
core_pdf_bucket_arn = "arn:aws:s3:::comms-${var.core_account_id}-eu-west-2-${var.core_environment}-api-stg-pdf-pipeline"
33-
core_s3_kms_key_alias_name = "alias/comms-${var.core_environment}-api-s3"
32+
core_pdf_bucket_arn = "arn:aws:s3:::comms-${var.core_account_id}-eu-west-2-${var.core_environment}-api-stg-pdf-pipeline"
33+
core_s3_kms_key_alias_name = "alias/comms-${var.core_environment}-api-s3"
3434
}

infrastructure/terraform/components/api/module_lambda_get_letter_data.tf

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -71,12 +71,12 @@ data "aws_iam_policy_document" "get_letter_data_lambda" {
7171
statement {
7272
sid = "S3ListBucketForPresign"
7373
actions = [
74-
"s3:ListBucket"
74+
"s3:ListBucket"
7575
]
7676
resources = [
7777
module.s3bucket_test_letters.arn,
7878
local.core_pdf_bucket_arn
79-
]
79+
]
8080
}
8181

8282
statement {
@@ -88,7 +88,7 @@ data "aws_iam_policy_document" "get_letter_data_lambda" {
8888
resources = [
8989
"${module.s3bucket_test_letters.arn}/*",
9090
"${local.core_pdf_bucket_arn}/*",
91-
]
91+
]
9292
}
9393

9494
statement {
@@ -102,9 +102,9 @@ data "aws_iam_policy_document" "get_letter_data_lambda" {
102102
"arn:aws:kms:${var.region}:${var.core_account_id}:key/*"
103103
]
104104
condition {
105-
test = "ForAnyValue:StringEquals"
105+
test = "ForAnyValue:StringEquals"
106106
variable = "kms:ResourceAliases"
107-
values = [local.core_s3_kms_key_alias_name]
107+
values = [local.core_s3_kms_key_alias_name]
108108
}
109109
}
110110
}

infrastructure/terraform/components/api/variables.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ variable "core_account_id" {
142142
}
143143

144144
variable "core_environment" {
145-
type = string
145+
type = string
146146
description = "Environment of Core"
147147
default = "prod"
148148
}

0 commit comments

Comments
 (0)