aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Clean up codeGravatar mbaruh2021-08-21-4/+2
| | | | Removed unused import, corrected docstring, and removed unnedded type annotation.
* Fix MessageCache slicing bugs, improve testsGravatar mbaruh2021-08-21-26/+41
|
* Identify DeletionContext by members, not channelsGravatar mbaruh2021-08-21-17/+14
| | | | | | | | Since the anti-spam now works cross-channels, it makes no sense to identify it by the channel in which it was invoked. The DeletionContext class was changed to accept a frozenset of members, and the message_deletion_queue dict uses the frozensets as keys. DeletionContext still accepts a channel on creation, because while it might get added more channels, there's only one channel in which the mute message will be sent. Using members as the key can run into the issue of one member becoming irrelevant to the filter while others still are, resulting in another log message being sent, but it's an unlikely edge case since the users should be muted almost immediately, and we're currently not using any multi-member filters in the first place.
* AntiSpam deletes from all spammed channelsGravatar mbaruh2021-08-21-7/+16
| | | | The anti-spam cog was amended to handle cross-channel spam.
* AntiSpam modified to work with cacheGravatar mbaruh2021-08-21-9/+406
| | | | | | | | | | | | | The anti-spam cog now uses a cache instead of reading channel history. The cache is for all channels in the guild, and does not remove deleted messages. That means that the anti-spam logic now works cross-channel and counts deleted messages. The size of the cache is determined via a new field in the config YAML file. The cache was implemented as a separate class, MessageCache, which uses circular buffer logic. This allows for constant time addition and removal form either side, and lookup. The cache does not support removal from the middle of the cache. The cache additionally stores a mapping from message ID's to the index of the message in the cache, to allow constant time lookup by message ID. The commit additionally adds accompanying tests, and renames `cache.py` to `caching.py` to better distinguish it from the new `message_cache.py` and convey that it's for general caching utilities.
* Move max_interval to initGravatar Boris Muratov2021-08-17-8/+8
| | | | The on_message event calculated the max interval value every time for no reason. The value is constant throughout the bot's up time.
* Merge #1752 - help channels: fix initialisation of available channels setGravatar Mark2021-08-16-14/+8
|\
| * Merge branch 'main' into bug/help/1715/avail-set-initGravatar Mark2021-08-16-54/+74
| |\ | |/ |/|
* | Merge pull request #1753 from D0rs4n/pr/deletewebhookGravatar ChrisJL2021-08-16-7/+16
|\ \ | | | | | | Delete Webhooks when one is accidentally posted.
| * \ Merge branch 'main' into pr/deletewebhookGravatar ChrisJL2021-08-16-47/+58
| |\ \ | |/ / |/| |
* | | Merge pull request #1716 from ↵Gravatar ChrisJL2021-08-16-46/+57
|\ \ \ | | | | | | | | | | | | | | | | python-discord/bug/info/bot-13b/code-snippet-msg-404 Don't send code snippets if the original message was deleted
| * \ \ Merge branch 'main' into bug/info/bot-13b/code-snippet-msg-404Gravatar ChrisJL2021-08-16-282/+584
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1754 from python-discord/wookie184-patch-1Gravatar Boris Muratov2021-08-16-1/+1
|\ \ \ \ | | | | | | | | | | Remove the admins mention from the modmail tag
| * | | | Remove the admins mention from the modmail tagGravatar wookie1842021-08-16-1/+1
|/ / / /
| * | | CodeSnippets: don't send snippets if the original message was deletedGravatar MarkKoz2021-08-03-3/+7
| | | | | | | | | | | | | | | | Fixes BOT-13B
| * | | CodeSnippets: refactor snippet parsing into a separate functionGravatar MarkKoz2021-08-03-8/+12
| | | |
| * | | CodeSnippets: move __init__ to the top of the class definitionGravatar MarkKoz2021-08-03-11/+11
| | | |
| * | | CodeSnippets: refactor on_messageGravatar MarkKoz2021-08-03-35/+38
| | | | | | | | | | | | | | | | Reduce nesting and code duplication.
| | * | Improve output message consistency in webhook_removerGravatar D0rs4n2021-08-16-1/+1
| | | | | | | | | | | | Co-authored-by: Bluenix <[email protected]>
| | * | Patch the regEx pattern in webhook_remover to match edge casesGravatar D0rs4n2021-08-16-1/+4
| | | | | | | | | | | | | | | | | | | | The regEx pattern did not match the 'https://' part in case of canary urls Thus, from now on the regEx pattern matches the full canary urls as well.
| | * | Fix up, and improve code consistency in webhook_removerGravatar D0rs4n2021-08-16-2/+2
| | | |
| | * | Improve code consistency in webhook_removerGravatar D0rs4n2021-08-16-9/+7
| | | |
| | * | Enhance security in the Webhook remover.Gravatar D0rs4n2021-08-16-6/+14
| |/ / |/| | | | | | | | | | | From now on, webhooks that were posted in the chat will be also deleted from Discord in order to eliminate the risk.
| | * HelpChannels: use utility method to get available channelsGravatar MarkKoz2021-08-16-3/+1
| | |
| | * HelpChannels: simplify formatting of category names in dormant msgGravatar MarkKoz2021-08-15-9/+6
| | | | | | | | | | | | | | | Remove redundant retrieval of category objects. Include the asking guide URL in the message via an f-string.
| | * HelpChannels: fix incomplete init of available channel setGravatar MarkKoz2021-08-15-5/+4
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the cog is reloaded while there are less than the maximum amount of available channels, it makes some channels available until the limit is reached. When a channel is made available, it updates the `available_help_channels` set. The `update_available_help_channels()` function would not update this set if it saw that the set already contains elements. This resulted in only the channels that were just made available being in the set; the set would not contain the channels that were already available when the bot started. Fix this by unconditionally populating the set, but moving it to `init_available()` so it only happens once. Fix BOT-Z1 Fix #1715
* | Merge pull request #1677 from python-discord/mbaruh/jam-managementGravatar Boris Muratov2021-08-16-214/+390
|\ \ | | | | | | Additional code jam management utilities
| * \ Merge branch 'main' into mbaruh/jam-managementGravatar Boris Muratov2021-08-16-571/+1612
| |\ \ | |/ / |/| |
* | | Merge pull request #1745 from python-discord/fix-nomination-archive-checkGravatar Matteo Bertucci2021-08-15-8/+10
|\ \ \ | | | | | | | | fix: Nomination message now checks historic and new style nominations
| * | | fix: Nomination message now checks historic and new style nominationsGravatar Chris Lovering2021-08-12-8/+10
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Previously nomination messages had role pings in them, now they don't as we moved them into a thread. Due to this, we need to detect both in the interim of historic nominations existing. A 'proper' fix for this is to store the nomination message IDs when we post them against the nomination object in the site api. We are planing to work on this soon, this commit is a short term fix.
* | | Remove role pings when a helper vote is posted (#1744)Gravatar ChrisJL2021-08-11-2/+2
| | |
* | | Merge #1726 - catch various 404s from message operationsGravatar Mark2021-08-11-12/+37
|\ \ \
| * \ \ Merge branch 'main' into bug/message-404Gravatar Mark2021-08-11-13/+16
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1740 from python-discord/TizzySaurus-patch-1Gravatar Matteo Bertucci2021-08-11-1/+1
|\ \ \ \ | | | | | | | | | | Remove added punctuation from reminder
| * | | | Remove added punctuation from reminderGravatar TizzySaurus2021-08-11-1/+1
|/ / / /
* | | | Merge pull request #1738 from python-discord/dynamic-dormant-messageGravatar ChrisJL2021-08-11-5/+13
|\ \ \ \ | | | | | | | | | | Make DORMANT_MSG Dynamic
| * | | | Fetch dormant category rather than use channel.categoryGravatar TizzySaurus2021-08-11-1/+2
| | | | | | | | | | | | | | | channel.category doesn't get updated in cache so the category ends up still linking to "In Use", whereas we want the "Dormant".
| * | | | Change bot.get_channel to utils.channels.try_get_channelGravatar TizzySaurus2021-08-11-1/+1
| | | | |
| * | | | Make DORMANT_MSG a string instead of f-stringGravatar TizzySaurus2021-08-11-1/+1
| | | | |
| * | | | Update embed sent when channel moves to dormant categoryGravatar TizzySaurus2021-08-11-1/+8
| | | | |
| * | | | Update DORMANT_MSG to allow kwargs in str.format()Gravatar TizzySaurus2021-08-11-3/+3
| | | | |
| * | | | Update DORMANT_MSG to be compatible with str.format()Gravatar TizzySaurus2021-08-11-3/+3
|/ / / /
* | | | Merge pull request #1733 from ↵Gravatar Matteo Bertucci2021-08-10-5/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | python-discord/chore/vcokltfre/blocking-tag-newlines chore: remove some newlines in the blocking tag
| * | | | chore: remove some newlines in the blocking tagGravatar vcokltfre2021-08-10-5/+0
|/ / / /
* | | | Merge pull request #1732 from SuperstalkerX/patch-1Gravatar Xithrius2021-08-10-1/+1
|\ \ \ \ | | | | | | | | | | Fixed error message to match true value
| * | | | Fixed error message to match true valueGravatar stalkerx2021-08-09-1/+1
|/ / / / | | | | | | | | | | | | When the limit was raised, the error message was not edited to reflect the change. https://github.com/python-discord/bot/commit/6f45d6896adb3f05962733cec8e5db199def20bc
* | | | Merge pull request #1727 from onerandomusername/patch-1Gravatar Joe Banks2021-08-08-1/+1
|\ \ \ \ | | | | | | | | | | change dockerfile python version to latest 3.9.x
| * | | | change dockerfile python version to latest 3.9.xGravatar aru2021-08-08-1/+1
|/ / / /
| * | | Duck pond: abort if reaction's message or author can't be foundGravatar MarkKoz2021-08-07-1/+7
| | | | | | | | | | | | | | | | Fixes BOT-1J7
| * | | Code block: catch 404s when editing or deleting the messageGravatar MarkKoz2021-08-07-7/+10
| | | | | | | | | | | | | | | | Fixes BOT-J2