From a9e17714331d35d54407914f7232a53f807bc2e0 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Mon, 25 Jan 2021 15:44:02 +0100
Subject: [PATCH] Don't break completely when creating the digest fail for one
 user

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 lib/DigestSender.php | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/lib/DigestSender.php b/lib/DigestSender.php
index 0ec19b2dd..8341b1e6c 100644
--- a/lib/DigestSender.php
+++ b/lib/DigestSender.php
@@ -108,7 +108,15 @@ public function sendDigests(int $now): void {
 				continue;
 			}
 
-			$this->sendDigestForUser($user, $now, $timezone, $language);
+			try {
+				$this->sendDigestForUser($user, $now, $timezone, $language);
+			} catch (\Throwable $e) {
+				$this->logger->error('Exception occurred while sending user digest email', [
+					'exception' => $e,
+				]);
+			}
+			// We still update the digest time after an failed email,
+			// so it hopefully works tomorrow
 			$this->config->setUserValue($user, 'activity', 'digest', $timezoneDigestDay[$timezone]);
 		}
 	}