From de6da81dfb6c838a9871a24b7b87e888c515f996 Mon Sep 17 00:00:00 2001 From: Phil Booth Date: Tue, 2 Apr 2019 17:31:39 +0100 Subject: [PATCH] fix(scripts): ignore bouncing accounts in verification reminders --- scripts/verification-reminders.js | 19 +++++++++++++------ 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/scripts/verification-reminders.js b/scripts/verification-reminders.js index df372ac40..85a1a1307 100644 --- a/scripts/verification-reminders.js +++ b/scripts/verification-reminders.js @@ -79,12 +79,19 @@ async function run () { }); sent[uid] = true; } catch (err) { - if (err.errno === error.ERRNO.ACCOUNT_UNKNOWN) { - console.log(` * ignoring deleted account ${uid}`); - } else { - console.log(` * failed ${uid}`); - console.error(err.stack); - failed.push({ timestamp, uid }); + const { errno } = err; + switch (errno) { + case error.ERRNO.ACCOUNT_UNKNOWN: + case error.ERRNO.BOUNCE_COMPLAINT: + case error.ERRNO.BOUNCE_HARD: + case error.ERRNO.BOUNCE_SOFT: + console.log(` * ignoring deleted/bouncing account ${uid}, errno: ${errno}`); + await verificationReminders.delete(uid); + break; + default: + console.log(` * failed ${uid}, errno: ${errno}`); + console.error(err.stack); + failed.push({ timestamp, uid }); } }