Skip to content

Commit 26fbe63

Browse files
committed
CCM-9874: review feedback
1 parent d3985f2 commit 26fbe63

File tree

4 files changed

+15
-16
lines changed

4 files changed

+15
-16
lines changed

infrastructure/terraform/components/app/cloudfront_distribution_main.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
resource "aws_cloudfront_distribution" "main" {
22
provider = aws.us-east-1
33

4-
enabled = true
5-
is_ipv6_enabled = true
6-
comment = "NHS Notify templates files CDN (${local.csi})"
7-
default_root_object = "index.html"
4+
enabled = true
5+
is_ipv6_enabled = true
6+
comment = "NHS Notify templates files CDN (${local.csi})"
7+
88
# https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-cloudfront-distribution-distributionconfig.html#cfn-cloudfront-distribution-distributionconfig-priceclass
99
price_class = "PriceClass_100"
1010

infrastructure/terraform/components/app/module_download_authorizer_lambda.tf

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -22,16 +22,15 @@ module "download_authorizer_lambda" {
2222
body = data.aws_iam_policy_document.authorizer.json
2323
}
2424

25-
function_s3_bucket = local.acct.s3_buckets["artefacts_us_east_1"]["id"]
26-
function_code_base_path = local.lambdas_source_code_dir
27-
function_code_dir = "download-authorizer/dist"
28-
handler_function_name = "handler"
29-
runtime = "nodejs20.x"
30-
memory = 128
31-
timeout = 3
32-
lambda_at_edge = true
33-
enable_lambda_insights = false
34-
force_lambda_code_deploy = true
25+
function_s3_bucket = local.acct.s3_buckets["artefacts_us_east_1"]["id"]
26+
function_code_base_path = local.lambdas_source_code_dir
27+
function_code_dir = "download-authorizer/dist"
28+
handler_function_name = "handler"
29+
runtime = "nodejs20.x"
30+
memory = 128
31+
timeout = 3
32+
lambda_at_edge = true
33+
enable_lambda_insights = false
3534
}
3635

3736
data "aws_iam_policy_document" "authorizer" {

lambdas/authorizer/jest.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import type { Config } from 'jest';
2-
import { baseJestConfig } from 'nhs-notify-web-template-management-utils'; // eslint-disable-line no-restricted-exports
2+
import { baseJestConfig } from 'nhs-notify-web-template-management-utils';
33

44
const config: Config = {
55
...baseJestConfig,

lambdas/download-authorizer/jest.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import type { Config } from 'jest';
2-
import { baseJestConfig } from 'nhs-notify-web-template-management-utils'; // eslint-disable-line no-restricted-exports
2+
import { baseJestConfig } from 'nhs-notify-web-template-management-utils';
33

44
const config: Config = {
55
...baseJestConfig,

0 commit comments

Comments
 (0)