Changed mailing logic to get receivers out of database not envs anymore

Signed-off-by: Florian Oberndörfer <[email protected]>
amosproj · Dec 7, 2024 · 6ae49f9 · 6ae49f9
1 parent 68fccde
commit 6ae49f9
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/apps/backend/.env.example b/apps/backend/.env.example
@@ -10,5 +10,4 @@ MAIL_HOST=smtp.example.com
 MAIL_PORT=465
 MAIL_USER=[email protected]
 MAIL_PASSWORD=topsecret
-MAIL_FROM=[email protected]
-MAILING_LIST=[email protected],[email protected]
+MAIL_FROM=[email protected]
diff --git a/apps/backend/src/app/utils/mail/mail.service.ts b/apps/backend/src/app/utils/mail/mail.service.ts
@@ -28,8 +28,18 @@ export class MailService {
   }
 
   async sendAlertMail(alert: Alert) {
-    const receivers =
-      this.configService.getOrThrow<string>('MAILING_LIST').split(',') || [];
+    const receiverEntities = await this.getAllMailReceiver();
+
+    if (receiverEntities.length === 0) {
+      this.logger.log('No mail receivers found. Skipping sending mail');
+      return;
+    }
+    const receivers = receiverEntities
+      .map((receiver) => receiver.mail)
+      .join(',')
+      .split(',');
+
+    console.log('receivers', receivers);
 
     let reason = '';
     let description = '';