bot - A mirrored copy of the python-discord/bot repository.

	Commit message (Collapse)	Author	Lines
2020-02-23	Reminders: don't cancel task if reminder is invalid when rescheduling	MarkKoz	-6/+11
	If a reminder is invalid, it won't get rescheduled. Therefore, there wouldn't exist a task to cancel and it'd raise a warning. Fixes BOT-1C
2020-02-23	Add Sentdex server to whitelist	Leon Sandøy	-0/+1

2020-02-24	Change verification post log level to info, tidy code.	scragly	-14/+13

2020-02-24	Reduce log level of tag cooldown notice.	scragly	-2/+4

2020-02-23	Remove call to delete reminder, as ensure method already does it.	scragly	-1/+0

2020-02-23	Check reminder user and channel before send and schedule.	scragly	-17/+38

2020-02-23	Don't log exception traceback on Forbidden for welcomes.	scragly	-12/+9

2020-02-23	Don't set project log level so it uses root level.	scragly	-1/+1

2020-02-23	Define `_count` in `__init__`.	scragly	-0/+1

2020-02-23	Suppress NotFound on react clear, tidy imports.	scragly	-18/+21

2020-02-21	Change defcon log levels from warning to info	Joseph Banks	-2/+2

2020-02-21	Enable logging warnings to Sentry	Joseph Banks	-3/+12

2020-02-21	Bot: send empty cache warning to a webhook	MarkKoz	-3/+12
	This is more visible than it would be if it was only logged. * Add a webhook for the dev-log channel to constants
2020-02-21	Attach jump_to url to the event	Joseph Banks	-0/+8

2020-02-21	Use push_scope instead of configure_scope	Joseph Banks	-2/+2

2020-02-21	Use log.error instead of capture_exception	Joseph Banks	-2/+2

2020-02-21	Remove AIOHTTP integration from Sentry since it is intended for AIOHTTP servers.	Joseph Banks	-3/+1

2020-02-21	Remove tests for custom bot log	Joseph Banks	-62/+2

2020-02-21	Remove the space that makes the test fail	Deniz	-1/+1

2020-02-21	Use the code provided by sco1 to fix the checks failing.	Deniz	-5/+10

2020-02-21	Add basic sentry usage for command errors giving user information and ↵	Joseph Banks	-85/+10
	command context.
2020-02-21	Attach extra information on a command error	Joseph Banks	-1/+13

2020-02-21	Set BOT_SENTRY_DSN environment variable for tests	Joseph Banks	-1/+1

2020-02-21	Implement basic Sentry SDK usage	Joseph Banks	-2/+14

2020-02-20	Update flake8-annotations to v2.0.	scragly	-10/+51

2020-02-20	Change to explicit logs, remove logmatic.	scragly	-160/+76

2020-02-19	API: add comment explaining class attributes	MarkKoz	-0/+2
	Explain changes caused by 22a55534ef13990815a6f69d361e2a12693075d5.
2020-02-19	Sync: add confirmation timeout and max diff to config	MarkKoz	-16/+24

2020-02-19	Moderation: show avatar in infraction deactivation log	MarkKoz	-0/+5

2020-02-17	Pagination migrations - Emoji Data Structure Modified	F4zi	-1/+1
	Changed the pagination emoji collection from list to tuple This change was suggested since this collection is constant
2020-02-17	Change snekbox api url to internal docker domain.	scragly	-1/+1
	NGINX has been dockerised, and proxy passes now reference internal container domains rather than referencing host or external domains. This will have a few extra benefits: - Less external factors involved for resolving the service address - Can work with the same address on development envs - Snekbox can be closed down entirely so it's inaccessible from external networks.
2020-02-16	Fix missing Django logs when using Docker Compose	MarkKoz	-0/+2
	Fixed by allocating a pseudo-tty to the web and bot services in Docker Compose.
2020-02-15	Moderation: show member as a mention in expired infraction log	MarkKoz	-1/+1

2020-02-15	Moderation: show correct actor in expired infraction log	MarkKoz	-3/+4
	The bot user was always being used instead of using the actor field of the infraction.
2020-02-15	Moderation: add creation date & duration to expired infraction log	MarkKoz	-1/+7
	Closes #685
