aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | | | | | | | | | | * | | | Replace InfractionSearchQuery with a generic Snowflake converterGravatar MarkKoz2020-08-11-13/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's unnecessarily precise to do an fetch user API call in order to distinguish between a user and a reason. Furthermore, a User object isn't actually required for infraction searches - only an ID is.
| | | | | | | | | | | | * | | | Resolve conflicts in filtering.pyGravatar MarkKoz2020-08-10-562/+894
| | | | | | | | | | | | |\ \ \ \
| | | | | | | | | | | | * | | | | Zero-fill discriminators in infraction searchesGravatar MarkKoz2020-08-10-1/+1
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Use format_user in token remover testGravatar MarkKoz2020-08-10-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The point of format_user is to have a consistent format across the code base. That should apply to tests too.
| | | | | | | | | | | | * | | | | Concatenate string in one lineGravatar MarkKoz2020-08-10-2/+1
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Fix conflict with disabling filter ping for DMsGravatar MarkKoz2020-08-02-68/+307
| | | | | | | | | | | | |\ \ \ \ \ | | |_|_|_|_|_|_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Fix test for token remover log messageGravatar MarkKoz2020-07-21-2/+1
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Use user mentions in infraction search resultsGravatar MarkKoz2020-07-21-14/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using our expanded API is more efficient than making a request to the Discord API for potentially every user in the search results. The data may not be up to date, but that's an acceptable compromise.
| | | | | | | | | | | | * | | | | ModLog: use user mentionsGravatar MarkKoz2020-07-21-22/+12
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Filtering: use user mentions in mod logsGravatar MarkKoz2020-07-21-6/+5
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Filtering: refactor _add_stats to return a NamedTupleGravatar MarkKoz2020-07-21-25/+27
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Filtering: refactor sending of mod logGravatar MarkKoz2020-07-21-59/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A lot of redundant code existed between the message and eval filters.
| | | | | | | | | | | | * | | | | Use user mentions in mod logsGravatar MarkKoz2020-07-21-10/+14
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Moderation: use user mentions in mod logsGravatar MarkKoz2020-07-21-18/+14
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Moderation: remove multiple active infractions checkGravatar MarkKoz2020-07-21-36/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The API was change a long time ago to not allow such a situation.
| | | | | | | | | | | | * | | | | Superstarify: escape Markdown in nicknamesGravatar MarkKoz2020-07-21-1/+4
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | * | | | | Superstarify: use user mentions in mod logsGravatar MarkKoz2020-07-21-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `format_user` isn't used in the apply mod log cause it already shows both the old and new nicknames elsewhere.
| | | | | | | | | | | | * | | | | Add util function to format user namesGravatar MarkKoz2020-07-21-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will be used a lot when sending mod logs and will help with reducing redundancy and maintaining consistency.
| | | | | | | | | | | | * | | | | Clean up importsGravatar MarkKoz2020-07-21-14/+13
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | Constants: remove staff_channelsGravatar MarkKoz2020-09-02-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | No longer being used anywhere.
| | | | | | | | | | | | | | * | | Fix user command testsGravatar MarkKoz2020-08-26-13/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mocking the bot commands channel constant no longer worked after switching to `in_whitelist_check`.
| | | | | | | | | | | | | | * | | Fix infraction counts being shown in wrong channelsGravatar MarkKoz2020-08-26-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They should only be shown in mod channels, not all staff channels. Fix #1127
| | | | | | | | | | | | | | * | | Info: show verbose infractions if user cmd invoked in modmailGravatar MarkKoz2020-08-25-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix #1125
| | | | | | | | | | | | | | * | | Info: simplify channel redirection for the user commandGravatar MarkKoz2020-08-25-9/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `in_whitelist_check` is a convenient utility that does the same thing as the previous code.
| | | | | | | | | | | | | | | | * Lock: make LockGuard.locked a propertyGravatar MarkKoz2020-10-03-1/+2
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | * Lock: rename variable to avoid shadowingGravatar MarkKoz2020-10-03-3/+3
| | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | | | * Reminders: fix conflict with the _can_modify checkGravatar MarkKoz2020-09-09-1319/+1347
| | | | | | | | | | | | | | | | |\ | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | Merge pull request #1144 from Numerlor/text-link-fixGravatar Shirayuki Nekomata2020-09-05-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move bolding markdown outside of text link in help channel messages.
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into text-link-fixGravatar Shirayuki Nekomata2020-09-05-8/+17
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | Disable everyone_ping filter in AntiSpam cogGravatar Sebastiaan Zeeff2020-09-05-4/+8
| |_|_|_|_|_|_|_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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]>
* | | | | | | | | | | | | | | | | Disable burst_shared filter of the AntiSpam cogGravatar Sebastiaan Zeeff2020-09-04-4/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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]>
| * | | | | | | | | | | | | | | | Move bolding markdown outside of text link.Gravatar Numerlor2020-09-04-2/+2
|/ / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On some devices the markdown gets rendered improperly, leaving the asterisks in the message without bolding.
* | | | | | | | | | | | | | | | Merge pull request #1132 from ↵Gravatar Mark2020-09-03-1/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/filters/1130/ignore-codeblock-emoji AntiSpam: ignore custom emojis in code blocks
| * | | | | | | | | | | | | | | | AntiSpam: ignore custom emojis in code blocksGravatar MarkKoz2020-08-27-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | | | | | | | | | | | | Merge pull request #1135 from ↵Gravatar Daniel Brown2020-09-03-1/+48
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | MrAwesomeRocks/feat/filter/1131/everyone-ping-responder Auto-respond to everyone pings.
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into feat/filter/1131/everyone-ping-responderGravatar Daniel Brown2020-09-03-513/+48
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | Merge pull request #1134 from python-discord/remove-wolframGravatar Leon Sandøy2020-09-02-475/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Completely gutted the wolfram command.
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into remove-wolframGravatar Leon Sandøy2020-09-02-34/+38
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | | Merge pull request #1141 from wookie184/make-antimalware-ignore-bot-and-webhookGravatar Dennis Pham2020-09-01-0/+26
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added checks to ignore webhook and bot messages for antimalware
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into make-antimalware-ignore-bot-and-webhookGravatar Dennis Pham2020-09-01-28/+6
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | | | Merge pull request #1123 from AtieP/masterGravatar Shirayuki Nekomata2020-09-01-28/+6
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use wait_for_deletion from bot/utils/messages.py rather than help_cleanup
| * \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into masterGravatar Shirayuki Nekomata2020-09-01-161/+259
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | Remove unused variables and importsGravatar AtieP2020-08-29-2/+1
| | | | | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | | Use wait_for_deletion from bot/utils/messages.py rather than help_cleanupGravatar AtieP2020-08-22-26/+5
| | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | Fixed old tests and added 2 new onesGravatar wookie1842020-09-01-0/+22
| | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | Added checks to ignore webhook and bot messagesGravatar wookie1842020-09-01-0/+4
| |/ / / / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | | | Allow moderators to use defconGravatar Dennis Pham2020-08-30-6/+6
| | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | Removed loading of the Wolfram cog.Gravatar Xithrius2020-08-31-1/+0
| | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | Removed ImagePaginator testing.Gravatar Xithrius2020-08-31-15/+0
| | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | Removed image pagination utility.Gravatar Xithrius2020-08-31-164/+0
| | | | | | | | | | | | | | | | | | | |