Merge pull request #71 from schubergphilis/fvb/sg-hotfix

bug: Fix no VPC SG checks
schubergphilis · Mar 8, 2024 · 37ec884 · 37ec884
2 parents 9396bbc + 9571570
commit 37ec884
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/main.tf b/main.tf
@@ -9,7 +9,6 @@ locals {
   source_code_hash           = var.source_code_hash != null ? var.source_code_hash : var.filename != null ? filebase64sha256(var.filename) : null
   tracing_config             = var.tracing_config_mode != null ? { create : true } : {}
   vpc_config                 = var.subnet_ids != null ? { create : true } : {}
-  security_group_ids         = length(var.security_group_ids) > 0 ? var.security_group_ids : [aws_security_group.default[0].id]
 }
 
 data "aws_iam_policy_document" "default" {
@@ -205,7 +204,7 @@ resource "aws_lambda_function" "default" {
 
     content {
       subnet_ids         = var.subnet_ids
-      security_group_ids = local.security_group_ids
+      security_group_ids = length(var.security_group_ids) > 0 ? var.security_group_ids : [aws_security_group.default[0].id]
     }
   }