flow-go icon indicating copy to clipboard operation
flow-go copied to clipboard

Mercilessly refactor `panic()` calls into `zerolog` `.Panic()` calls

Open simonhf opened this issue 3 years ago • 1 comments

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.

simonhf avatar Jul 29 '22 01:07 simonhf

More info here [1].

[1] https://www.notion.so/dapperlabs/Comprehend-logs-forward-port-debug-logs-804323ea1973438ea0671998df2484bf

simonhf avatar Jul 29 '22 01:07 simonhf