Merge pull request #41 from angstwad/fix-bucket-policy

Omitting log_location_prefix creates an invalid bucket policy

Author: brandonjbjelland

data.tf

@@ -6,7 +6,7 @@ data "aws_iam_policy_document" "bucket_policy" {
   statement {
     sid       = "AllowToPutLoadBalancerLogsToS3Bucket"
     actions   = ["s3:PutObject"]
-    resources = ["arn:aws:s3:::${var.log_bucket_name}/${var.log_location_prefix}/AWSLogs/${data.aws_caller_identity.current.account_id}/*"]
+    resources = ["arn:aws:s3:::${var.log_bucket_name}/${var.log_location_prefix == "" ? "" : format("%s/", var.log_location_prefix)}AWSLogs/${data.aws_caller_identity.current.account_id}/*"]
 
     principals {
       type        = "AWS"