Merge pull request #72 from companieshouse/s3-access-logs-module

marcransome · web-flow · commit 2cb9074d826e · 2024-05-07T10:15:22.000+01:00
Enable S3 server access logging
diff --git a/groups/storage/s3.tf b/groups/storage/s3.tf
@@ -42,3 +42,11 @@ resource "aws_s3_bucket_ownership_controls" "data" {
     object_ownership = "BucketOwnerEnforced"
   }
 }
+
+module "s3_access_logging" {
+  source = "git@github.com:companieshouse/terraform-modules//aws/s3_access_logging?ref=1.0.264"
+
+  aws_account           = var.aws_account
+  aws_region            = var.region
+  source_s3_bucket_name = aws_s3_bucket.data.id
+}

Original file line number	Diff line number	Diff line change
`@@ -42,3 +42,11 @@ resource "aws_s3_bucket_ownership_controls" "data" {`
`42`	`42`	`object_ownership = "BucketOwnerEnforced"`
`43`	`43`	`}`
`44`	`44`	`}`
	`45`	`+`
	`46`	`+module "s3_access_logging" {`
	`47`	`+ source = "[email protected]:companieshouse/terraform-modules//aws/s3_access_logging?ref=1.0.264"`
	`48`	`+`
	`49`	`+ aws_account = var.aws_account`
	`50`	`+ aws_region = var.region`
	`51`	`+ source_s3_bucket_name = aws_s3_bucket.data.id`
	`52`	`+}`