aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | | * | | | | | | | | | | | | | | Error handler: handle TooManyArgumentsGravatar MarkKoz2020-02-16-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Send a message specifying the error reason.
| | | * | | | | | | | | | | | | | | Error handler: handle MissingRequiredArgumentGravatar MarkKoz2020-02-16-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Send a message indicating which argument is missing.
| | | * | | | | | | | | | | | | | | Error handler: simplify error importsGravatar MarkKoz2020-02-16-29/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Import the errors module and qualify the error types with it rather than importing a large list of error types.
| | | * | | | | | | | | | | | | | | Error handler: create separate function to handle UserInputErrorGravatar MarkKoz2020-02-16-12/+17
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: move help command retrieval to a separate functionGravatar MarkKoz2020-02-16-11/+18
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: move tag retrieval to a separate functionGravatar MarkKoz2020-02-16-18/+28
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: log unhandled exceptions instead of re-raisingGravatar MarkKoz2020-02-16-2/+2
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: create separate function to handle ResponseCodeErrorGravatar MarkKoz2020-02-16-14/+17
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Error handler: create separate function to handle CheckFailureGravatar MarkKoz2020-02-16-15/+29
| | | | | | | | | | | | | | | | | |
| | | * | | | | | | | | | | | | | | Moderation: fix member not found error not being shownGravatar MarkKoz2020-02-16-1/+1
| | | | |_|_|_|/ / / / / / / / / / | | | |/| | | | | | | | | | | | |
| | | | | | | * | | | | | | | | | Merge branch 'master' into fuzzy_zero_divGravatar Sebastiaan Zeeff2020-02-27-33/+65
| | | | | | | |\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / / / / / / / / |/| | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | Merge pull request #798 from python-discord/bug/mod/bot-1v/infr-edit-task-cancelGravatar Sebastiaan Zeeff2020-02-27-1/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Moderation: fix task cancellation for permanent infraction when editing
| * | | | | | | | | | | | | | | | | Moderation: fix task cancellation for permanent infraction when editingGravatar MarkKoz2020-02-27-1/+3
|/ / / / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A task should not be cancelled if an infraction is permanent because tasks don't exist for permanent infractions. Fixes BOT-1V
* | | | | | | | | | | | | | | | | Merge pull request #785 from Numerlor/channel_constants_checkGravatar Mark2020-02-27-29/+53
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|/ / / / / / / |/| | | | | | | | | | | | | | | | Verify channels against server and reddit OAuth2 in constants on startup.
| * | | | | | | | | | | | | | | | Merge branch 'master' into channel_constants_checkGravatar Mark2020-02-27-9/+20
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | Merge pull request #790 from ks129/masterGravatar S. Co12020-02-26-2/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added ignoring staff to Antimalware check.
| * | | | | | | | | | | | | | | | | Add clarifying comment to role checking logic implementationGravatar S. Co12020-02-26-0/+1
| | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Merge branch 'master' into masterGravatar S. Co12020-02-26-7/+12
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | |
* | | | | | | | | | | | | | | | | | Disable TRACE logging for Sentry breadcrumbs.Gravatar Joseph2020-02-26-1/+1
| |_|_|/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Added DMs ignoring to antimalware checkGravatar ks1232020-02-26-1/+2
| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Added to AntiMalware staff ignore check.Gravatar Karlis. S2020-02-26-1/+5
| | |_|_|_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Fix comparison operator when checking token expiration.Gravatar Numerlor2020-02-26-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With `<` the check only went through when the token was already expired, making revoking redundant; and didn't go through when the token still had some time before expiration.
| | * | | | | | | | | | | | | | Check for falsy values instead of ``""` and `None` explicitly.Gravatar Numerlor2020-02-26-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After the change to also check empty strings to avoid unucessary requests, it is no longer necessary to do an explicit value check, as the only values that can come from the .env file are `None` and strings Co-authored-by: Karlis S <[email protected]>
| | * | | | | | | | | | | | | | Specify the logged time is in UTC.Gravatar Numerlor2020-02-26-1/+1
| | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Create task for `revoke_access_token` when unloading cog to ensure it's ↵Gravatar Numerlor2020-02-26-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | executed.
| | * | | | | | | | | | | | | | Make sure token exists before checking its expiration.Gravatar Numerlor2020-02-26-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without the check and an invalid token, an AttributeError is raised; blocking the relevant ClientError from being raised in `get_access_token`.
| | * | | | | | | | | | | | | | Check for empty strings alongside None before loading cog.Gravatar Numerlor2020-02-26-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Docker fetches values from the .env itself and defaults to "" instead of None, needing to do invalid access token requests before unloading itself.
| | * | | | | | | | | | | | | | Restyle if body to include the error instead of adding the cog.Gravatar Numerlor2020-02-25-3/+3
| | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Fix `__iter__` for classes without subsections.Gravatar Numerlor2020-02-25-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous implementation assumed the config class was a subsection, failing with a KeyError if it wasn't one. Co-authored-by: kwzrd <[email protected]>
| | * | | | | | | | | | | | | | Assign created task to a variable.Gravatar Numerlor2020-02-25-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: SebastiaanZ <[email protected]>
| | * | | | | | | | | | | | | | Wait for available guild instead of bot startup.Gravatar Numerlor2020-02-25-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: SebastiaanZ <[email protected]>
| | * | | | | | | | | | | | | | Merge branch 'master' into channel_constants_checkGravatar Numerlor2020-02-25-551/+1704
| | |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Do not attempt to load Reddit cog when environment variables are not provided.Gravatar Numerlor2020-02-24-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When environment variables weren't provided; the cog attempted to create a BasicAuth object with None as values resulting in an exception before the event loop was started and a subsequent crash.
| | * | | | | | | | | | | | | | Remove unused `chunks` function and its tests.Gravatar Numerlor2020-02-24-26/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The function was only used in the since removed `Events` cog.
| | * | | | | | | | | | | | | | Change docstring mood.Gravatar Numerlor2020-02-24-1/+1
| | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Change warning text.Gravatar Numerlor2020-02-24-1/+1
| | | | | | | | | | | | | | | |
| | * | | | | | | | | | | | | | Add ConfigVerifier cog.Gravatar Numerlor2020-02-24-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds ConfigVerifier which verifies channels when loaded.
| | * | | | | | | | | | | | | | Implement `__iter__` on constants YAMLGetter.Gravatar Numerlor2020-02-24-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Python tries to fall back on passing indices to `__getitem__` without iter implemented; failing on the first line.
| | | | | | | | * | | | | | | | Remove unnecessary newlines from call.Gravatar Numerlor2020-02-27-4/+1
| | | | | | | | | | | | | | | |
| | | | | | | | * | | | | | | | Log invalid tag names in the error handler tag fallback.Gravatar Numerlor2020-02-26-5/+13
| | | | | | | | | | | | | | | |
| | | | | | | | * | | | | | | | Remove logging from tag converters.Gravatar Numerlor2020-02-26-10/+0
| | | | | | | | | | | | | | | |
| | | | | | | | * | | | | | | | Adjust tests for new converter behavior.Gravatar Numerlor2020-02-26-1/+1
| | | | | | | | | | | | | | | |
| | | | | | | | * | | | | | | | Remove number check on tags.Gravatar Numerlor2020-02-26-14/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This case is already covered by checking if at least one letter is included.
| | | | | | | | * | | | | | | | Pass error handler tag fallback through TagNameConverter.Gravatar Numerlor2020-02-26-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tag fallback didn't convert tags, resulting in possible invalid tag names being passed to the `tags_get_command`. This makes sure they're valid and ignores the risen exception if they are not.
| | | | | | | | * | | | | | | | Make sure tag name contains at least one letter.Gravatar Numerlor2020-02-26-0/+6
| |_|_|_|_|_|_|/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With only ascii and numbers being allowed to go through, possible values still included things like `$()` which don't match anything in `REGEX_NON_ALPHABET` from tags.py resulting in an error.
* | | | | | | | | | | | | | | Merge pull request #781 from ↵Gravatar S. Co12020-02-25-6/+11
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/bug/utils/bot-1c/reminder-unschedule-failure Reminders: don't cancel task if reminder is invalid when rescheduling
| * | | | | | | | | | | | | | Merge branch 'master' into bug/utils/bot-1c/reminder-unschedule-failureGravatar Joseph2020-02-25-551/+1704
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | |
* | | | | | | | | | | | | | | Merge pull request #711 from python-discord/bug/backend/b704/ready-missing-cacheGravatar Mark2020-02-23-536/+1657
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / / |/| | | | | | | | | | | | | | Prevent the role syncer from wiping the database table during API latency
| * | | | | | | | | | | | | | Merge remote-tracking branch 'origin/master' into ↵Gravatar MarkKoz2020-02-23-621/+997
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug/backend/b704/ready-missing-cache
* | | | | | | | | | | | | | | Merge pull request #749 from python-discord/reminder-enhancementsGravatar Leon Sandøy2020-02-23-15/+47
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / / |/| | | | | | | | | | | | | | Reminder Embed Enhancements & Whitelist Expansion