valyala/fasthttp

Request should not fail on small ReadBufferSize

Chiusa

#1873 aperta il 24 set 2024

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

Metriche repository

Star
 (21.741 stelle)
Metriche merge PR
 (Merge medio 3g) (44 PR mergiate in 30 g)

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

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.
Tech stack
gorest api
Dominio
backendperformance
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Understanding of GoFamiliarity with fasthttp's bufferingReading header.go file
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30