From 608f7e775ffa65faa71668877252ee865020db37 Mon Sep 17 00:00:00 2001
From: mamrajyadav <themamraj0131@gmail.com>
Date: Fri, 8 Sep 2023 20:04:12 +0530
Subject: [PATCH] fix: add security group module count

---
 main.tf      | 3 ++-
 variables.tf | 6 ++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/main.tf b/main.tf
index e87b804..20cb39d 100644
--- a/main.tf
+++ b/main.tf
@@ -168,7 +168,7 @@ resource "aws_ec2_client_vpn_endpoint" "default" {
   split_tunnel           = var.split_tunnel_enable
   vpc_id                 = var.vpc_id
   session_timeout_hours  = var.session_timeout_hours
-  security_group_ids     = concat([aws_security_group.this.id], var.security_group_ids)
+  security_group_ids     = concat([aws_security_group.this[0].id], var.security_group_ids)
   vpn_port               = var.vpn_port
   self_service_portal    = var.self_service_portal
 
@@ -200,6 +200,7 @@ resource "aws_ec2_client_vpn_endpoint" "default" {
 #tfsec:ignore:aws-ec2-add-description-to-security-group
 #tfsec:ignore:aws-ec2-add-description-to-security-group-rule
 resource "aws_security_group" "this" {
+  count       = var.enabled && var.enable_security_group ? 1 : 0
   name_prefix = var.name
   vpc_id      = var.vpc_id
   tags        = module.labels.tags
diff --git a/variables.tf b/variables.tf
index b90d75d..b17c68c 100644
--- a/variables.tf
+++ b/variables.tf
@@ -214,3 +214,9 @@ variable "security_group_egress" {
   ]
   description = "List of maps of egress rules to set on the default security group"
 }
+
+variable "enable_security_group" {
+  type        = bool
+  default     = true
+  description = "create for security group module this value is enable 'true'"
+}