flow-go
flow-go copied to clipboard
Mercilessly refactor `panic()` calls into `zerolog` `.Panic()` calls
Problem Definition
Currently zerolog is used for all logging to JSON format except for panic() calls which get output as non-JSON. This is a source of inconsistency in the logging.
Proposed Solution
zerolog has its own .Panic() function which does everything that panic() does and outputs the zerolog formatted JSON log line too.
Definition of Done
No more panic() statements.
More info here [1].
[1] https://www.notion.so/dapperlabs/Comprehend-logs-forward-port-debug-logs-804323ea1973438ea0671998df2484bf