aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* (Webhook Detection): Renamed `url` variable to `redacted_url` to avoid ↵Gravatar ks1232020-03-28-2/+2
| | | | confusion in `delete_and_respond` function.
* (Webhook Detection): Fixed docstrings.Gravatar ks1232020-03-28-2/+2
|
* (Webhook Detection): Removed unnecessary URL hiding in `delete_and_respond`.Gravatar ks1232020-03-28-5/+0
|
* (Webhook Detection): Added grouping to RegEx compilation, removed ↵Gravatar ks1232020-03-28-13/+4
| | | | unnecessary function `scan_message`, moved this content to `on_message` event.
* (Webhook Detection): Call `on_message` instead repeating code.Gravatar ks1232020-03-28-3/+1
|
* (Webhook Detection): Fixed order of cog loading.Gravatar ks1232020-03-28-1/+1
|
* (Webhook Detection): Added `on_message_edit` listener for Discord Webhooks ↵Gravatar ks1232020-03-28-0/+7
| | | | detecting.
* (Webhook Detection): Alert message formatting changes, added `on_message` ↵Gravatar ks1232020-03-28-2/+9
| | | | listener.
* (Webhook Detection): Added URL returning to `scan_message` helper function.Gravatar ks1232020-03-28-3/+4
|
* (Webhook Detection): Created `delete_and_respond` helper function to handle ↵Gravatar ks1232020-03-28-2/+41
| | | | Webhook URLs.
* (Webhook Detection): Added `ModLog` fetching property.Gravatar ks1232020-03-28-0/+6
|
* (Webhook Detection): Added cog loading to __main__.py, created ↵Gravatar ks1232020-03-28-0/+10
| | | | `scan_message` helper function to detect Webhook URL.
* (Webhook Detection): Added webhook match regex.Gravatar ks1232020-03-28-0/+4
|
* (Webhook Detection): Created cog.Gravatar ks1232020-03-28-0/+15
|
* Merge pull request #842 from python-discord/bug/util/831/snekbox-reeval-parsingGravatar S. Co12020-03-26-8/+59
|\ | | | | Fix snekbox's message parsing to extract code for re-evaluation
| * Merge branch 'master' into bug/util/831/snekbox-reeval-parsingGravatar S. Co12020-03-26-2/+2
| |\ | |/ |/|
* | Change Alias warnings to infoGravatar Leon Sandøy2020-03-26-2/+2
| | | | | | | | | | | | | | | | | | Stuff like "{name} tried to run {command}" and "{command} could not be found" was set as a warning, and so Sentry issues were being created for these. Our rule of thumb is that only actionable things should be warnings. Changed these to Info logs.
| * Snekbox tests: use `get_code` in `test_continue_eval_does_continue`Gravatar MarkKoz2020-03-25-3/+6
| |
| * Snekbox tests: test `get_code`Gravatar MarkKoz2020-03-25-4/+32
| | | | | | | | | | Should return 1st arg (or None) if eval cmd in message, otherwise return full content.
| * Snekbox: fix re-eval when '!eval' is removed from edited messageGravatar MarkKoz2020-03-25-1/+21
|/ | | | | | | The previous parsing method was naïve in assuming there would always be something preceding the code (e.g. the '!eval' command invocation) delimited by a space. Now it will only split if it's sure the eval command was used in the edited message.
* Merge pull request #840 from ks129/zen-tag-removeGravatar Mark2020-03-25-20/+0
|\ | | | | Removed `zen` tag due `!zen` command exist.
| * Merge branch 'master' into zen-tag-removeGravatar Mark2020-03-25-254/+259
| |\ | |/ |/|
* | Merge pull request #841 from python-discord/update-depsGravatar scragly2020-03-25-179/+98
|\ \ | | | | | | Remove unused safety & dodgy dev dependencies
| * | Remove unused safety & dodgy dev dependenciesGravatar S. Co12020-03-24-179/+98
|/ / | | | | Relock
* | Merge pull request #837 from python-discord/update-contribGravatar kwzrd2020-03-23-22/+37
|\ \ | | | | | | Update contributor doc
| * \ Merge branch 'master' into update-contribGravatar kwzrd2020-03-23-1/+1
| |\ \ | |/ / |/| |
* | | Fix regression in verification cogGravatar S. Co12020-03-21-1/+1
| | | | | | | | | | | | A stray `bot` was removed from the `on_message` listener, causing it to raise an exception rather than generate a `Context` object from incoming verification channel messages.
| * | Update contributor docGravatar S. Co12020-03-20-22/+37
|/ /
* | Merge pull request #821 from python-discord/hemlock-perma-ban-watch-removalGravatar S. Co12020-03-20-44/+94
|\ \ | | | | | | Automatically Remove Users from BigBrother Watch List on Perma Ban
| * | Changed a logging levelGravatar Daniel Brown2020-03-20-1/+1
| | | | | | | | | - Changed the log for when the big brother cog doesn't load in the `apply_ban()` method doesn't properly load from a trace to an error.
| * | Merge branch 'master' into hemlock-perma-ban-watch-removalGravatar Daniel Brown2020-03-20-117/+1052
| |\ \ | |/ / |/| |
* | | Merge pull request #530 from python-discord/spoiler-checkGravatar Leon Sandøy2020-03-17-10/+31
|\ \ \ | | | | | | | | Strip spoiler tags for watchlist triggers
| * \ \ Merge branch 'master' into spoiler-checkGravatar Jeremiah Boby2020-03-17-107/+1021
| |\ \ \ | |/ / / |/| | |
| * | | Escape markdown in voice updatesGravatar Jeremiah Boby2020-03-11-7/+14
| | | |
| * | | Escape markdown in member updatesGravatar Jeremiah Boby2020-03-11-1/+3
| | | |
| * | | Escape markdown in watchlist triggersGravatar Jeremiah Boby2020-03-11-2/+3
| | | |
| * | | Merge branch 'master' into spoiler-checkGravatar Jeremiah Boby2020-03-11-965/+903
| |\ \ \
| * \ \ \ Merge branch 'master' into spoiler-checkGravatar Leon Sandøy2020-02-29-297/+857
| |\ \ \ \
| * \ \ \ \ Merge branch 'master' into spoiler-checkGravatar Jeremiah Boby2020-02-27-2042/+5374
| |\ \ \ \ \
| * \ \ \ \ \ Merge branch 'master' into spoiler-checkGravatar Jeremiah Boby2019-11-28-2694/+5095
| |\ \ \ \ \ \
| * | | | | | | Expand spoilers to match multiple interpretationsGravatar Jeremiah Boby2019-11-28-1/+11
| | | | | | | |
| * | | | | | | Update spoiler regex to support multi-line spoilersGravatar Jeremiah Boby2019-10-21-1/+1
| | | | | | | |
| * | | | | | | Check partially hidden words against the wordlistGravatar Jeremiah Boby2019-10-13-1/+2
| | | | | | | |
| | | | | | * | Updated Docstrings, parameters, and log messagesGravatar Daniel Brown2020-03-20-13/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Docstrings for `apply_ban()` have been edited to mention that the method also removes a banned user from the watch list. - Parameter `banned` in `apply_unwatch()` was changed to `send_message` in order to be more general. Boolean logic was swapped to coincide with that change. - `apply_unwatch()`'s sent message moved to the bottom of the method for clarity. Added `return`s to the method to exit early if no message needs to be sent. Signed-off-by: Daniel Brown <[email protected]>
| | | | | | * | Updated doc strings to be more descriptiveGravatar Daniel Brown2020-03-20-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| | | | | | * | Nesting reduced, logging cleaned up and made clearerGravatar Daniel Brown2020-03-16-12/+15
| | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| | | | | | * | Update bot/cogs/moderation/infractions.pyGravatar Daniel Brown2020-03-14-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Mark <[email protected]>
| | | | | | * | Verification Cog Kaizen ChangesGravatar Daniel Brown2020-03-13-35/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Kaizen: - Cut down on the size of the import line by changing the imports from bot.constants to instead just importing the constants. This will help clarify where certain constants are coming from. - The periodic checkpoint message will no longer ping `@everyone` or `@Admins` when the bot detects that it is being ran in a debug environment. Message is now a simple confirmation that the periodic ping method successfully ran. Signed-off-by: Daniel Brown <[email protected]>
| | | | | | * | Corrected expiration check logic and cog loadingGravatar Daniel Brown2020-03-12-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bugs fixed: - Previously, the code would check to see if `'expires_at'` was in the kwargs, which after testing I came to find out that it is regardless of the duration of the ban. It has sense been changed to use a `.get()` in order to do a proper comparison. - Code previously attempted to load from the `"BigBrother"` cog which is the incorrect spelling. Changed it to `"Big Brother"` to correct this. Logging Added: - Additional trace logs added to both the `infractions.py` file as well as `bigbrother.py` to assist with future debugging or testing. Signed-off-by: Daniel Brown <[email protected]>
| | | | | | * | Merge remote-tracking branch 'origin/hemlock-perma-ban-watch-removal' into ↵Gravatar Daniel Brown2020-03-10-0/+0
| | | | | | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | hemlock-perma-ban-watch-removal