Feat: Dynamically using retention policy for SFTP's log group. (#38)

* feat: dynamically using retention policy for sftp log group

* fix- fixed spacing in variable file

---------

Co-authored-by: Anmol Nagpal <[email protected]>

Author: nileshgadgi

main.tf

@@ -45,7 +45,7 @@ data "aws_s3_bucket" "landing" {
 
 resource "aws_cloudwatch_log_group" "sftp_log_group" {
   name              = "/aws/transfer/${module.labels.id}"
-  retention_in_days = 90
+  retention_in_days = var.retention_in_days
 }
 
 ##----------------------------------------------------------------------------------

variables.tf

@@ -2,7 +2,6 @@
 #Module      : LABEL
 #Description : Terraform label module variables.
 ##----------------------------------------------------------------------------------
-
 variable "name" {
   type        = string
   default     = ""
@@ -49,21 +48,18 @@ variable "enabled" {
 #Module      : SFTP
 #Description : Terraform sftp module variables.
 ##----------------------------------------------------------------------------------
-
 variable "enable_sftp" {
   type        = bool
   default     = true
   description = "Set to false to prevent the module from creating any resources."
 }
 
-
 variable "identity_provider_type" {
   type        = string
   default     = "SERVICE_MANAGED"
   description = "The mode of authentication enabled for this service. The default value is SERVICE_MANAGED, which allows you to store and access SFTP user credentials within the service. API_GATEWAY."
 }
 
-
 variable "s3_bucket_name" {
   type        = string
   description = "This is the bucket that the SFTP users will use when managing files"
@@ -117,13 +113,18 @@ variable "subnet_ids" {
   default     = []
 }
 
-
 variable "security_policy_name" {
   type        = string
   description = "Specifies the name of the security policy that is attached to the server. Possible values are TransferSecurityPolicy-2018-11, TransferSecurityPolicy-2020-06, and TransferSecurityPolicy-FIPS-2020-06. Default value is: TransferSecurityPolicy-2018-11."
   default     = "TransferSecurityPolicy-2018-11"
 }
 
+variable "retention_in_days" {
+  type        = number
+  description = "Specifies the number of days you want to retain log events in the specified log group. Possible values are: 1, 3, 5, 7, 14, 30, 60, 90, 120, 150, 180, 365, 400, 545, 731, 1096, 1827, 2192, 2557, 2922, 3288, 3653, and 0. If you select 0, the events in the log group are always retained and never expire."
+  default     = 3
+}
+
 variable "domain_name" {
   type        = string
   description = "Domain to use when connecting to the SFTP endpoint"