Skip to content

Refactor LoggerWrapper for better outputs to LogStash

Ecarg requested to merge grace/T364413/update-logging-to-standard into main
  • This removes the helper functions (that do not do anything unique) so that we can just use one 'standard' to log (i.e. logger.log()), rather than having two ways to do the same things and mixing them up everywhere.
  • This adds two beneficial info to our logger data: a timestamp and stack trace, regardless of level.
  • A console.log() has also been added so the same log info can be output to our docker logs when we run things locally. This way, we can mimic what would be seen in Logstash during local test/debugging.

Bug: T369001 Bug: T369213 Bug: T364413

Edited by Ecarg

Merge request reports