Skip to content

Commit

Permalink
Merge pull request #312 from FlowFuse/311-include-crash-detail-in-aud…
Browse files Browse the repository at this point in the history
…it-transmition

include crash detail in audit log transmission
  • Loading branch information
knolleary authored Dec 18, 2024
2 parents c3a1b52 + fc95237 commit 4bacf40
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion lib/launcher.js
Original file line number Diff line number Diff line change
Expand Up @@ -603,7 +603,13 @@ class Launcher {
await this.logAuditEvent('stopped')
} else {
this.state = States.CRASHED
await this.logAuditEvent('crashed')
// get last 20 lines of log
// NOTE: the log is added to the body in a specific property that should not actually
// be logged but rather extrated and examined to provide better reporting
// NOTE: 20 is typically enough to see the likes of Out Of Memory errors entries
const __launcherLog = this.getLog()?.toArray()?.slice(-20) || []
const body = { __launcherLog, info: { code, signal, info: 'Node-RED exited with non zero exit code' } }
await this.logAuditEvent('crashed', body)

// Only restart if our target state is not stopped
if (this.targetState !== States.STOPPED) {
Expand Down

0 comments on commit 4bacf40

Please sign in to comment.