akka/akka-http

Restrict / review `Transfer-Encoding` header rendering

Aperta

#3761 aperta il 15 feb 2021

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

Metriche repository

Star
 (1311 stelle)
Metriche merge PR
 (Merge medio 1g 10h) (2 PR mergiate 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
Esaminare la gestione corrente di Transfer Encoding in akka http, in particolare il modello e la logica di parsing. Determinare come imporre che solo 'chunked' sia l'ultima codifica e ignorare le intestazioni Transfer Encoding fornite dall'utente.
Tech stack
scala
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.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ScalaHTTPAkka
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40