Skip to content

Refactor Logging #170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Refactor Logging #170

wants to merge 10 commits into from

Conversation

p0rtL6
Copy link

@p0rtL6 p0rtL6 commented Apr 6, 2025

This is a combination of #168 and #169 with more additions on top of it.

  • The option -q to disable the coloring of console logs has been added.
  • Log formats have been standardized between share/dir/file
  • Logs in the console are padded so they line up in columns for easier reading
  • Colors have been changed to make things more readable
  • HTML logging has been updated to reflect these changes
  • Fixed a bug in JSON logging that templated the triage level with a ToString() of the message class, this also fixes the JSON logs themselves so the message is under the correct name in event properties instead of the triage name.
  • Other reworks and refactors of the logging system to make things easier to change and maintain

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

Successfully merging this pull request may close these issues.

1 participant