aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* | | | | | | | | | | | | | Adding helpers to the Filtering whitelistGravatar Leon Sandøy2020-03-02-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Resolves an issue mentioned in https://github.com/python-discord/bot/issues/767, giving Helpers access to post invites and other things caught by the Filtering cog.
| * | | | | | | | | | | | | Fix typo in commentGravatar Mark2020-03-03-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Leon Sandøy <[email protected]>
| * | | | | | | | | | | | | Merge remote-tracking branch 'origin/master' into ↵Gravatar MarkKoz2020-02-25-1156/+2657
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug/backend/b748/resolver-in-coro
| * | | | | | | | | | | | | | API: close existing open session before recreating itGravatar MarkKoz2020-02-15-3/+4
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | API: add argument to force recreation of the sessionGravatar MarkKoz2020-02-15-4/+10
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Bot: send not-closed warnings as log messagesGravatar MarkKoz2020-02-15-12/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Real" warnings weren't showing up for some reason.
| * | | | | | | | | | | | | | Bot: warn when connector/session not closed when recreatingGravatar MarkKoz2020-02-15-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | aiohttp does warn too, but these warnings will provide more immediate feedback.
| * | | | | | | | | | | | | | Bot: call _recreate() in clear()Gravatar MarkKoz2020-02-15-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because discord.py recreates the HTTPClient session, may as well follow suite and recreate our own stuff here too.
| * | | | | | | | | | | | | | Bot: move connector/session recreation to a separate functionGravatar MarkKoz2020-02-15-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The function itself doesn't need to be a coroutine. It just has to be called in a coroutine (or, more indirectly, in an async context?).
| * | | | | | | | | | | | | | Bot: override login() instead of start()Gravatar MarkKoz2020-02-15-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The client can be used without running a bot so it makes more sense for the connector to be created when logging in, which is done in both cases, rather than in start(), which is only used when running a bot.
| * | | | | | | | | | | | | | Bot: fix error trying to close a None sessionGravatar MarkKoz2020-02-14-1/+3
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Bot: add warning for when connector is a specified kwargGravatar MarkKoz2020-02-13-0/+6
| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Bot: avoid DeprecationWarning for aiohttp.AsyncResolver (fix #748)Gravatar MarkKoz2020-02-13-15/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AsyncResolver has to be created inside a coroutine so it's moved inside start(). Consequently, the APIClient session is also recreated inside start() now. When using clear(), the default connector is used for the session it recreates because clear() is not a coroutine. This should only affect requests made to the Discord API via the Client when not using it to run a bot; starting the bot will re-create the session with the custom connector. * Close connector and resolver when bot closes
| * | | | | | | | | | | | | | API: accept additional session kwargs for recreate()Gravatar MarkKoz2020-02-13-7/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These kwargs are merged with the kwargs given when the APIClient was created. This is useful for facilitating changing the session's connector with a new instance when the session needs to be recreated. * Rename _session_args attribute to _default_session_kwargs
| | | * | | | | | | | | | | | Add logging to antimalware cog & expand user feedbackGravatar S. Co12020-03-01-17/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add generic handling for multi-file uploads * Log user, id, and blocked extensions * Provide the full list of attachment filenames as a logging extra * Provide feedback on all blacklisted file types uploaded
| | | | | | | * | | | | | | | Convert "get_tags()" and "_get_tag()" to sync functionsGravatar RohanJnr2020-03-04-10/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "get_tags()" and "_get_tag()" functions need not be async as we are no longer doing any API call but instead reading from local files.
| | | | | | | * | | | | | | | Use "pathlib" instead of "os" module and context managerGravatar RohanJnr2020-03-04-11/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The pathlib module simplifies opening and reading files, hence the os module and the context manager are no longer used.
| | | | | | | * | | | | | | | Merge branch 'tags_overhaul' of https://github.com/RohanJnr/bot into ↵Gravatar RohanJnr2020-02-29-895/+2636
| | | | | | | |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tags_overhaul
| | | | | | | | * \ \ \ \ \ \ \ Merge branch 'master' into tags_overhaulGravatar Rohan2020-02-29-895/+2636
| | | | | | | | |\ \ \ \ \ \ \ \
| | | | | | | * | | | | | | | | | Caching all the tags when the bot has loaded(caching only once) insted of ↵Gravatar RohanJnr2020-02-29-21/+18
| | | | | | | |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | caching it after the tags command is used.
| | | | | | | * | | | | | | | | Re-corrected the lines which I had changed by mistakeGravatar RohanJnr2020-02-29-1/+2
| | | | | | | | | | | | | | | |
| | | | | | | * | | | | | | | | added white spaces on statements before bullet points for proper rendering ↵Gravatar RohanJnr2020-02-29-75/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of points on github
| | | | | | | * | | | | | | | | Added all the tag files in resources and modified cogs/tags.py file to ↵Gravatar RohanJnr2020-02-29-84/+919
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | access the static tag files rather than sending an API get request. Removed all methods calling the API so the tags cannot be edited, added nor deleted.
| | | | | | | | | | | | | * | | Add tests for `HushDurationConverter`.Gravatar Numerlor2020-03-17-0/+30
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Add tests ensuring permissions get preserved.Gravatar Numerlor2020-03-17-0/+31
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Use set `discard` instead of `remove`.Gravatar Numerlor2020-03-17-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Discard ignores non present values, allowing us to skip the KeyError suppress.
| | | | | | | | | | | | | * | | Add docstring to test.Gravatar Numerlor2020-03-17-0/+1
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Fix test name.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `removed` was describing the opposite behaviour.
| | | | | | | | | | | | | * | | Remove `alert_channel` mention from docstring.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After removing the optional channel arg and changing output message channels we're only testing `ctx`'s `send`.
| | | | | | | | | | | | | * | | Remove one indentation level.Gravatar Numerlor2020-03-17-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
| | | | | | | | | | | | | * | | Reword `unsilence` docstring.Gravatar Numerlor2020-03-17-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
| | | | | | | | | | | | | * | | Remove `channel` mentions from command docstrings.Gravatar Numerlor2020-03-17-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the new behaviour of not accepting channels and muting the current one, it's no longer neccessary to keep the channel param in the docstring. Co-authored-by: MarkKoz <[email protected]>
| | | | | | | | | | | | | * | | Rename `cog_unload` tests.Gravatar Numerlor2020-03-17-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previous names were undescriptive from testing phases.
| | | | | | | | | | | | | * | | Fix `test_cog_unload` passing tests with invalid values.Gravatar Numerlor2020-03-17-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The first assert - `asyncio_mock.create_task.assert_called_once_with` called `alert_channel`'s send resulting in an extra call. `send` on `alert_channel` was not tested properly because of a typo and a missing assert in the method call.
| | | | | | | | | | | | | * | | Handle and log `CommandErrors` on `.can_run`.Gravatar Numerlor2020-03-16-1/+6
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Remove long indentation from docstrings.Gravatar Numerlor2020-03-15-6/+6
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Set `_get_instance_vars_event` in test's `setUp`.Gravatar Numerlor2020-03-15-0/+2
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Adjust tests for new calling behaviour.Gravatar Numerlor2020-03-15-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `.set_permissions` calls were changed to use kwargs directly instead of an overwrite, this reflects the changes in tests.
| | | | | | | | | | | | | * | | Document returns values of private methods.Gravatar Numerlor2020-03-15-0/+2
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Block commands until all instance vars are loaded.Gravatar Numerlor2020-03-15-0/+4
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Assign created task to a var.Gravatar Numerlor2020-03-15-1/+1
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Pass kwargs directly instead of a PermissionOverwrite.Gravatar Numerlor2020-03-15-9/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `set_permissions` method creates a `PermissionOverwrite` from kwargs internally, so we can skip creating it ourselves and unpack the dict directly into kwargs.
| | | | | | | | | | | | | * | | Change BadArgument error wording.Gravatar Numerlor2020-03-15-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
| | | | | | | | | | | | | * | | Add Silence cog load to docstring.Gravatar Numerlor2020-03-15-1/+1
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Move coro execution outside of if condition.Gravatar Numerlor2020-03-15-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This gives us a clearer look at the general flow control and what's getting executed. Comment was also moved to its relevant line. Co-authored-by: MarkKoz <[email protected]>
| | | | | | | | | | | | | * | | Adjust docstring styling.Gravatar Numerlor2020-03-15-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: MarkKoz <[email protected]>
| | | | | | | | | | | | | * | | Use dict instead of a set and custom class.Gravatar Numerlor2020-03-15-44/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The FirstHash class is no longer necessary with only channels and the current loop in tuples. FirstHash was removed, along with its tests and tests were adjusted for new dict behaviour.
| | | | | | | | | | | | | * | | Add tests for `_notifier`.Gravatar Numerlor2020-03-11-0/+19
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Improve commands help.Gravatar Numerlor2020-03-11-3/+5
| | | | | | | | | | | | | | | |
| | | | | | | | | | | | | * | | Add logging to commands.Gravatar Numerlor2020-03-11-0/+3
| | | | | | | | | | | | | | | |