fix: Knew I should have not accepted this change request

Author: bryantbiggs

modules/service/main.tf

@@ -1638,7 +1638,7 @@ locals {
 }
 
 data "aws_subnet" "this" {
- count = local.create_security_group && var.vpc_id == null ? 1 : 0
+ count = local.create_security_group ? 1 : 0
 
  region = var.region
 
@@ -1653,7 +1653,7 @@ resource "aws_security_group" "this" {
  name = var.security_group_use_name_prefix ? null : local.security_group_name
  name_prefix = var.security_group_use_name_prefix ? "${local.security_group_name}-" : null
  description = var.security_group_description
- vpc_id = try(data.aws_subnet.this[0].vpc_id, var.vpc_id)
+ vpc_id = coalesce(var.vpc_id, try(data.aws_subnet.this[0].vpc_id, null))
 
  tags = merge(
  var.tags,