2020-02-15	Utils: refactor format_infraction_with_duration and support absolute val	MarkKoz	-13/+23
	The absolute value is useful for preventing negative values in the case that date_to is in the past relative to date_from. * Add an absolute parameter to take the absolute value of the duration * Rename expiry to date_to * Rewrite the docstring
2020-02-15	Add missed signature reformat from review	S. Co1	-1/+4
	Co-Authored-By: Mark <[email protected]>
2020-02-15	Add "cancel" as a reminder delete alias	S. Co1	-1/+1

2020-02-15	Add full capture of reason string to superstarify invocation	S. Co1	-1/+2

2020-02-15	Styling & refactors from review	S. Co1	-17/+29
	* Refactor confirmation embed footer string generation to be more concise * Multiline long method calls * Refactor humanized delta f string generation for readability * Switch from `datetime.isoformat` to `dateutils.parser.isoparse` to align with changes elsewhere in the codebase (should be more robust) * Shift reminder channel whitelist to constants Co-Authored-By: Mark <[email protected]>
2020-02-14	Stop scheduling expiration of permanent infractions on edit	Sebastiaan Zeeff	-2/+5
	https://github.com/python-discord/bot/issues/751 The infraction edit command defined in `bot.cogs.moderation.management` contained a bug causing it to attempt to schedule an expiration task when turning a temporary infraction into a permanent infraction. Since the "expires_at" field of a permanent infractions is `None`, this caused an exception to occur in the scheduler: Traceback (most recent call last): File "/bot/bot/cogs/moderation/scheduler.py", line 415, in _scheduled_task expiry = dateutil.parser.isoparse(infraction["expires_at"]).replace(tzinfo=None) File "/usr/local/lib/python3.7/site-packages/dateutil/parser/isoparser.py", line 37, in func return f(self, str_in, args, *kwargs) File "/usr/local/lib/python3.7/site-packages/dateutil/parser/isoparser.py", line 134, in isoparse components, pos = self._parse_isodate(dt_str) File "/usr/local/lib/python3.7/site-packages/dateutil/parser/isoparser.py", line 208, in _parse_isodate return self._parse_isodate_common(dt_str) File "/usr/local/lib/python3.7/site-packages/dateutil/parser/isoparser.py", line 213, in _parse_isodate_common len_str = len(dt_str) TypeError: object of type 'NoneType' has no len() I have solved this by adding a check that makes sure we only schedule an expiration task when the `"expires_at"` field has a truthy value (which all valid datetime strings are) using `if request_data['expires_at']`. IMPORTANT NOTE: While it's tempting to just skip the entire scheduling block for permanent infractions, it's essential to unschedule existing expiration tasks for this infraction as we're changing a temporary infraction to a permanent infraction. This commit closes #751
2020-02-13	Add thousand separators to Members count, closes #744	Deniz	-4/+4

2020-02-13	Update the tests accordingly to reflect the new changes	Deniz	-3/+3

2020-02-12	Expand reminder channel whitelist to dev-contrib for non-staff	S. Co1	-1/+3
	Add channel ID to config files
2020-02-12	Add reminder target datetime to footer of confirmation message	S. Co1	-5/+16

2020-02-12	Add reminder ID to footer of confirmation message	S. Co1	-5/+8

2020-02-12	Use collections.Counter properly. Use the ChannelType enum instead of the ↵	Deniz	-12/+6
	__class__ attribute, and re-add the None check for !user roles.
2020-02-12	Tests: fix unawaited error for MockAPIClient	MarkKoz	-6/+5
	This error is due to the use of an actual instance of APIClient as the spec for the mock. recreate() is called in __init__ which in turn creates a task for the _create_session coroutine. The approach to the solution is to use the type for the spec rather than and instance, thus avoiding any call of __init__. However, without an instance, instance attributes will not be included in the spec. Therefore, they are defined as class attributes on the actual APIClient class definition and given default values. Alternatively, a subclass of APIClient could have been made in the tests.helpers module to define those class attributes. However, it seems easier to maintain if the attributes are in the original class definition.
2020-02-12	Sync tests: use async_test decorator	MarkKoz	-94/+135

2020-02-12	Sync tests: test on_member_join	MarkKoz	-0/+52
	Should PUT user's data or POST it if the user doesn't exist. ResponseCodeError should be re-raised if status code isn't a 404. A helper method was added to reduce code redundancy between the 2 tests.