envoyproxy/envoy

HTTP/1.1 codec: Audit ASSERTs

Open

#14.959 geöffnet am 5. Feb. 2021

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/httparea/securityhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

This issue describes a scan of ASSERTs in the HTTP/1.1 codec and recommendations to change some: https://gist.github.com/asraa/9d5d60b0af53dc88ce39d8cfd71ef12a

Those marked with a verdict have been audited or unknown if they need a pair of eyes.

Contributor Guide

Research-Richtung
Überprüfe die Audit Ergebnisse unter https://gist.github.com/asraa/9d5d60b0af53dc88ce39d8cfd71ef12a. Implementiere dann Änderungen für die beiden auslösbaren Bedingungen in den Zeilen 341 und 381 in source/common/http/http1/codec impl.cc. Analysiere für die als UNKNOWN markierten Zeilen (141, 804, 1148) die Assertions, um festzustellen, ob sie durch nicht vertrauenswürdige Eingaben ausgelöst werden können; falls ja, wandle sie in eine geeignete Fehlerbehandlung um. Die Maintainer haben in den Kommentaren keine weiteren Klarstellungen gegeben.
Tech Stack
cpp
Domain
backend
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.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Envoy development setup
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40