Disable level detection if GITLAB_LOGGER_MINLEVEL not set (!18) · Merge requests · GitLab.org / Cloud Native / Gitlab Logger

Stan Hu requested to merge sh-disable-level-detection into master Aug 31, 2023

Level detection doesn't really work at the moment because the function does some dumb string matching. If the word fatal appears the message could be erroneously labeled as a fatal level. This also might cause some invalid JSON parsing if the log message includes a level field of its own. For now, disable this field if GITLAB_LOGGER_MINLEVEL is set to 0 and consider removing it in the future.

Relates to #12 (closed)

Disable level detection if GITLAB_LOGGER_MINLEVEL not set

Merge request reports