aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | | | | | | | | | | | | | | | | | | | Remove null chars before posting deleted messagesGravatar MarkKoz2020-09-30-1/+1
| |/ / / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Our API doesn't allow null characters in the content field. It may be present because of a self bot that is able to send such character. Fixes #1182 Fixes BOT-8E
| | * / / / / / / / / / / / / / / / / / / Catch 404 in wait_for_deletion when reactingGravatar MarkKoz2020-09-30-1/+5
| |/ / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The message may be deleted before the bot gets a chance to react. Fixes #1181
| * | | | | | | | | | | | | | | | | | | PR #1191 Verification: broaden 50_007 error handlingGravatar kwzrd2020-09-29-11/+30
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| | * | | | | | | | | | | | | | | | | | | Incidents: reduce timeout log to info levelGravatar kwzrd2020-09-27-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This shouldn't be a warning, as we cannot do anything about it. Fixes BOT-8X
| | * | | | | | | | | | | | | | | | | | | Verification: apply 'safe_dm' to all DM dispatchesGravatar kwzrd2020-09-27-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now, when we send a DM and it fails: * Ignore if due to disabled DMs * Log exception otherwise
| | * | | | | | | | | | | | | | | | | | | Verification: move disabled DM handling into helperGravatar kwzrd2020-09-27-5/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Note that we were previously only catching 403. As the docstring explains, we will now catch any Discord exception and only look at the the code, rather than the status.
| * | | | | | | | | | | | | | | | | | | | Merge remote-tracking branch 'origin/master' into masterGravatar Matteo Bertucci2020-09-27-585/+101
| |\| | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | Relock Pipfile to update async-redis.Gravatar Leon Sandøy2020-09-27-62/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also bumps minor versions of several other packages. I've spun up the bot and played around with it, and run all unit tests. Everything still seems to be in order.
| | * | | | | | | | | | | | | | | | | | | Add the video to the welcome DM.Gravatar Leon Sandøy2020-09-27-4/+7
| | | |_|_|_|/ / / / / / / / / / / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This rewords the welcome DM, and adds the new Welcome To Python Discord video to it.
| | * | | | | | | | | | | | | | | | | | PR #1151 Sync: remove confirmationGravatar kwzrd2020-09-26-519/+24
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / | | |/| | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | Sync: update sync() docstringGravatar MarkKoz2020-09-26-3/+1
| | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | Sync: remove _asdict commentGravatar MarkKoz2020-09-23-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The comment doesn't contribute anything.
| | | * | | | | | | | | | | | | | | | | Merge branch 'master' into feat/backend/1075/remove-syncer-confirmGravatar MarkKoz2020-09-22-2227/+2207
| | | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | |_|_|_|_|_|_|_|_|/ / / / / / / | | | | |/| | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | Sync: refactor conditional for sending message Gravatar Mark2020-09-22-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ternary is a bit confusing. Co-authored-by: Numerlor <[email protected]>
| | | * | | | | | | | | | | | | | | | | Fix test for sync message being editedGravatar MarkKoz2020-09-07-1/+3
| | | | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | | | Sync: remove confirmationGravatar MarkKoz2020-09-07-518/+20
| | | | |_|_|/ / / / / / / / / / / / / | | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The confirmation was intended to be a safe guard against cache issues that would cause a huge number of roles/users to deleted after syncing. With `wait_until_guild_available`, such cache issue shouldn't arise. Therefore, this feature is obsolete. Resolve #1075
| * | | | | | | | | | | | | | | | | | Allow !role for any staff roleGravatar Matteo Bertucci2020-09-27-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #1173
| * | | | | | | | | | | | | | | | | | Allow !eval in #code-help-voiceGravatar Matteo Bertucci2020-09-27-1/+3
| |/ / / / / / / / / / / / / / / / /
| * | | | | | | | | | | | | | | | | Merge pull request #1161 from bast0006/feature-bast-user-token-notifyGravatar Mark2020-09-25-39/+188
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add userid decoding and user token detection to the token removal moderation feature
| | * | | | | | | | | | | | | | | | | Remove redundant is_valid_userid functionGravatar Bast2020-09-25-33/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | extract_user_id(id) is not None does the same job and is not worth the extra function
| | * | | | | | | | | | | | | | | | | Change the mod alert message component for the user token detectionGravatar Bast2020-09-24-46/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clean up mock usage, docstrings, unnecessarily split-lined function calls
| | * | | | | | | | | | | | | | | | | Add BOT vs USER token detection, properly handling bot tokens for bots in ↵Gravatar Bast2020-09-24-13/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the current server Also adjust the naming and purposes of the format messages to KNOWN and UNKNOWN token messages.
| | * | | | | | | | | | | | | | | | | Implement review-suggested changesGravatar Bast2020-09-24-67/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | userid -> user ID maybevalid -> maybe_valid remove collections import and added a new function that handles the "format user ID log message" and should_ping_everyone feature
| | * | | | | | | | | | | | | | | | | Merge branch 'master' into feature-bast-user-token-notifyGravatar Bast2020-09-21-270/+686
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resynchronize changes with the new cog folder arrangement
| | * | | | | | | | | | | | | | | | | | Make token_remover check basic HMAC validity (not low entropy)Gravatar Bast2020-09-19-6/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handles cases like xxx.xxxxx.xxxxxxxx where a user has intentionally censored part of a token, and will not consider them "valid"
| | * | | | | | | | | | | | | | | | | | Add feature to token_remover: log detected user ID, and ping if it's a user ↵Gravatar Bast2020-09-19-19/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | in the server Updated tests This comes with a change that a user ID must actually be able to be decoded into an integer to be considered a valid token
| | | | | | | | * | | | | | | | | | | | remove redundant type hints and improve existing function annotationsGravatar RohanJnr2020-10-01-8/+8
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | * | | | | | | | | | | | modify tests to use paginated response.Gravatar RohanJnr2020-09-27-6/+37
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | * | | | | | | | | | | | Merge branch 'master' into smart_syncing_usersGravatar Rohan Reddy Alleti2020-09-25-1970/+1178
| | | | | | | | |\ \ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|_|_|/ / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | | Merge pull request #1171 from python-discord/kwzrd/log-levelsGravatar Mark2020-09-23-2/+2
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adjust verification & incidents log levels
| | * | | | | | | | | | | | | | | | | | | Incidents: bump archive log to INFO levelGravatar kwzrd2020-09-23-1/+1
| | | | | | | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | | | | | | Verification: reduce request dispatch log levelGravatar kwzrd2020-09-23-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid information duplication in production logs.
| * | | | | | | | | | | | | | | | | | | | Fix use of expanded infraction response for usernameGravatar MarkKoz2020-09-23-1/+1
| |/ / / / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes BOT-9A
| | | | | | | | * | | | | | | | | | | | Update users on bot start via HTTP PATCH method and send only user ID and ↵Gravatar RohanJnr2020-09-25-3/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the modified user data.
| | | | | | | | * | | | | | | | | | | | fix type and add variable type hintingGravatar RohanJnr2020-09-22-2/+1
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | * | | | | | | | | | | | Refactor code to make use of bulk create and update API endpoints.Gravatar RohanJnr2020-09-22-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | instead of creating and updating a single user at a time, a list of dicts will be sent for bulk update and creation.
| | | | | | | | * | | | | | | | | | | | Merge remote-tracking branch 'upstream/master' into smart_syncing_usersGravatar RohanJnr2020-09-21-257/+301
| | | | | | | | |\ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | * | | | | | | | | | | | | refactor code to GET users from site endpoint `bot/users` with paginationGravatar RohanJnr2020-09-21-1/+29
| | | |_|_|_|_|/ / / / / / / / / / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added method to recursively GET users if paginated and another method to parse URL and return endpoint and query parameters.
| | | | | | | | | | | | | | * | | | | | Silence: fix scheduled tasks not being cancelled on unloadGravatar MarkKoz2020-10-09-1/+1
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | | | | Silence tests: make _async_init attribute tests more robustGravatar MarkKoz2020-10-08-7/+11
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | | | | Silence: remove _mod_log_channel attributeGravatar MarkKoz2020-10-08-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's only used as an argument to `SilenceNotifier`, so it doesn't need to be an instance attribute.
| | | | | | | | | | | | | | * | | | | | Silence: add to notifier when indefinite rather than temporaryGravatar MarkKoz2020-10-08-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Accidentally swapped the logic in a previous commit during a refactor.
| | | | | | | | | | | | | | * | | | | | Silence tests: fix unawaited coro warningsGravatar MarkKoz2020-10-07-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because the Scheduler is mocked, it doesn't actually do anything with the coroutines passed to the schedule() functions, hence the warnings.
| | | | | | | | | | | | | | * | | | | | Silence: rename _init_cog to _async_initGravatar MarkKoz2020-10-06-15/+15
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | | | | Silence: refactor _silenceGravatar MarkKoz2020-10-06-33/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Rename to `_silence_overwrites` * Reduce responsibilities to only setting permission overwrites * Log in `silence` instead * Add to notifier in `silence` instead
| | | | | | | | | | | | | | * | | | | | Silence: move unsilence scheduling to a separate functionGravatar MarkKoz2020-10-06-8/+13
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | | | | Add license & copyright for autospec's _decoration_helperGravatar MarkKoz2020-09-25-0/+52
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | | | | Silence: use f-string for messageGravatar Mark2020-09-22-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Numerlor <[email protected]>
| | | | | | | | | | | | | | * | | | | | Silence: cancel init task when cog unloadsGravatar MarkKoz2020-08-18-4/+12
| | | | | | | | | | | | | | | | | | | |
| | | | | | | | | | | | | | * | | | | | Silence: rename cachesGravatar MarkKoz2020-08-18-29/+29
| | | | | | | | | | | | | | | | | | | |