Skip to content

Commit 6a36bd9

Browse files
authored
Merge pull request #27 from companieshouse/security-remediation-around-root-level-write-access
make root level read only but add /tmp for writing
2 parents 13efae9 + 2f9e10e commit 6a36bd9

File tree

1 file changed

+2
-1
lines changed
  • terraform/groups/ecs-service

1 file changed

+2
-1
lines changed

terraform/groups/ecs-service/main.tf

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,8 @@ module "ecs-service" {
6464
docker_repo = local.docker_repo
6565
container_version = var.limited_partnerships_api_version
6666
container_port = local.container_port
67+
volumes = [ { "name": "tmp" } ]
68+
mount_points = [ { "sourceVolume": "tmp", "containerPath": "/tmp", "readOnly": false } ]
6769

6870
# Service configuration
6971
service_name = local.service_name
@@ -80,7 +82,6 @@ module "ecs-service" {
8082
use_capacity_provider = var.use_capacity_provider
8183
use_fargate = var.use_fargate
8284
fargate_subnets = local.application_subnet_ids
83-
read_only_root_filesystem = false
8485

8586
# Cloudwatch
8687
cloudwatch_alarms_enabled = var.cloudwatch_alarms_enabled

0 commit comments

Comments
 (0)