Merge pull request #4 from synapsestudios/kms-count-fix

dragonfleas · web-flow · commit eaf5dfa54e51 · 2023-01-24T11:51:39.000-06:00
add index to kms reference
diff --git a/log.tf b/log.tf
@@ -22,8 +22,8 @@ resource "aws_s3_bucket_server_side_encryption_configuration" "this-logs" {
 
   rule {
     apply_server_side_encryption_by_default {
-      kms_master_key_id = aws_kms_key.this.arn
-      sse_algorithm     = "aws:kms"
+      kms_master_key_id = var.use_kms ? aws_kms_key.this[0].arn : null
+      sse_algorithm     = var.use_kms ? "aws:kms" : "AES256"
     }
   }
 }
diff --git a/main.tf b/main.tf
@@ -38,7 +38,7 @@ resource "aws_s3_bucket_server_side_encryption_configuration" "this" {
 
   rule {
     apply_server_side_encryption_by_default {
-      kms_master_key_id = var.use_kms ? aws_kms_key.this.arn : null
+      kms_master_key_id = var.use_kms ? aws_kms_key.this[0].arn : null
       sse_algorithm     = var.use_kms ? "aws:kms" : "AES256"
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -22,8 +22,8 @@ resource "aws_s3_bucket_server_side_encryption_configuration" "this-logs" {`
`22`	`22`
`23`	`23`	`rule {`
`24`	`24`	`apply_server_side_encryption_by_default {`
`25`		`- kms_master_key_id = aws_kms_key.this.arn`
`26`		`- sse_algorithm = "aws:kms"`
	`25`	`+ kms_master_key_id = var.use_kms ? aws_kms_key.this[0].arn : null`
	`26`	`+ sse_algorithm = var.use_kms ? "aws:kms" : "AES256"`
`27`	`27`	`}`
`28`	`28`	`}`
`29`	`29`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ resource "aws_s3_bucket_server_side_encryption_configuration" "this" {`
`38`	`38`
`39`	`39`	`rule {`
`40`	`40`	`apply_server_side_encryption_by_default {`
`41`		`- kms_master_key_id = var.use_kms ? aws_kms_key.this.arn : null`
	`41`	`+ kms_master_key_id = var.use_kms ? aws_kms_key.this[0].arn : null`
`42`	`42`	`sse_algorithm = var.use_kms ? "aws:kms" : "AES256"`
`43`	`43`	`}`
`44`	`44`	`}`