aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Fix a bad type annotation.Gravatar Leon Sandøy2020-07-29-2/+2
|
* Fix imports in converters.pyGravatar Leon Sandøy2020-07-29-5/+6
|
* Refactor filtering: use non-compiled expressions.Gravatar Leon Sandøy2020-07-29-10/+6
|
* Make a helper for validating guild invites.Gravatar Leon Sandøy2020-07-29-16/+18
|
* Make a helper for inserting filter lists.Gravatar Leon Sandøy2020-07-29-21/+16
|
* Kaizen - group private methods together.Gravatar Leon Sandøy2020-07-29-43/+43
|
* Use a defaultdict(list) for filter_list_cache.Gravatar Leon Sandøy2020-07-29-6/+7
|
* Expect status 400 for duplicates.Gravatar Leon Sandøy2020-07-29-2/+2
|
* Bump flake8 version to 3.8Gravatar Leon Sandøy2020-07-27-173/+191
| | | | This is necessary to support walrus operators.
* Rename AllowDenyList to FilterListsGravatar Leon Sandøy2020-07-27-45/+45
|
* More explicit dict indexing Gravatar Leon Sandøy2020-07-24-3/+3
| | | | | Addresses reviews from MarkKoz Co-authored-by: Mark <[email protected]>
* Add sanity to partner and verification check in filtering.py.Gravatar Leon Sandøy2020-07-24-2/+2
|
* No need for all() in cog_check for AllowDenyLists.Gravatar Leon Sandøy2020-07-24-4/+1
|
* Put valid_types_list inside the conditional.Gravatar Leon Sandøy2020-07-24-1/+1
|
* Catch ResponseCodeError in the ValidAllowDenyListType converter.Gravatar Leon Sandøy2020-07-24-7/+7
|
* Implement new guild invite filtering logic.Gravatar Leon Sandøy2020-07-19-7/+19
| | | | | | | | | | | We now filter guild invites the following way: - Whitelisted invites are always permitted. - Blacklisted invites are never permitted. - If the invite is not blacklisted, it is permitted only if it is a Verified or a Partnered server, otherwise not. This strategy was decided on during the June 7th staff meeting, see https://github.com/python-discord/organisation/issues/261
* Validation of guild invites for delete.Gravatar Leon Sandøy2020-07-19-2/+14
| | | | | We want to support deletion of both IDs and guild invites, so we need a bit of special handling for that.
* Validation of guild invites.Gravatar Leon Sandøy2020-07-19-17/+78
| | | | | | | | | | We will now validate and convert any standard discord server invite to a guild ID, and automatically add the name of the server as a comment. This will ensure that the list of whitelisted guild IDs will be readable and nice. This also makes minor changes to list output aesthetics.
* Support the new AllowDenyList field, 'comment'.Gravatar Leon Sandøy2020-07-19-10/+46
|
* Include the guild ID in mod-log embed.Gravatar Leon Sandøy2020-07-19-3/+4
| | | | | This gives easier access to the Guild ID in the place where you're most likely to want to use the whitelist command.
* Fix an absolutely terrible comment.Gravatar Leon Sandøy2020-07-19-1/+1
|
* Remove AntiMalWare constants, use cache data.Gravatar Leon Sandøy2020-07-18-55/+30
| | | | Also updates the tests for this cog.
* Remove Filtering constants, use cache data.Gravatar Leon Sandøy2020-07-18-135/+32
| | | | | Instead of fetching the guild invite IDs from config-default.yml, we will now be using the AllowDenyList cache to check these.
* Add more logging to AllowDenyLists cog.Gravatar Leon Sandøy2020-07-18-2/+11
|
* Add AllowDenyLists cog.Gravatar Leon Sandøy2020-07-18-0/+145
| | | | | | | This includes commands to add, remove and show the items in the whitelists and blacklists for the different list types. Commands are limited to Moderators+.
* Make the cache more convenient to access.Gravatar Leon Sandøy2020-07-18-1/+13
| | | | | | Instead of just dumping the JSON response from the site, we'll build a data structure that it will be convenient to access from our new cog, and from the Filtering cog.
* Bulletlist with valid file types in converter.Gravatar Leon Sandøy2020-07-18-2/+2
|
* Better BadArgument exception text.Gravatar Leon Sandøy2020-07-18-2/+3
|
* Kaizen: Make error_handler.py more embeddy.Gravatar Leon Sandøy2020-07-17-7/+25
| | | | | | | | | | | | | | Currently, some types of errors are returning plain strings that repeat the input (which can be exploited to deliver stuff like mentions), and others are returning generic messages that don't give any exception information. This commit unifies our approach around putting as much information as we can (including the exception message), but always putting it inside an embed, so that stuff like pings will not fire. This, combined with the 1.4.0a `allowed_mentions` functionality, seems like a reasonable compromise between security and usability.
* Add ValidAllowDenyListType converter.Gravatar Leon Sandøy2020-07-17-1/+27
| | | | | | | | We'll use this to ensure the input is valid when people try to whitelist or blacklist stuff. It will fetch its data from an Enum maintained on the site, so that the types of lists we support will only need to be maintained in a single place, instead of duplicating that data in the bot and the site.
* Cache AllowDenyList data at bot startup.Gravatar Leon Sandøy2020-07-17-0/+7
| | | | | | | | | We shouldn't be making an API call for every single message posted, so what we're gonna do is cache the data in the Bot, and then update the cache whenever we make changes to it via our new AllowDenyList cog. Since this cog will be the only way to make changes to this, this level of lazy caching should be enough to always keep the cache up to date.
* Kaizen: Move OffTopicName to converters.py.Gravatar Leon Sandøy2020-07-17-29/+28
|
* Bugfix: Show ID for embed DM relays, too.Gravatar Leon Sandøy2020-07-16-1/+1
|
* Merge pull request #1046 from ItsCinnabar/patch-1Gravatar Sebastiaan Zeeff2020-07-16-1/+1
|\ | | | | Update or-gotcha.md
| * Merge branch 'master' into patch-1Gravatar Sebastiaan Zeeff2020-07-16-1/+8
| |\ | |/ |/|
* | Merge pull request #1045 from python-discord/bug/980/fuzzy-processingGravatar Dennis Pham2020-07-15-1/+8
|\ \ | | | | | | Invoke fuzzywuzzy's processor before matching
| * \ Merge branch 'master' into bug/980/fuzzy-processingGravatar Dennis Pham2020-07-15-177/+427
| |\ \ | |/ / |/| |
| * | Pass processed string to `extractBests`Gravatar kosayoda2020-07-15-2/+2
| | | | | | | | | | | | | | | Fixes a regression where the string to be matched was not processed beforehand.
| * | Invoke fuzzywuzzy's processor before matchingGravatar kosayoda2020-07-14-1/+8
| | | | | | | | | | | | | | | | | | | | | Trying to match a string with only non-alphanumeric characters results in a warning by fuzzywuzzy. Processing the string before matching lets us avoid the warning, which which uses the root logger and thus isn't supressible.
| | * Merge branch 'master' into patch-1Gravatar Dennis Pham2020-07-15-175/+425
| | |\ | |_|/ |/| |
* | | Merge pull request #1054 from python-discord/dm_relayGravatar Leon Sandøy2020-07-15-6/+43
|\ \ \ | | | | | | | | DM relay - minor fixes
| * \ \ Merge branch 'master' into dm_relayGravatar Leon Sandøy2020-07-15-70/+202
| |\ \ \ | |/ / / |/| | |
* | | | Merge pull request #1041 from python-discord/dm_relayGravatar Kieran Siek2020-07-15-107/+198
|\ \ \ \ | | | | | | | | | | Relay all DMs sent to the bot to #dm_log
| * \ \ \ Merge branch 'master' into dm_relayGravatar Kieran Siek2020-07-15-15/+49
| |\ \ \ \ | |/ / / / |/| | | |
* | | | | Merge pull request #1044 from python-discord/talentpool-oldestGravatar Kieran Siek2020-07-15-5/+43
|\ \ \ \ \ | | | | | | | | | | | | Allow ordering talentpool/bigbrother watched users by oldest added
| * \ \ \ \ Merge branch 'master' into talentpool-oldestGravatar kosayoda2020-07-15-10/+6
| |\ \ \ \ \ | |/ / / / / |/| | | | |
* | | | | | Merge pull request #1051 from ↵Gravatar Dennis Pham2020-07-14-7/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/info/1050/remove-help-reactions-404 Suppress NotFound when removing help command reactions
| * \ \ \ \ \ Merge branch 'master' into bug/info/1050/remove-help-reactions-404Gravatar Dennis Pham2020-07-14-3/+0
| |\ \ \ \ \ \ | |/ / / / / / |/| | | | | |
* | | | | | | HelpChannels: remove cooldown info from available messageGravatar MarkKoz2020-07-14-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Users can no longer see available channels if they're on cooldown. They will instead see a special "cooldown" channel which will explain what's going on.
| * | | | | | Suppress NotFound when removing help cmd reactionsGravatar MarkKoz2020-07-14-7/+6
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The message may be deleted somehow before the wait_for times out. Fixes #1050 Fixes BOT-6X