nodejs / nodejs/node

`size` is unused in `OutgoingMessage.prototype._writeRaw`

Abierto
#57,985 4 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
JavaScript
Estrellas
122k
Forks
37.3k
Merge medio
4 d 2 h
PR fusionados (30 d)
283

Descripción

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?

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza en lib/_http_outgoing.js, en OutgoingMessage.prototype._send y _writeRaw, y compara después sus argumentos con el cambio introducido en #46601. Verifica si size y byteLength afectan al comportamiento; el trabajo estará terminado cuando esté justificado su uso o eliminación y las pruebas HTTP existentes relevantes sigan pasando.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
javascript
Área
api, backend
Tipo de issue
Refactorización
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Tranquilo
Claridad
Bastante claro
Aptitud para principiantes
55/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.