rethink-public/dpy_logger
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: discord send failure logs WARNING, not ERROR (D4)

Browse Source 
_send mirrors the record to the stdlib sink BEFORE attempting the discord send, so a
swallowed send failure is recovered cleanly - the log survives, only the channel mirror
was lost. that is a WARNING (recovered), not an ERROR. demote from _log.exception (ERROR +
traceback) to _log.warning with the reason folded in lazily; the traceback belonged on a
terminal/unhandled path, and this one recovers via the guaranteed stdlib record.

Signed-off-by: disqualifier <dev@disqualifier.me>
This commit is contained in:
dsql
2026-08-09 02:14:11 -04:00
parent d21de01ada
commit 270d7df9ae
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/dpy_logger/dpy_logger.py
+5 -2
View File
@@ -213,8 +213,11 @@ class DPYLogger:
content=content, content=content,
embed=self.build_embed(level, action, actor, log_msg), embed=self.build_embed(level, action, actor, log_msg),
) )
except Exception: except Exception as exc:
_log.exception(f"[dpy_logger] failed to send {level} log: {log_msg}") # WARNING, not ERROR: the record was already mirrored to the stdlib sink BEFORE the
# send (see _emit_stdlib above), so a swallowed discord-send failure is recovered
# cleanly - the log survives, only the channel mirror was lost. lazy interpolation.
_log.warning("[dpy_logger] failed to send %s log: %s (%s)", level, log_msg, exc)
return None return None
async def debug(self, log, action=None, actor=None, guild=None, log_to_file=None): async def debug(self, log, action=None, actor=None, guild=None, log_to_file=None):