diff options
| author |  Mark <[email protected]> | 2020-03-08 23:05:33 -0700 |
|---|---|---|
| committer |  GitHub <[email protected]> | 2020-03-08 23:05:33 -0700 |
| commit | 3fe0f5cba6d3f8ba7016fc651ea12a0da7f9e5c5 (patch) | |
| tree | 58bb0a8c60f37b3137703157e7746857d689663e | |
| parent | Merge pull request #823 from python-discord/tag-search-searches-tags-via-cont... (diff) | |
| parent | Merge branch 'master' into antimalware-fix (diff) | |
Merge pull request #824 from python-discord/antimalware-fix
Fix filtered extension string out of scope for antismalware log message
| -rw-r--r-- | bot/cogs/antimalware.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/bot/cogs/antimalware.py b/bot/cogs/antimalware.py index 373619895..79bf486a4 100644 --- a/bot/cogs/antimalware.py +++ b/bot/cogs/antimalware.py @@ -29,8 +29,9 @@ class AntiMalware(Cog): return embed = Embed() - file_extensions = {splitext(message.filename.lower())[1] for message in message.attachments} + file_extensions = {splitext(attachment.filename.lower())[1] for attachment in message.attachments} extensions_blocked = file_extensions - set(AntiMalwareConfig.whitelist) + blocked_extensions_str = ', '.join(extensions_blocked) if ".py" in extensions_blocked: # Short-circuit on *.py files to provide a pastebin link embed.description = ( @@ -38,7 +39,6 @@ class AntiMalware(Cog): f"please use a code-pasting service such as {URLs.site_schema}{URLs.site_paste}" ) elif extensions_blocked: - blocked_extensions_str = ', '.join(extensions_blocked) whitelisted_types = ', '.join(AntiMalwareConfig.whitelist) meta_channel = self.bot.get_channel(Channels.meta) embed.description = ( |