You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The thing is winston documentation says const { level, message, ...meta } = info; is ok, also I tried to connect this module with https://github.com/logdna/logdna-winston/blob/master/index.js#L29 assuming meta is all other message fields except message itself and level...
Consequently I have meta.meta.res.statusCode in my logs... Visually it's ok ok, but it's not so important.
However, the express-winston library calls winston using the syntax: logger.log({level, message: msg, meta})
when it should be: logger.log(level, msg, meta)
Doing this will correctly insert the information in the correct location in the logs. PR forthcoming however this will dramatically change the output from the library and I have a feeling will not be accepted unless we have a major version upgrade.
Hi!
Thank you for awesome plugin 👍
Is it possible to allow users send meta not inside separate
meta
field, but in the same object with message and level here https://github.com/bithavoc/express-winston/blob/master/index.js#L360?The thing is winston documentation says
const { level, message, ...meta } = info;
is ok, also I tried to connect this module with https://github.com/logdna/logdna-winston/blob/master/index.js#L29 assuming meta is all other message fields except message itself and level...Consequently I have meta.meta.res.statusCode in my logs... Visually it's ok ok, but it's not so important.
But I haven't written this issue if I haven't bumped into
Depth of parsed nested fields
limitation of logging service https://docs.logdna.com/docs/ingestion#section-service-limitsThe text was updated successfully, but these errors were encountered: