gulpjs / gulpjs/gulp-cli

potential for black-holing log messages via transitive dependency on gulplog

Offen
#237 9 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
408
Forks
112
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

`gulp-util.log` checks for `hasGulplog` on module initialization and if present, defers logging to `gulplog.info`, which in turn will eat any messages below `error` (I believe) unless there is explicitly an event handler for the other levels (info, warn, etc.).

This can happen 'accidentally' if a transitive dependency uses gulplog.

It _seems_ like the reasonable thing to do is to use `gulplog.error` to ensure all logging is output properly.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Start by tracing gulp-util.log's module initialization and its hasGulplog path into gulplog.info. Reproduce the case where a transitive dependency loads gulplog and lower-severity messages disappear, then verify that the chosen logging behavior preserves output for those messages.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
cli
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.