From 564aa7c1b8727a11f698d73c70f71684cbb68a2a Mon Sep 17 00:00:00 2001
From: nmemmert <67961292+nmemmert@users.noreply.github.com>
Date: Thu, 11 Dec 2025 14:25:08 -0500
Subject: [PATCH] Fix email stats rendering when avgResponseTime is non-numeric

---
 server/notificationService.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/server/notificationService.js b/server/notificationService.js
index 1f4e12a..0649dc9 100644
--- a/server/notificationService.js
+++ b/server/notificationService.js
@@ -129,11 +129,14 @@ class NotificationService {
       `;
 
       if (stats) {
+        const uptimePct = Number(stats.uptime);
+        const avgMs = Number(stats.avgResponseTime);
+
         htmlContent += `
           <div style="margin-top: 20px; padding: 15px; background-color: #f5f5f5; border-radius: 5px;">
             <h3 style="margin-top: 0;">Last 24 Hours Performance</h3>
-            <p><strong>Uptime:</strong> ${(stats.uptime * 100).toFixed(2)}%</p>
-            <p><strong>Avg Response Time:</strong> ${stats.avgResponseTime.toFixed(0)}ms</p>
+            <p><strong>Uptime:</strong> ${Number.isFinite(uptimePct) ? (uptimePct * 100).toFixed(2) : 'N/A'}%</p>
+            <p><strong>Avg Response Time:</strong> ${Number.isFinite(avgMs) ? avgMs.toFixed(0) : 'N/A'}ms</p>
             <p><strong>Last Check:</strong> ${stats.lastCheck || 'Never'}</p>
         `;