cadence/out-of-your-element
cadence/out-of-your-element
4
10
Fork 7
Code Issues 15 Pull requests 3 Releases 29 Activity

Various error log improvements #100

Open
l1s5 wants to merge 6 commits from l1s5/out-of-your-element:log-improvements into main
pull from: l1s5/out-of-your-element:log-improvements
merge into: cadence:main
Conversation 0 Commits 6 Files changed 5 +4 -5
2 changed files with 4 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit e505bfae82 - Show all commits

Render error messages in a single log statement

Prev Next 
Getting 2 log events for every error made them a bit hard to both read
and query for in a log ingester.
l1s5 2026-10-25 08:49:34 +09:00
commit e505bfae82
Signed by untrusted user: l1s5
GPG key ID: D0A90CC905B58D9B

5
src/d2m/discord-packets.js
View file

@ -113,9 +113,10 @@ async function onPacket(client, message, listen) {
await eventDispatcher.checkMissedLeaves(client, message.d) await eventDispatcher.checkMissedLeaves(client, message.d)
} catch (e) { } catch (e) {
if (firstReady) { if (firstReady) {
console.error("Failed to sync missed events. To retry, please fix this error and restart OOYE:") console.error("Failed to sync missed events. To retry, please fix this error and restart OOYE:", e)
} else {
console.error(e)
cadence commented 2026-10-26 13:23:25 +00:00
Copy link

Errors in this branch are caused by internet outage. Logging it is noise because it can error from just checking if anything needed to be done, even if nothing actually took place. That's why I moved this into an if branch.

Errors in this branch are caused by internet outage. Logging it is noise because it can error from just checking if anything needed to be done, even if nothing actually took place. That's why I moved this into an if branch.
} }
console.error(e)
} }
} }

4
src/m2d/event-dispatcher.js
View file

@ -47,9 +47,7 @@ const {reg} = require("../matrix/read-registration")
const errorReportStatus = sync.remember(() => ({lastReportedEvent: 0})) const errorReportStatus = sync.remember(() => ({lastReportedEvent: 0}))
function printError(type, source, e, payload) { function printError(type, source, e, payload) {
console.error(`Error while processing a ${type} ${source} event:`) console.error(`Error while processing a ${type} ${source} event:\n${util.inspect(e, false, 4, false)}\n${util.inspect(payload, false, 4, false)}`)
console.error(e)
console.dir(payload, {depth: null})
} }
/** @param {string} stack */ /** @param {string} stack */