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	API: close existing open session before recreating it	MarkKoz	-3/+4

2020-02-15	API: add argument to force recreation of the session	MarkKoz	-4/+10

2020-02-15	Bot: send not-closed warnings as log messages	MarkKoz	-12/+8
	"Real" warnings weren't showing up for some reason.
2020-02-15	Bot: warn when connector/session not closed when recreating	MarkKoz	-0/+18
	aiohttp does warn too, but these warnings will provide more immediate feedback.
2020-02-15	Bot: call _recreate() in clear()	MarkKoz	-2/+8
	Because discord.py recreates the HTTPClient session, may as well follow suite and recreate our own stuff here too.
2020-02-15	Bot: move connector/session recreation to a separate function	MarkKoz	-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?).
2020-02-15	Bot: override login() instead of start()	MarkKoz	-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.
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	Bot: fix error trying to close a None session	MarkKoz	-1/+3

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	Bot: add warning for when connector is a specified kwarg	MarkKoz	-0/+6