search

jue89 / node-systemd-journald

Native bindings to journald
MIT License
28 stars 10 forks source link

Display NULL instead of just ignoring it? #13

Closed jue89 closed 5 years ago

jue89 commented 6 years ago

Actually null is an Object for Javascript.

With that in mind fields would not be flattened down to some entry in the Journal. They are just ignored by the converter function - and this more by accident - tbh:

https://github.com/jue89/node-systemd-journald/blob/5b9b59a8c288bfe4deb6799e127994fd328986ca/log.js#L7-L23

This is maybe not the best behaviour.

Propose: Display fields that are null with the string "null" in the Journal? Any different opinions on that?