From 712b61d35c47afc2240e61aec6ad3883d12d78f1 Mon Sep 17 00:00:00 2001
From: ashle001 <roy.d.ashley.jr@census.gov>
Date: Thu, 24 Sep 2020 11:07:51 -0400
Subject: [PATCH] fix logging prefix

---
 main.tf | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/main.tf b/main.tf
index 161d7ce..bca66e1 100644
--- a/main.tf
+++ b/main.tf
@@ -52,7 +52,8 @@ resource "aws_s3_bucket" "this" {
 
   logging {
     target_bucket = var.access_log_bucket
-    target_prefix = format("%s/%s/", var.access_log_bucket_prefix, var.access_log_bucket)
+    target_prefix = format("%s/%s/", var.access_log_bucket_prefix, var.bucket_name)
+    #target_prefix = format("%s/%s/", var.access_log_bucket_prefix, var.access_log_bucket)
   }
 
   lifecycle {
@@ -99,9 +100,11 @@ data "aws_iam_policy_document" "this" {
   }
   statement  { 
     sid         = "enforceSSL"
-    effe        = "Deny"
-    principals  =  "*"
-    actions     = "s3:*"
+    effect      = "Deny"
+    actions     = ["s3:*"]
+    principals {
+      type        =  "AWS"
+      identifiers =  ["*"]
     resources   = [aws_s3_bucket.this.arn, "${aws_s3_bucket.this.arn}/*"]
     condition {
          test     = "Bool"
@@ -180,4 +183,4 @@ resource "aws_kms_alias" "key" {
   name          = "alias/${local.kms_key_name}"
   target_key_id = aws_kms_key.key.key_id
 }
-
+}