gulpjs / gulpjs/gulp-cli

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

オープン
#237 コメント 9 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
408
フォーク
112
PR マージ指標
30日以内にマージされた PR はありません

説明

`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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

まず、gulp-util.log のモジュール初期化と、そこから gulplog.info に至る hasGulplog パスを追跡します。推移的依存関係が gulplog を読み込み、低い重大度のメッセージが消えるケースを再現し、選択したロギング動作によってそれらのメッセージの出力が維持されることを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript
領域
cli
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。