Fix notifications send

ybizeul · Oct 14, 2024 · f7739d1 · f7739d1
1 parent 4e76f58
commit f7739d1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/internal/feed/pushnotifications.go b/internal/feed/pushnotifications.go
@@ -23,12 +23,16 @@ func (f *Feed) sendPushNotification() error {
 	// For each subscription we send the notification
 	for _, subscription := range f.Config.Subscriptions {
 		pnL.Logger.Debug("Sending push notification", slog.String("endpoint", subscription.Endpoint))
-		resp, _ := webpush.SendNotification([]byte(fmt.Sprintf("New item posted to feed %s", f.Name())), &subscription, &webpush.Options{
+		resp, err := webpush.SendNotification([]byte(fmt.Sprintf("New item posted to feed %s", f.Name())), &subscription, &webpush.Options{
 			Subscriber:      "ybfeed@tynsoe.org", // Do not include "mailto:"
 			VAPIDPublicKey:  f.NotificationSettings.VAPIDPublicKey,
 			VAPIDPrivateKey: f.NotificationSettings.VAPIDPrivateKey,
 			TTL:             30,
 		})
+		if err != nil {
+			pnL.Logger.Error("Sending push notification failed:", slog.String("error", err.Error()))
+			continue
+		}
 		if pnL.Level() == slog.LevelDebug {
 			b, _ := io.ReadAll(resp.Body)
 			pnL.Logger.Debug("Response", slog.String("resp", string(b)), slog.String("status", resp.Status))