aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| * | | | | | | | | | | | Merge pull request #1100 from python-discord/bug/filters/1099/dupe-offensive-msgGravatar Senjan212020-08-19-3/+11
| |\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Filtering: ignore errors for duplicate offensive messages
| | * | | | | | | | | | | | Filtering: ignore errors for duplicate offensive messagesGravatar MarkKoz2020-08-12-3/+11
| | | |/ / / / / / / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The error happens when a filter is triggered by a message edit. Fixes #1099 Fixes BOT-6B
| * | | | | | | | | | | | Merge pull request #1107 from python-discord/bug/filters/1005/antispam-ctx-patchGravatar Senjan212020-08-19-4/+9
| |\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't patch ctx.message.author in antispam
| | * | | | | | | | | | | | Truncate mod log contentGravatar MarkKoz2020-08-15-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Discord has a limit of 2000 characters for messages.
| | * | | | | | | | | | | | Don't patch ctx.message.author in antispamGravatar MarkKoz2020-08-15-4/+5
| | | |/ / / / / / / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The modification propagated across all code that is using the same `Message` object, including all other `on_message` listeners. This caused weird bugs e.g. the filtering cog thinking the bot authored a message that triggered a filter. Patching only `ctx.author` means the implementation is more fragile. Infraction code must ensure it only retrieves the author via `ctx.author` and not through `ctx.message`. Fixes #1005 Fixes BOT-7D
| * | | | | | | | | | | | Merge pull request #1108 from ↵Gravatar Mark2020-08-18-0/+3
| |\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/frontend/1030/reddit-title-escape Escape Markdown in reddit post titles
| | * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into bug/frontend/1030/reddit-title-escapeGravatar Mark2020-08-18-9/+0
| | |\ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / | |/| | | | | | | | | | | |
| * | | | | | | | | | | | | Remove the !ask tagGravatar Leon Sandøy2020-08-16-9/+0
| | |/ / / / / / / / / / / | |/| | | | | | | | | | |
| | * | | | | | | | | | | Escape Markdown in reddit post titlesGravatar MarkKoz2020-08-15-0/+3
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use a Unicode look-alike character to replace square brackets, since they'd otherwise interfere with the Markdown. Fixes #1030
| * | | | | | | | | | | Merge pull request #1102 from AtieP/patch-1Gravatar kwzrd2020-08-13-1/+1
| |\ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / | |/| | | | | | | | | | Fix typo on the traceback tag
| | * | | | | | | | | | Fix typo on the traceback tagGravatar AtieP2020-08-13-1/+1
| |/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | See issue #1101
| * | | | | | | | | | Merge pull request #1097 from Numerlor/remove-api-endpointsGravatar Dennis Pham2020-08-11-33/+4
| |\ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|_|/ / | |/| | | | | | | | | Remove api endpoint config values.
| | * | | | | | | | | Merge branch 'master' into remove-api-endpointsGravatar Dennis Pham2020-08-11-0/+3
| | |\ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / | |/| | | | | | | | |
| * | | | | | | | | | Create kindling-projects tagGravatar Dennis Pham2020-08-09-0/+3
| | | | | | | | | | |
| | * | | | | | | | | Reorder site URL constants.Gravatar Numerlor2020-08-10-2/+4
| | | | | | | | | | |
| | * | | | | | | | | Remove unused api endpoint config constants.Gravatar Numerlor2020-08-10-31/+0
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The constants aren't used anywhere in the bot, and are incompatible with the APIClient.
| * | | | | | | | | Merge pull request #1084 from ↵Gravatar Mark2020-08-09-1/+4
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/filters/1027/ignore-webhook-names Ignore webhooks for nickname filter
| | * \ \ \ \ \ \ \ \ Merge branch 'master' into bug/filters/1027/ignore-webhook-namesGravatar Dennis Pham2020-08-09-136/+155
| | |\ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / | |/| | | | | | | | |
| * | | | | | | | | | Merge pull request #1085 from python-discord/bug/util/1079/long-poll-titlesGravatar Dennis Pham2020-08-09-0/+2
| |\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Utils: show error message for long poll titles
| | * \ \ \ \ \ \ \ \ \ Merge branch 'master' into bug/util/1079/long-poll-titlesGravatar Dennis Pham2020-08-09-136/+153
| | |\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / | |/| | | | | | | | | |
| * | | | | | | | | | | Remove unnecessary edits during paginationGravatar Joe Banks2020-08-08-10/+0
| | | | | | | | | | | |
| * | | | | | | | | | | Merge pull request #1086 from python-discord/bug/help/1082/unpin-id-noneGravatar Dennis Pham2020-08-07-27/+64
| |\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Help channel message pin fixes
| | * \ \ \ \ \ \ \ \ \ \ HelpChannels: fix conflict in move_to_dormantGravatar MarkKoz2020-08-07-100/+90
| | |\ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / | |/| | | | | | | | | | |
| * | | | | | | | | | | | Merge pull request #1092 from python-discord/bug/help/1074/duplicate-claimGravatar scragly2020-08-07-3/+1
| |\ \ \ \ \ \ \ \ \ \ \ \
| | * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into bug/help/1074/duplicate-claimGravatar scragly2020-08-07-98/+90
| | |\ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / | |/| | | | | | | | | | | |
| | * | | | | | | | | | | | HelpChannels: clear claimant cache when channel goes dormantGravatar MarkKoz2020-08-05-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The claimed channel check in `on_message` relies on the cache being cleared when a channel goes dormant. If it's not cleared, it will think the channel is still in use.
| | | * | | | | | | | | | | HelpChannels: simplify control flow in is_emptyGravatar MarkKoz2020-08-07-4/+2
| | | | | | | | | | | | | |
| | | * | | | | | | | | | | HelpChannels: add logging to is_emptyGravatar MarkKoz2020-08-04-0/+3
| | | | | | | | | | | | | |
| | | * | | | | | | | | | | HelpChannels: more accurate empty checkGravatar MarkKoz2020-08-04-5/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The bot's pin message was being picked up as the last message, so the system was not considering the channel empty.
| | | * | | | | | | | | | | HelpChannels: create separate function to pin a messageGravatar MarkKoz2020-08-04-9/+7
| | | | | | | | | | | | | |
| | | * | | | | | | | | | | HelpChannels: create a generic function to handle pin errorsGravatar MarkKoz2020-08-04-11/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This can be used for both pinning and unpinning messages. The error handling code was largely similar between them.
| | | * | | | | | | | | | | HelpChannels: don't unpin message if ID is NoneGravatar MarkKoz2020-08-04-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #1082 Fixes BOT-7G
| | | * | | | | | | | | | | HelpChannels: add more detail to unpin log messagesGravatar MarkKoz2020-08-04-3/+7
| | | | | | | | | | | | | |
| | | * | | | | | | | | | | HelpChannels: move unpinning to separate functionGravatar MarkKoz2020-08-04-12/+16
| | | | | | | | | | | | | |
| | | | * | | | | | | | | | Utils: show error message for long poll titlesGravatar MarkKoz2020-08-04-0/+2
| | | |/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Embeds have a maximum length of 256 for titles. Fixes #1079 Fixes BOT-7Q
| | | | * / / / / / / / / Filtering: ignore webhooks for nickname filterGravatar MarkKoz2020-08-04-1/+4
| | | |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #1027
| | | | * | | | | | | | Verification: move constants to configGravatar kwzrd2020-09-10-25/+43
| | | | | | | | | | | |
| | | | * | | | | | | | Verification: set 'tasks_running' to 0 on suspicious 403sGravatar kwzrd2020-08-29-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevent the tasks from starting again if the bot restarts.
| | | | * | | | | | | | Verification: denote `_maybe_start_tasks` as privateGravatar kwzrd2020-08-29-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Consistency with the new `_stop_tasks` method.
| | | | * | | | | | | | Verification: stop tasks on suspicious 403Gravatar kwzrd2020-08-29-1/+4
| | | | | | | | | | | |
| | | | * | | | | | | | Verification: add helper for stopping tasksGravatar kwzrd2020-08-29-5/+17
| | | | | | | | | | | |
| | | | * | | | | | | | Verification: improve allowed mentions handlingGravatar kwzrd2020-08-27-7/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I really didn't like the constants, but the construction of allowed mentions instances is syntactically noisy, so I prefer to keep it out of the important logic. Abstracting it behind a function seems to be the best approach yet.
| | | | * | | | | | | | Verification: document StopExecution handlingGravatar kwzrd2020-08-27-0/+3
| | | | | | | | | | | |
| | | | * | | | | | | | Verification: stop kicking members on suspicious 403Gravatar kwzrd2020-08-26-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A Discord error code 50_0007 signifies that the DM dispatch failed because the target user does not accept DMs from the bot. Such errors are ignored as before. Any other 403s will however cause the bot to stop making requests. This is in case the bot gets caught by an anti-spam filter and should immediately stop.
| | | | * | | | | | | | Verification: add helper for alerting adminsGravatar kwzrd2020-08-26-0/+18
| | | | | | | | | | | |
| | | | * | | | | | | | Verification: retain ping in edited confirmation msgGravatar kwzrd2020-08-26-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevent a ghost ping from occurring upon reaction. Co-authored-by: Senjan21 <[email protected]>
| | | | * | | | | | | | Verification: remove explicit everyones from allowed mentionsGravatar kwzrd2020-08-26-6/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the kwarg isn't passed, it uses the value that was given to the bot on init (False), despite the kwarg defaulting to True. Thanks to Mark and Senjan for helping me understand this. Co-authored-by: MarkKoz <[email protected]> Co-authored-by: Senjan21 <[email protected]>
| | | | * | | | | | | | Verification: separate guild invite by empty lineGravatar kwzrd2020-08-26-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Joe Banks <[email protected]>
| | | | * | | | | | | | Verification: send guild invite with kick messageGravatar kwzrd2020-08-26-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Makes it easy for users to re-join. Co-authored-by: Joe Banks <[email protected]>
| | | | * | | | | | | | Verification: add guild invite to configGravatar kwzrd2020-08-26-0/+2
| | | | | | | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-30 13:53:48 +0000