Merge pull request #16 from clouddrove/PROX-100

add new enable_subscription variable
clouddrove · Jul 27, 2021 · 875a4a5 · 875a4a5
2 parents 329a2c6 + 1e6eada
commit 875a4a5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/main.tf b/main.tf
@@ -65,7 +65,7 @@ resource "aws_sns_topic" "default" {
 #Module      : SNS TOPIC SUBSCRIPTION
 #Description : Terraform module which creates SNS Topic Subscription resources on AWS
 resource "aws_sns_topic_subscription" "default" {
-  count = var.enabled && var.enable_topic ? 1 : 0
+  count = var.enabled && var.enable_subscription ? 1 : 0
 
   topic_arn                       = aws_sns_topic.default[count.index].arn
   protocol                        = var.protocol

diff --git a/variables.tf b/variables.tf
@@ -282,7 +282,7 @@ variable "subscription_delivery_policy" {
 
 variable "enable_sms_preference" {
   type        = bool
-  default     = true
+  default     = false
   description = "Boolean indicating whether or not to update SNS SMS Preference."
 }
 
@@ -292,6 +292,12 @@ variable "enable_topic" {
   description = "Boolean indicating whether or not to create topic."
 }
 
+variable "enable_subscription" {
+  type        = bool
+  default     = false
+  description = "Boolean indicating whether or not to create subscription."
+}
+
 variable "enable_sns" {
   type        = bool
   default     = false