Add configurable logger #434
Conversation
This PR copies the `Log` implementation from Miniflare 2, and allows logging to be configured via the `log` shared option.
|
| export enum LogLevel { | ||
| NONE, | ||
| ERROR, | ||
| WARN, | ||
| INFO, | ||
| DEBUG, | ||
| VERBOSE, | ||
| } | ||
|
|
||
| const LEVEL_PREFIX: { [key in LogLevel]: string } = { | ||
| [LogLevel.NONE]: "", | ||
| [LogLevel.ERROR]: "err", | ||
| [LogLevel.WARN]: "wrn", | ||
| [LogLevel.INFO]: "inf", | ||
| [LogLevel.DEBUG]: "dbg", | ||
| [LogLevel.VERBOSE]: "vrb", | ||
| }; | ||
|
|
||
| const LEVEL_COLOUR: { [key in LogLevel]: Colorize } = { | ||
| [LogLevel.NONE]: reset, | ||
| [LogLevel.ERROR]: red, | ||
| [LogLevel.WARN]: yellow, | ||
| [LogLevel.INFO]: green, | ||
| [LogLevel.DEBUG]: grey, | ||
| [LogLevel.VERBOSE]: (input) => dim(grey(input as any)) as any, | ||
| }; |
There was a problem hiding this comment.
Non blocking but Enums are rough on the TS server and compiler and turn into functions at runtime.
Could use something like an Object literal or Map for the types?
There was a problem hiding this comment.
In the interest of minimising breaking API changes from Miniflare 2, I think I'll keep this as an enum, but agree, a const object literal would be nice here.
This PR copies the
Logimplementation from Miniflare 2, and allows logging to be configured via thelogshared option. Even though none of our gateways need a logger at the moment, I've added it in case we need it later.