From 76275edb967fcd1339dbd4d49df02eed685876ae Mon Sep 17 00:00:00 2001
From: badra001 <donald.e.badrak.ii@census.gov>
Date: Wed, 6 Sep 2023 14:22:46 -0400
Subject: [PATCH] change  enable_cloudwatch_logs default to true

---
 CHANGELOG.md            | 2 +-
 cloudtrail/README.md    | 2 +-
 cloudtrail/variables.tf | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 84ad158..7445364 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -285,6 +285,6 @@
 * 2.4.6 -- 2023-09-06
   - cloudtrail
     - update permissions for SNS, SQS to allow s3
-    - add enable_cloudwatch_logs with a default of false
+    - add enable_cloudwatch_logs with a default of true
   - cloudtrail-key
     - add s3.amazonaws.com 
diff --git a/cloudtrail/README.md b/cloudtrail/README.md
index 6910f8e..91fe9c9 100644
--- a/cloudtrail/README.md
+++ b/cloudtrail/README.md
@@ -207,7 +207,7 @@ No modules.
 | <a name="input_additional_sqs_names"></a> [additional\_sqs\_names](#input\_additional\_sqs\_names) | List of additional SQS queues to create and subscribe to the SNS topic (if enabled) | `list(string)` | `[]` | no |
 | <a name="input_cloudtrail_bucket_prefix"></a> [cloudtrail\_bucket\_prefix](#input\_cloudtrail\_bucket\_prefix) | Access log bucket prefix, to which the bucket name will be appended to make the target\_prefix | `string` | `"cloudtrail"` | no |
 | <a name="input_component_tags"></a> [component\_tags](#input\_component\_tags) | Additional tags for Components (s3, kms, ddb) | `map(map(string))` | <pre>{<br>  "ddb": {},<br>  "kms": {},<br>  "s3": {}<br>}</pre> | no |
-| <a name="input_enable_cloudwatch_logs"></a> [enable\_cloudwatch\_logs](#input\_enable\_cloudwatch\_logs) | Enable CloudWatch Logs for this CloudTrail | `bool` | `false` | no |
+| <a name="input_enable_cloudwatch_logs"></a> [enable\_cloudwatch\_logs](#input\_enable\_cloudwatch\_logs) | Enable CloudWatch Logs for this CloudTrail | `bool` | `true` | no |
 | <a name="input_enable_organization"></a> [enable\_organization](#input\_enable\_organization) | Enable CloudTrail as an organization trail. This will only work in the organization master account | `bool` | `false` | no |
 | <a name="input_enable_s3_sns"></a> [enable\_s3\_sns](#input\_enable\_s3\_sns) | Flag to enable or disable the creation of SNS for the Cloudtrail S3 bucket | `bool` | `false` | no |
 | <a name="input_enable_s3_sqs"></a> [enable\_s3\_sqs](#input\_enable\_s3\_sqs) | Flag to enable or disable the creation of SQS attached to SNS for Cloudtrail S3 bucket | `bool` | `false` | no |
diff --git a/cloudtrail/variables.tf b/cloudtrail/variables.tf
index ae1aa04..c74214f 100644
--- a/cloudtrail/variables.tf
+++ b/cloudtrail/variables.tf
@@ -111,5 +111,5 @@ variable "additional_s3_sqs_names" {
 variable "enable_cloudwatch_logs" {
   description = "Enable CloudWatch Logs for this CloudTrail"
   type        = bool
-  default     = false
+  default     = true
 }