Merge pull request #8666 from ministryofjustice/feature/setting-up-la…

…nding-bucket Adding bucket policy
ministryofjustice · Nov 14, 2024 · 967dd32 · 967dd32
2 parents 827c755 + 94f0e7f
commit 967dd32
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/terraform/environments/analytical-platform-ingestion/ext-user-2024.tf b/terraform/environments/analytical-platform-ingestion/ext-user-2024.tf
@@ -69,7 +69,7 @@ data "aws_iam_policy_document" "ext_2024_target_bucket_policy" {
     ]
     resources = [
       "arn:aws:s3:::mojap-ingestion-${local.environment}-ext-2024-target/*",
-      "arn:aws:s3:::mojap-ingestion-${local.environment}-ext-2024-target/"
+      "arn:aws:s3:::mojap-ingestion-${local.environment}-ext-2024-target"
     ]
   }
 }
@@ -89,8 +89,8 @@ module "ext_2024_target_bucket" {
   versioning = {
     enabled = true
   }
-  # attach_policy = true
-  # policy        = data.aws_iam_policy_document.ext_2024_target_bucket_policy.json
+  attach_policy = true
+  policy        = data.aws_iam_policy_document.ext_2024_target_bucket_policy.json
 
   server_side_encryption_configuration = {
     rule = {