From 1d5d8d96de3b11a352bef3c2e7fbf5e25645b733 Mon Sep 17 00:00:00 2001 From: Dragory <2606411+Dragory@users.noreply.github.com> Date: Wed, 3 Nov 2021 16:53:11 +0200 Subject: [PATCH] fix(phisherman): reset validApiKey on 'missing permissions' error; log getDomainInfo() errors --- .../src/plugins/Phisherman/functions/getDomainInfo.ts | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/backend/src/plugins/Phisherman/functions/getDomainInfo.ts b/backend/src/plugins/Phisherman/functions/getDomainInfo.ts index d5b9cbf3..8f61bd41 100644 --- a/backend/src/plugins/Phisherman/functions/getDomainInfo.ts +++ b/backend/src/plugins/Phisherman/functions/getDomainInfo.ts @@ -11,7 +11,14 @@ export async function getDomainInfo( return null; } - const info = await getPhishermanDomainInfo(domain).catch(() => null); + const info = await getPhishermanDomainInfo(domain).catch((err) => { + // tslint:disable-next-line:no-console + console.warn(`[PHISHERMAN] Error in getDomainInfo() for server ${pluginData.guild.id}: ${err.message}`); + if (err.message === "missing permissions") { + pluginData.state.validApiKey = null; + } + return null; + }); if (info != null && !phishermanDomainIsSafe(info)) { trackPhishermanCaughtDomain(pluginData.state.validApiKey, domain); }