|  | Commit message (Collapse) | Author | Age | Lines | 
|---|
| ... |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | test cases listing, added icon to test cases in `notify_pardon` test | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `has_active_infraction` test. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | testing, removed old. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `post_user` and `send_private_embed` test. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `send_private_embed` test | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | content test), improved test cases. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `discord.Embed` in `notify_pardon` test. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `notify_pardon`, applied changes to test cases. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `Embed`. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | moved `send_private_embed` mock reset to beginning of subtest. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | removed these in test cases too in `notify_infraction` test. | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | hard-coded args, moved mocks resetting to beginning of subtest, added `ctx.send` check only is infraction nr and type in sent string. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `has_active_infraction` test. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `utils` instead and added `utils` before variables and function that was imported directly before. | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | errors | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `test_send_private_embed`. | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `ctx.send` raising errors, added check for return values and `send_private_embed` call. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `ctx.send` raising errors, added check for return values and `send_private_embed` call. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | `has_active_infraction` tests: `ctx.send` and `bot.api_client.get` calling. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | instead creating new one | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | (`self.bot.api_client.get`) | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | cases testing to test cases listing, added side_effect resetting. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | created __init__.py for moderation tests | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | Fixed errors. | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | created new params for tests class | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | |  | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | Make their names more in line with `has_any_role` for consistency. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | Use `has_any_role` to reduce redundancy.
Because discord.py always makes a check's predicate a coroutine, the
checks now have to be awaited. | 
| | | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | In some places, it's more appropriate than using with_role_check since
it will raise CheckFailures. This applies to `cog_check`s and other
things which effectively act as command checks. | 
| | | | | | | | | | | | | | | | |  | 
| | |_|_|_|_|_|_|_|_|_|/ / / /  
|/| | | | | | | | | | | | |   
| | | | | | | | | | | | | |   
| | | | | | | | | | | | | | | `with_role` is obsolete because discord.py provides `has_any_role`. | 
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \  
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | | Move bolding markdown outside of text link in help channel messages. | 
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \  
| |/ / / / / / / / / / / / / /  
|/| | | | | | | | | | | | | | |  | 
| | |_|_|_|_|_|_|_|_|_|_|/ / /  
|/| | | | | | | | | | | | |   
| | | | | | | | | | | | | |   
| | | | | | | | | | | | | |   
| | | | | | | | | | | | | |   
| | | | | | | | | | | | | |   
| | | | | | | | | | | | | |   
| | | | | | | | | | | | | | | As there are a few bugs in the implementation, I've temporarily disabled
the at-everyone ping filter in the AntiSpam cog. We can disable it after
we've fixed the bugs.
Signed-off-by: Sebastiaan Zeeff <[email protected]> | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | Our AntiSpam cog suffers from a race condition that causes it to try and
infract the same user multiple times. As that happens frequently with
the burst_shared filter, it means that our bot joins in and starts
spamming the channel with error messages.
Another issue is that burst_shared may cause our bot to send a lot of
DMs to a lot of different members. This caused our bot to get a DM ban
from Discord after a recent `everyone` ping incident.
I've decided to disable the `burst_shared` filter by commenting out the
relevant lines but leave the code in place otherwise. This means we
still have the implementation handy in case we want to re-enable it on
short notice.
Signed-off-by: Sebastiaan Zeeff <[email protected]> | 
| |/ / / / / / / / / / / / /  
| | | | | | | | | | | | |   
| | | | | | | | | | | | |   
| | | | | | | | | | | | |   
| | | | | | | | | | | | | | On some devices the markdown gets rendered improperly,
leaving the asterisks in the message without bolding. | 
| |\ \ \ \ \ \ \ \ \ \ \ \ \  
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | python-discord/bug/filters/1130/ignore-codeblock-emoji
AntiSpam: ignore custom emojis in code blocks | 
| | | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | 
| | | | | | | | | | | | | | | In code blocks, custom emojis render as text rather than as images.
Therefore, they probably aren't being spammed and should be ignored.
Fix #1130 |