`size` is unused in `OutgoingMessage.prototype._writeRaw`
未关闭
还没有人认领这个 Issue。
- 主要语言
- JavaScript
- 星标
- 122k
- 派生
- 37.3k
- 平均合并
- 4 天 2 小时
- 30 天内合并 PR
- 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 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 lib/_http_outgoing.js 中的 OutgoingMessage.prototype._send 和 _writeRaw 开始,然后将它们的参数与 #46601 中引入的更改进行比较。验证 size 和 byteLength 是否会影响行为;当能够证明使用或移除它们的合理性,并且相关的现有 HTTP 测试仍然通过时,工作即告完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- javascript
- 领域
- api, backend
- Issue 类型
- 重构
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 活跃度
- 冷清
- 描述清晰度
- 基本清楚
- 新手友好度
- 55/100