[PRM-624] Remove logging_config block from "aws_cloudfront_distribution" (#528)

Author: chrisbloe

infrastructure/modules/cloudfront/main.tf

@@ -49,14 +49,6 @@ resource "aws_cloudfront_distribution" "distribution" {
     }
   }
   web_acl_id = var.web_acl_id
-
-  dynamic "logging_config" {
-    for_each = var.log_bucket_id != "" ? [1] : []
-    content {
-      bucket = var.log_bucket_id
-      prefix = "cloudfront/"
-    }
-  }
 }
 
 resource "aws_cloudfront_distribution" "distribution_with_secondary_bucket" {
@@ -104,15 +96,6 @@ resource "aws_cloudfront_distribution" "distribution_with_secondary_bucket" {
       event_type = "origin-request"
       lambda_arn = var.qualifed_arn
     }
-
-  }
-
-  dynamic "logging_config" {
-    for_each = var.log_bucket_id != "" ? [1] : []
-    content {
-      bucket = var.log_bucket_id
-      prefix = "cloudfront/"
-    }
   }
 
   viewer_certificate {
@@ -126,7 +109,6 @@ resource "aws_cloudfront_distribution" "distribution_with_secondary_bucket" {
     }
   }
   web_acl_id = var.web_acl_id
-
 }
 
 

infrastructure/modules/dynamo_db/README.md

@@ -106,7 +106,7 @@ module "document_reference_dynamodb_table" {
 | <a name="input_hash_key"></a> [hash\_key](#input\_hash\_key) | Primary partition key for the table. | `string` | `null` | no |
 | <a name="input_owner"></a> [owner](#input\_owner) | Identifies the team or person responsible for the resource (used for tagging). | `string` | n/a | yes |
 | <a name="input_point_in_time_recovery_enabled"></a> [point\_in\_time\_recovery\_enabled](#input\_point\_in\_time\_recovery\_enabled) | Enables PITR for backups. | `bool` | `false` | no |
-| <a name="input_sort_key"></a> [sort\_key](#input\_sort\_key) | Optional sort key for composite primary key. | `string` | `null` | no |
+| <a name="input_sort_key"></a> [sort\_key](#input\_sort\_key) | Optional range/sort key for composite primary key. | `string` | `null` | no |
 | <a name="input_stream_enabled"></a> [stream\_enabled](#input\_stream\_enabled) | Whether DynamoDB Streams are enabled. | `bool` | `false` | no |
 | <a name="input_stream_view_type"></a> [stream\_view\_type](#input\_stream\_view\_type) | Type of stream view (e.g., OLD\_IMAGE). | `string` | `"NEW_AND_OLD_IMAGES"` | no |
 | <a name="input_table_name"></a> [table\_name](#input\_table\_name) | Name of the DynamoDB table. | `string` | `null` | no |