Keep notification settings in sync without restart

server/notificationService.js

@@ -4,19 +4,47 @@ require('dotenv').config();
 
 class NotificationService {
   constructor() {
-    this.emailEnabled = process.env.EMAIL_ENABLED === 'true';
-    this.webhookEnabled = process.env.WEBHOOK_ENABLED === 'true';
+    this.applyConfigFromEnv();
+  }
 
-    if (this.emailEnabled) {
+  applyConfigFromEnv() {
+    this.setConfig({
+      email_enabled: process.env.EMAIL_ENABLED === 'true',
+      email_host: process.env.EMAIL_HOST,
+      email_port: parseInt(process.env.EMAIL_PORT),
+      email_user: process.env.EMAIL_USER,
+      email_pass: process.env.EMAIL_PASS,
+      email_from: process.env.EMAIL_FROM,
+      email_to: process.env.EMAIL_TO,
+      webhook_enabled: process.env.WEBHOOK_ENABLED === 'true',
+      webhook_url: process.env.WEBHOOK_URL,
+    });
+  }
+
+  setConfig(config) {
+    this.emailEnabled = !!config.email_enabled;
+    this.webhookEnabled = !!config.webhook_enabled;
+    this.config = config;
+
+    // Recreate transporter when email config changes; skip if incomplete
+    if (
+      this.emailEnabled &&
+      config.email_host &&
+      config.email_port &&
+      config.email_user &&
+      config.email_pass
+    ) {
       this.transporter = nodemailer.createTransport({
-        host: process.env.EMAIL_HOST,
-        port: parseInt(process.env.EMAIL_PORT),
+        host: config.email_host,
+        port: parseInt(config.email_port),
         secure: false,
         auth: {
-          user: process.env.EMAIL_USER,
-          pass: process.env.EMAIL_PASS,
+          user: config.email_user,
+          pass: config.email_pass,
         },
       });
+    } else {
+      this.transporter = null;
     }
   }
 
@@ -39,10 +67,15 @@ class NotificationService {
   }
 
   async sendEmail(resource, message, type) {
+    if (!this.transporter) {
+      console.error('Email config incomplete; skipping email');
+      return;
+    }
+
     try {
       await this.transporter.sendMail({
-        from: process.env.EMAIL_FROM,
-        to: process.env.EMAIL_TO,
+        from: this.config.email_from,
+        to: this.config.email_to,
         subject: `Alert: ${resource.name} is ${type === 'started' ? 'DOWN' : 'UP'}`,
         text: message,
       });
@@ -54,7 +87,7 @@ class NotificationService {
 
   async sendWebhook(resource, message, type) {
     try {
-      await axios.post(process.env.WEBHOOK_URL, {
+      await axios.post(this.config.webhook_url || process.env.WEBHOOK_URL, {
         resource: resource.name,
         url: resource.url,
         status: type === 'started' ? 'down' : 'up',