winstonjs/winston

Custom log levels can be silent unless level is also specified

Open

#1491 aperta il 3 ott 2018

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (1789 fork)batch import
BugGood First IssueHelp Wanted

Metriche repository

Star
 (21.967 star)
Metriche merge PR
 (Merge medio 68g 4h) (9 PR mergiate in 30 g)

Descrizione

It is easy to write the following code and not realize that one has made a mistake:

const logger = winston.createLogger({
  levels: { eeny: 0, meeny: 1, miney: 2, moe: 3},
  transports: [
    new winston.transports.Console(),
  ]
});

logger.meeny('catch a tiger by the toe'); // never hollers

This could be addressed by the following check in the Logger constructor:

    if (!Object.keys(this.levels).includes(this.level)) {
      throw new Error(`Logger level ${this.level} is not in the levels: ${this.levels}`);
    }

I can open a PR if people agree it is worth fixing.

Guida contributor

Direzione di ricerca
Aggiungere un controllo di convalida nel costruttore di Logger per assicurarsi che il livello predefinito esista nell'oggetto dei livelli personalizzati.
Tech stack
javascriptnodejs
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Node.jswinston basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80