fix(api): handle failing to send push notification (#930)

services/api/src/core/notifications/client/gateways/notification.web-push.gateway.ts

@@ -1,13 +1,16 @@
 import { Injectable } from "@nestjs/common";
 import { sendNotification, setVapidDetails } from "web-push";
 
+import { Logger } from "src/core/logging";
 import { getRequiredStringConfig } from "src/utils/config.helper";
 
 import { NotificationSubscriptionService } from "../../domain/services/notification-subscription.service";
 import { INotification } from "../../types";
 
 @Injectable()
 export class NotificationWebPushGateway {
+  private logger = new Logger(NotificationWebPushGateway.name);
+
   constructor(
     private readonly notificationSubscriptionService: NotificationSubscriptionService,
   ) {
@@ -29,12 +32,20 @@ export class NotificationWebPushGateway {
       return false;
     }
 
-    const result = await sendNotification(
-      pushSubscription,
-      JSON.stringify(notification),
-    );
-
-    return result.statusCode < 400;
+    try {
+      const result = await sendNotification(
+        pushSubscription,
+        JSON.stringify(notification),
+      );
+
+      return result.statusCode < 400;
+    } catch (error) {
+      this.logger.error("Unable to send push notification", {
+        error,
+        notification,
+      });
+      return false;
+    }
   }
 
   public async sendWebPush(