From acef2c8f861397ef21b45a916e39a4e6f203139f Mon Sep 17 00:00:00 2001 From: Michael-Phoenix <3187643-Michael-Phoenix@users.noreply.gitlab.com> Date: Wed, 7 Sep 2022 13:02:34 +0200 Subject: [PATCH] Handle scenarios where Lists get ivalid during export Stop trying to export all ban entries of a specific Export List when that list errors out. --- ban-importer/src/core.js | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/ban-importer/src/core.js b/ban-importer/src/core.js index 1dbdfcd..7ed546e 100644 --- a/ban-importer/src/core.js +++ b/ban-importer/src/core.js @@ -180,6 +180,7 @@ export default class Core { listChangeCount[exportBan.ExportBanList.id].count < DISCORD_ALERT_CAP; // Update the export bans. + let erroringLists = []; for (const exportBan of exportBans) { Logger.verbose( 'Core', @@ -190,15 +191,19 @@ export default class Core { ); try { - if (exportBan.status === 'TO_BE_CREATED') await Core.createExportBan(exportBan); - else await Core.deleteExportBan(exportBan); + if(!erroringLists.includes(exportBan.ExportBanList.id)) { + if (exportBan.status === 'TO_BE_CREATED') + await Core.createExportBan(exportBan); + else await Core.deleteExportBan(exportBan); + } } catch (err) { + erroringLists.push(exportBan.ExportBanList.id); Logger.verbose( 'Core', 1, `Failed to ${ exportBan.status === 'TO_BE_CREATED' ? 'create' : 'delete' - } export ban (ID: ${exportBan.id}): `, + } export ban (ID: ${exportBan.id}) - Ignoring exports for List ${exportBan.ExportBanList.id} for this run: `, err ); }