Merge pull request #219 from NASA-IMPACT/fix/assume-role-arns

Check length of assume role arns
NASA-IMPACT · Aug 14, 2024 · f252f3d · f252f3d
2 parents 4a6391e + b91871d
commit f252f3d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/infrastructure/main.tf b/infrastructure/main.tf
@@ -95,8 +95,8 @@ resource "local_file" "mwaa_variables" {
       ecs_cluster_name        = module.mwaa.cluster_name
       log_group_name          = module.mwaa.log_group_name
       mwaa_execution_role_arn = module.mwaa.mwaa_role_arn
-      assume_role_read_arn    = var.assume_role_arns[0]
-      assume_role_write_arn   = var.assume_role_arns[0]
+      assume_role_read_arn    = length(var.assume_role_arns) > 0 ? var.assume_role_arns[0] : ""
+      assume_role_write_arn   = length(var.assume_role_arns) > 0 ? var.assume_role_arns[1] : ""
       account_id              = local.account_id
       aws_region              = local.aws_region
       cognito_app_secret      = var.workflows_client_secret