Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: use r.msg as message, rather than entire object #14

Open
airandfingers opened this issue Oct 30, 2014 · 0 comments · May be fixed by #20
Open

Feature request: use r.msg as message, rather than entire object #14

airandfingers opened this issue Oct 30, 2014 · 0 comments · May be fixed by #20

Comments

@airandfingers
Copy link

At https://github.com/mcavage/node-bunyan-syslog/blob/master/lib/sys.js#L155, SyslogStream:write stringifies r, to be appended to the header as the "message" part of the syslog

I realize that bunyan is a JSON logger and that outputting JSON is what's expected of a bunyan log stream, but for our purposes we don't need or want the entire JSON object printed as the message, for three reasons:

  • it's redundant. bunyan-syslog already outputs hostname, name, level, and timestamp in the message header.
  • none of our other syslog logs are in JSON.
  • JSON is not very human-readable, and we can't simply pipe our syslog logs through the bunyan binary.

Is there a way around these issues, other than forking this repo and modifying the aforementioned line? If not, I'd like to request an option to use r.msg as the message instead of r.

@nicokaiser nicokaiser linked a pull request May 29, 2015 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant