valyala/fasthttp

It is not safe to read all stream body to memory without a max size limit.

Open

#1.765 geöffnet am 23. Apr. 2024

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

Repository-Metriken

Stars
 (21.741 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 11h) (34 gemergte PRs in 30 T)

Beschreibung

https://github.com/valyala/fasthttp/blob/57b9352ad1cc93a0aaaa72b2130e03ace8a5b118/http.go#L427 I think it would be safe to stop reading the request body into memory and return an error when it exceeds the maximum request body size. Otherwise, it may lead to an out-of-memory (OOM) error when the request body is too large.

Contributor Guide

Research-Richtung
Das Problem verweist auf Zeile 427 in http.go, wo der gesamte Anfragekörper ohne Größenbeschränkung in den Speicher gelesen wird. Zur Behebung sollte vor dem Lesen eine Prüfung auf eine maximale Anfragekörpergröße hinzugefügt werden, vorzugsweise mit einem konfigurierbaren Limit (z. B. MaxRequestBodySize). Wenn es überschritten wird, sollte das Lesen stoppen und einen Fehler zurückgeben, wahrscheinlich mit einem 413 Statuscode. Stellen Sie die Integration in die bestehende fasthttp Anfrageverarbeitungspipeline sicher und berücksichtigen Sie vorhandene Größen Einstellungen.
Tech Stack
go
Domain
backendsecurity
Issue Type
Fehler
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.
Klar
Voraussetzungen
Go basicsHTTP server internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45