`size` is unused in `OutgoingMessage.prototype._writeRaw`
オープン
まだ誰も着手していません。
- 主要言語
- JavaScript
- スター
- 122k
- フォーク
- 37.3k
- 平均マージ
- 4日 2時間
- マージ済み PR（30日）
- 283
説明
Observe that the size argument is unused in OutgoingMessage.prototype._writeRaw.
This also means that the byteLength argument is unused in OutgoingMessage.prototype._send – even though it's specified in a number of places.
Introduced in #46601 but weirdly it was unused in that commit too (perhaps lost during review?)
I worry that a bug was introduced – but if it truly should be unused, can it be removed?
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
lib/_http_outgoing.js の OutgoingMessage.prototype._send と _writeRaw から開始し、それらの引数を #46601 で導入された変更と比較します。size と byteLength が動作に影響するかを確認します。それらを使用すること、または削除することの妥当性が示され、関連する既存の HTTP テストが引き続きパスすれば、作業は完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript
- 領域
- api, backend
- issue の種類
- リファクタリング
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 静か
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 55/100