valyala/fasthttp

Request should not fail on small ReadBufferSize

Geschlossen

#1.873 geöffnet am 24.09.2024

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (1.755 Forks)batch import
help wantedwontfix

Repository-Metriken

Stars
 (21.741 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 3T) (44 gemergte PRs in 30 T)

Beschreibung

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?

Contributor Guide

Research-Richtung
Lesen Sie zunächst die fasthttp Datei header.go, insbesondere die tryRead Funktion in Zeile 2128 und die umgebende Pufferlogik. Verstehen Sie, wie ReadBufferSize die Anforderungsverarbeitung beeinflusst. Das Problem verweist auf einen bereits behobenen Go Bug (golang/go#14121); untersuchen Sie, ob diese Lösung auf fasthttp angewendet werden kann. Ziehen Sie außerdem die Implementierung eines Callback Mechanismus in Betracht, um Benutzer zu informieren, wenn die Puffergröße niedrig ist, damit sie Anpassungen vornehmen können. Überprüfen Sie vorhandene Kommentare auf Feedback der Maintainer oder zusätzlichen Kontext.
Tech Stack
gorest api
Domain
backendperformance
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
Understanding of GoFamiliarity with fasthttp's bufferingReading header.go file
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30