From 2cff0b7147a19dfee80aabd112d9e025249082e8 Mon Sep 17 00:00:00 2001
From: nmemmert <67961292+nmemmert@users.noreply.github.com>
Date: Thu, 11 Dec 2025 14:23:23 -0500
Subject: [PATCH] Improve email alert logging for per-resource targets

---
 server/notificationService.js | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/server/notificationService.js b/server/notificationService.js
index 1aec8b4..1f4e12a 100644
--- a/server/notificationService.js
+++ b/server/notificationService.js
@@ -71,7 +71,15 @@ class NotificationService {
 
     // Use resource-specific email if provided, otherwise fall back to global config
     if (this.emailEnabled && (resource.email_to || this.config.email_to)) {
+      const chosenEmail = resource.email_to || this.config.email_to;
+      console.log(`[Alert] Email target for ${resource.name}: ${chosenEmail || 'none'} (enabled=${this.emailEnabled})`);
       promises.push(this.sendEmail(resource, message, incident.type, stats));
+    } else {
+      if (!this.emailEnabled) {
+        console.log(`[Alert] Email disabled via config; skipping email for ${resource.name}`);
+      } else {
+        console.log(`[Alert] No email target for ${resource.name}; skipping email`);
+      }
     }
 
     if (this.webhookEnabled) {
@@ -105,7 +113,7 @@ class NotificationService {
 
   async sendEmail(resource, message, type, stats = null) {
     if (!this.transporter) {
-      console.error('Email config incomplete; skipping email');
+      console.error(`[Email] Config incomplete; skipping email (enabled=${this.emailEnabled}, host=${this.config.email_host})`);
       return;
     }