valyala/fasthttp

Request should not fail on small ReadBufferSize

Open

Aperta il 24 set 2024

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Go (21.741 star) (1755 fork)batch import
help wantedwontfix

Descrizione

Can we not fail request if the req/resp header size is greater than the read buffer size.

I can see a bufio bug was mentioned for tryRead function but looks like it's resolved now. Can we have the above mentioned behaviour?

Also, some callback could also be configured to inform that buffer size was low. Library users could emit metrics and increase the buffer size appropriately to have best balance between memory usage and CPU usage.

@valyala Could you review the request?

Guida contributor

Tech stack
gorest api
Dominio
backendperformance
Tipo issue
feature
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
mostly clear
Prerequisiti
Understanding of GoFamiliarity with fasthttp's bufferingReading header.go file
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30
Direzione di ricerca
Per prima cosa, leggi il file header.go di fasthttp, in particolare la funzione tryRead alla riga 2128 e la logica del buffer circostante. Comprendi come ReadBufferSize influisce sul parsing delle richieste. Il problema fa riferimento a un bug di Go (golang/go#14121) già risolto; indaga se tale risoluzione può essere applicata a fasthttp. Inoltre, considera l'implementazione di un meccanismo di callback per informare gli utenti quando la dimensione del buffer è bassa, consentendo loro di adattarsi. Controlla i commenti esistenti per eventuali feedback dei manutentori o contesto aggiuntivo.