validation + test

navapbc · Nov 8, 2024 · f2d43bf · f2d43bf
1 parent 52643a5
commit f2d43bf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/infra/app/app-config/env-config/notifications.tf b/infra/app/app-config/env-config/notifications.tf
@@ -10,7 +10,7 @@ locals {
 
     # Set to the email address to be used when sending emails.
     # If enable_notifications is true, this is required.
-    sender_email = "coilysiren@${var.domain_name}"
+    sender_email = "coilysiren@not-my-domain-name.com"
 
     # Configure the REPLY-TO email address if it should be different from the sender.
     reply_to_email = "coilysiren@${var.domain_name}"

diff --git a/infra/modules/notifications/resources/variables.tf b/infra/modules/notifications/resources/variables.tf
@@ -11,6 +11,11 @@ variable "name" {
 variable "sender_email" {
   type        = string
   description = "Email address to use to send notification emails"
+
+  validation {
+    condition     = regex("^.+@${var.domain_name}$", var.sender_email)
+    error_message = "The sender_email value must be an email address from the configured domain."
+  }
 }
 
 variable "sender_display_name" {