`size` is unused in `OutgoingMessage.prototype._writeRaw`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 122k
- Forks
- 37.3k
- Ø Merge
- 4 T. 2 Std.
- Gemergte PRs (30 T.)
- 283
Beschreibung
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?
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in lib/_http_outgoing.js bei OutgoingMessage.prototype._send und _writeRaw und vergleiche anschließend deren Argumente mit der in #46601 eingeführten Änderung. Überprüfe, ob size und byteLength das Verhalten beeinflussen; die Arbeit ist abgeschlossen, wenn ihre Verwendung oder Entfernung begründet ist und die relevanten bestehenden HTTP-Tests weiterhin bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- api, backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 55/100