From 59c0d4b4e46ae04ebcbd6df1ed50efc99382c273 Mon Sep 17 00:00:00 2001
From: Kevin Ansfield <kevin@lookingsideways.co.uk>
Date: Tue, 8 Feb 2022 18:36:06 +0000
Subject: [PATCH] Added Sentry error capture for failed email update after
 sending is complete

refs https://github.com/TryGhost/Team/issues/1321

- when updating the Email record after submitting all email batches we have a `catch` call but it was only logging the error
- added a call to Sentry so there's more visibility if those saves fail
---
 core/server/services/bulk-email/bulk-email-processor.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/core/server/services/bulk-email/bulk-email-processor.js b/core/server/services/bulk-email/bulk-email-processor.js
index da86bfed49..0e0f0ebdb2 100644
--- a/core/server/services/bulk-email/bulk-email-processor.js
+++ b/core/server/services/bulk-email/bulk-email-processor.js
@@ -130,6 +130,7 @@ module.exports = {
                 id: emailModel.id
             });
         } catch (err) {
+            sentry.captureException(err);
             logging.error(err);
         }