node-fetch/node-fetch

RFC: `Content-Length` on Response should be taken into account?

Open

#879 aperta il 12 giu 2020

Vedi su GitHub
 (8 commenti) (1 reazione) (0 assegnatari)JavaScript (1061 fork)batch import
Node-specificenhancementgood first issue

Metriche repository

Star
 (8563 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

node-fetch currently completely ignoring Content-Length header while consuming response. Fetch specification about handling Content-Length on server response says almost nothing: https://fetch.spec.whatwg.org/#concept-http-network-fetch (see https://github.com/whatwg/fetch/issues/67)

On other hand, we have a fetch-node specific extension to limit the size of the response.

My proposal - analyze and throw early when size for response is specified and Content-Length of response is greater than it.

Guida contributor

Direzione di ricerca
Inizia leggendo il codice sorgente di node fetch, concentrandoti sulla gestione del corpo della risposta e sull'opzione 'size' (probabilmente in src/body.js). Rivedi la specifica fetch (https://fetch.spec.whatwg.org/#concept http network fetch) e l'issue collegato (https://github.com/whatwg/fetch/issues/67). Quindi implementa un controllo che confronta l'intestazione Content Length con il limite di dimensione specificato e genera un errore in anticipo se superato. Controlla i commenti esistenti per eventuali discussioni sui dettagli di implementazione.
Tech stack
javascript
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
JavaScriptNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60