mirror of
https://github.com/Rapptz/discord.py.git
synced 2025-08-19 17:41:00 +00:00
Upstream change in Python
The prior changes to use of return in finally (#9981) are now insufficient. Without disclosing their intent when raising the issue, this was used by them as part of justifying a SyntaxError for working code outside of the normal process for adding errors, and with it presenting to end users in a way that breaks downstream user's existing CI While making the change, I've continued to not log errors like CancellationError or TimeoutError to users here by default, as it is not an error they need to be aware of during shutdown given the limited kinds of BaseException that could raise in this context, see: #9984 for prior analysis. I've added a debug log should anyone want access to this kind of failure while debugging gateway close, but due to how asyncio shutdown happens, this is unlikely to ever log anything useful even in a library debugging context.
This commit is contained in:
parent
705eb2c2a5
commit
439bd71c0d
@ -160,6 +160,8 @@ class KeepAliveHandler(threading.Thread):
|
||||
f.result()
|
||||
except Exception:
|
||||
_log.exception('An error occurred while stopping the gateway. Ignoring.')
|
||||
except BaseException as exc:
|
||||
_log.debug('A BaseException was raised while stopping the gateway', exc_info=exc)
|
||||
finally:
|
||||
self.stop()
|
||||
return
|
||||
|
Loading…
x
Reference in New Issue
Block a user