Debugging Improvements #39

Merged

newsch merged 3 commits from debugging into main

2024-04-28 18:20:52 +00:00

newsch commented

2024-01-24 20:43:07 +00:00

(Migrated from github.com)

Add check-html command to bulk-analyze html files
Add option to write matched, erroring, or panicing json to stdout
decide on parseable structured logging format, json or logfmt?

- Add check-html command to bulk-analyze html files - Add option to write matched, erroring, or panicing json to stdout - [x] decide on parseable structured logging format, json or logfmt?

biodranik (Migrated from github.com) approved these changes 2024-02-22 07:37:05 +00:00

biodranik (Migrated from github.com) left a comment

LGTM

newsch commented

2024-03-14 17:06:16 +00:00

(Migrated from github.com)

@biodranik do you care what format the logs are in?
Something like logfmt would be more readable than JSON but also more parseable for when I'm analyzing the logs than tracing's default. That way we can keep it simple without having multiple levels/formats/outputs or detection logic.

@biodranik do you care what format the logs are in? Something like [logfmt](https://brandur.org/logfmt) would be more readable than JSON but also more parseable for when I'm analyzing the logs than `tracing`'s default. That way we can keep it simple without having multiple levels/formats/outputs or detection logic.

biodranik commented

2024-03-15 00:55:34 +00:00

(Migrated from github.com)

I'm ok to try it )

biodranik commented

2024-04-21 12:58:13 +00:00

(Migrated from github.com)

@newsch should these changes be merged?

newsch commented

2024-04-28 18:21:00 +00:00

(Migrated from github.com)

@biodranik no need to rerun in production with this yet.