`size` is unused in `OutgoingMessage.prototype._writeRaw`
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 122k
- Forks
- 37.3k
- Merge moyen
- 4 j 2 h
- PR mergées (30 j)
- 283
Description
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?
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez dans lib/_http_outgoing.js, au niveau de OutgoingMessage.prototype._send et _writeRaw, puis comparez leurs arguments avec la modification introduite dans #46601. Vérifiez si size et byteLength ont un impact sur le comportement ; le travail est terminé lorsque leur utilisation ou leur suppression est justifiée et que les tests HTTP existants concernés passent toujours.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- api, backend
- Type d'issue
- Refactorisation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 55/100