nodejs/node

Merge streams handling code for http2 streams & net.Socket

Open

#19.060 aperta il 28 feb 2018

Vedi su GitHub
 (28 commenti) (0 reazioni) (0 assegnatari)JavaScript (35.535 fork)batch import
help wantedhttp2net

Metriche repository

Star
 (117.218 star)
Metriche merge PR
 (Merge medio 18g 17h) (219 PR mergiate in 30 g)

Descrizione

Hey :)

It would be great if we could unify all the code from net and http2 that is only concerned with pushing data to/from the underlying stream, ideally into a common base class of net.Socket and Http2Stream, so that we could also maybe port some of the other native streams (zlib, fs) to using StreamBase on the native side & generally just share a lot of code.

This is probably not an easy task, and probably not doable in one pass, because the http2 and net implementations are always just slightly different, but if anybody wants to try to start working on this, feel free to reach out by commenting here.

Guida contributor

Direzione di ricerca
Studia le implementazioni esistenti di net.Socket e Http2Stream, identifica i pattern di codice comuni e proponi un progetto per una classe base comune sotto StreamBase.
Tech stack
javascriptcpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Oltre 1 settimana
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30