fix

terraform-modules · Feb 29, 2024 · 5bddc8b · 5bddc8b
1 parent 97d7caa
commit 5bddc8b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/group-assignment/main.tf b/group-assignment/main.tf
@@ -59,7 +59,7 @@ resource "aws_ssoadmin_account_assignment" "accounts" {
 
 locals {
   ldap_access_dn = format("cn=%v,%v", local.name, var.ldap_sso_name, var.ldap_sso_base)
-  ldap_dn        = format("cn=%v,ou=SSO,%v,%v", local.name, var.ldap_sso_namevar.ldap_sso_base)
+  ldap_dn        = format("cn=%v,ou=SSO,%v,%v", local.name, var.ldap_sso_name, var.ldap_sso_base)
 }
 
 resource "ldap_object" "group" {

diff --git a/group-assignment/variables.tf b/group-assignment/variables.tf
@@ -106,7 +106,7 @@ variable "ldap_sso_name" {
   default     = null
 
   validation {
-    condition     = var.ldap_sso_name == null || contains(["ent-ew", "ent-gov", "lab-gov"], var.ldap_sso_name)
+    condition     = var.ldap_sso_name == null || try(contains(["ent-ew", "ent-gov", "lab-gov"], var.ldap_sso_name), false)
     error_message = "ldap_sso_name must be one of: ent-ew, ent-gov, lab-gov."
   }
 }