Skip to content

Commit d3c8b65

Browse files
committed
Removed unnecessary locals
1 parent 6efd755 commit d3c8b65

File tree

3 files changed

+3
-6
lines changed

3 files changed

+3
-6
lines changed

infrastructure/cloudwatch_rum.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ resource "aws_iam_policy" "cloudwatch_rum_cognito_access" {
4141
{
4242
"Effect" : "Allow",
4343
"Action" : "rum:PutRumEvents",
44-
"Resource" : "arn:aws:rum:${local.current_region}:${local.current_account_id}:appmonitor/${aws_rum_app_monitor.ndr[0].id}"
44+
"Resource" : "arn:aws:rum:${data.aws_region.current.region}:${data.aws_caller_identity.current.account_id}:appmonitor/${aws_rum_app_monitor.ndr[0].id}"
4545
}
4646
]
4747
})
@@ -65,7 +65,7 @@ resource "aws_cloudwatch_log_resource_policy" "rum_log" {
6565
"logs:DeleteLogGroup",
6666
"logs:DescribeLogGroups"
6767
],
68-
Resource = "arn:aws:logs:${local.current_region}:${local.current_account_id}:log-group:/aws/vendedlogs/RUMService-*"
68+
Resource = "arn:aws:logs:${data.aws_region.current.region}:${data.aws_caller_identity.current.account_id}:log-group:/aws/vendedlogs/RUMService-*"
6969
}
7070
]
7171
})

infrastructure/lambda-send-feedback.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ resource "aws_iam_policy" "ses_send_email_policy" {
104104
"ses:SendEmail",
105105
],
106106
Resource = [
107-
"arn:aws:ses:${local.current_region}:${local.current_account_id}:identity/*",
107+
"arn:aws:ses:${data.aws_region.current.region}:${data.aws_caller_identity.current.account_id}:identity/*",
108108
]
109109
}
110110
]

infrastructure/variable.tf

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,9 +203,6 @@ locals {
203203
api_gateway_subdomain_name = contains(["prod"], terraform.workspace) ? "${var.certificate_subdomain_name_prefix}" : "${var.certificate_subdomain_name_prefix}${terraform.workspace}"
204204
api_gateway_full_domain_name = contains(["prod"], terraform.workspace) ? "${var.certificate_subdomain_name_prefix}${var.domain}" : "${var.certificate_subdomain_name_prefix}${terraform.workspace}.${var.domain}"
205205

206-
current_region = data.aws_region.current.name
207-
current_account_id = data.aws_caller_identity.current.account_id
208-
209206
apim_api_url = "https://${var.apim_environment}api.service.nhs.uk/national-document-repository"
210207
}
211208

0 commit comments

Comments
 (0)