Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

ConsoleLogHandler Assumes Implementation #13

Closed
garrypolley opened this issue Aug 19, 2015 · 0 comments
Closed

ConsoleLogHandler Assumes Implementation #13

garrypolley opened this issue Aug 19, 2015 · 0 comments
Labels

Comments

@garrypolley
Copy link
Contributor

Right now this line https://github.com/cerner/canadarm/blob/master/lib/handler/console.js#L44 in the console log handler assumes there is a logAttribute.msg attribute. If you are not using the standard log appender then the attribute does not exist and the output just shows undefined, which is confusing.

Need to fix this to not assume the value exists. If it exists we should log it, if not don't log it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant