aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar MarkKoz <[email protected]>2020-06-15 10:52:01 -0700
committerGravatar MarkKoz <[email protected]>2020-06-15 10:52:01 -0700
commit94a4f8e52f52e98ea50fb0233fedcbbe9ebe6266 (patch)
tree0fffc0ee42ac59f2addc76805974aff1c0326c8e
parentWebhook remover: ignore DMs and bot messages (diff)
Webhook remover: exit early if message already deleted
-rw-r--r--bot/cogs/webhook_remover.py10
1 files changed, 8 insertions, 2 deletions
diff --git a/bot/cogs/webhook_remover.py b/bot/cogs/webhook_remover.py
index 74a353e98..543869215 100644
--- a/bot/cogs/webhook_remover.py
+++ b/bot/cogs/webhook_remover.py
@@ -1,7 +1,7 @@
import logging
import re
-from discord import Colour, Message
+from discord import Colour, Message, NotFound
from discord.ext.commands import Cog
from bot.bot import Bot
@@ -35,7 +35,13 @@ class WebhookRemover(Cog):
"""Delete `msg` and send a warning that it contained the Discord webhook `redacted_url`."""
# Don't log this, due internal delete, not by user. Will make different entry.
self.mod_log.ignore(Event.message_delete, msg.id)
- await msg.delete()
+
+ try:
+ await msg.delete()
+ except NotFound:
+ log.debug(f"Failed to remove webhook in message {msg.id}: message already deleted.")
+ return
+
await msg.channel.send(ALERT_MESSAGE_TEMPLATE.format(user=msg.author.mention))
message = (