update count logic

Author: abbas-khan10

infrastructure/buckets.tf

@@ -1,5 +1,5 @@
 locals {
-  access_logs_bucket_id = local.is_production ? aws_s3_bucket.access_logs[0].id : null
+  access_logs_bucket_id = local.is_production ? aws_s3_bucket.access_logs[0].id : ""
   access_logs_count     = local.is_production ? 1 : 0
 }
 
@@ -330,8 +330,8 @@ resource "aws_s3_bucket_policy" "logs_bucket_policy" {
 }
 
 resource "aws_s3_bucket_logging" "logs_bucket_logging" {
-  count         = local.access_logs_count > 0 ? 1 : 0
+  count         = local.access_logs_count
   bucket        = aws_s3_bucket.logs_bucket.id
-  target_bucket = aws_s3_bucket.access_logs[0].id
+  target_bucket = local.access_logs_bucket_id
   target_prefix = "${aws_s3_bucket.logs_bucket.id}/"
 }

infrastructure/modules/s3/README.md

@@ -36,7 +36,7 @@ No modules.
 
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
-| <a name="input_access_logs_bucket_id"></a> [access\_logs\_bucket\_id](#input\_access\_logs\_bucket\_id) | Enables access logs on the module's bucket | `string` | `null` | no |
+| <a name="input_access_logs_bucket_id"></a> [access\_logs\_bucket\_id](#input\_access\_logs\_bucket\_id) | Enables access logs on the module's bucket | `string` | n/a | yes |
 | <a name="input_access_logs_enabled"></a> [access\_logs\_enabled](#input\_access\_logs\_enabled) | n/a | `bool` | `false` | no |
 | <a name="input_bucket_name"></a> [bucket\_name](#input\_bucket\_name) | the name of the bucket | `string` | n/a | yes |
 | <a name="input_cloudfront_arn"></a> [cloudfront\_arn](#input\_cloudfront\_arn) | CloudFront Distribution ARN association and policy toggles | `string` | `"null"` | no |

infrastructure/modules/s3/variable.tf

@@ -50,6 +50,5 @@ variable "access_logs_enabled" {
 
 variable "access_logs_bucket_id" {
   type        = string
-  default     = null
   description = "Enables access logs on the module's bucket"
 }