akka/akka-http

Restrict / review `Transfer-Encoding` header rendering

Open

#3761 aperta il 15 feb 2021

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Scala (598 fork)batch import
1 - triageddiscusshackathonhelp wantedt:coret:model

Metriche repository

Star
 (1311 star)
Metriche merge PR
 (Merge medio 1h 17m) (1 PR mergiata in 30 g)

Descrizione

Since #3754, we only support Transfer-Encoding: chunked headers and no other ones. The reasoning is that akka-http is implementing the "Transfer" part of HTTP. Leaving parsing or providing custom Transfer-Encoding headers open to the user will lead to confusion and bugs.

We should check that:

  • The Transfer-Encoding model could make sure that the latest entry is always chunked (see RFC), though that would change the error reporting for parsing as now implemented with #3754
  • User-provided Transfer-Encoding headers will always be ignored

Guida contributor

Direzione di ricerca
Analizza l'implementazione corrente della PR #3754 per comprendere come vengono analizzati e renderizzati gli header Transfer Encoding. L'obiettivo è limitare il modello in modo che sia consentito solo 'chunked' e ignorare gli header forniti dall'utente. Esamina il modello HttpHeader e il codice di rendering per aggiungere la validazione. Il problema menziona due controlli: assicurarsi che l'ultima voce sia sempre 'chunked' e ignorare gli header forniti dall'utente. Considera come integrarli senza interrompere la gestione degli errori esistente.
Tech stack
Nessuno
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
HTTP Transfer Encoding knowledgeakka http internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
25