envoyproxy/envoy

HTTP/2 codec: Audit ASSERTs

Open

#14.960 geöffnet am 5. Feb. 2021

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (1 zugewiesene Person)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/2 codec and recommendations to change some: https://gist.github.com/asraa/eb8b90a6398d202dd584cf403e3c6f3a

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

Contributor Guide

Research-Richtung
Überprüfen Sie den im Issue verlinkten Gist auf Audit Empfehlungen. Untersuchen Sie die angegebenen ASSERT Zeilen in source/common/http/http2/codec impl.cc (Zeilen 223, 387, 391, 750, 773, 1085). Stellen Sie fest, ob jeder ASSERT durch externe Eingaben ausgelöst werden kann, und entscheiden Sie, ob er in einen robusteren Fehlerbehandlungsmechanismus geändert werden sollte (z. B. Fehler zurückgeben oder ENVOY LOG). Das Issue hat keine Kommentare oder PRs; prüfen Sie den Status des Zugewiesenen und den Gist auf etwaige Aktualisierungen.
Tech Stack
cpp
Domain
backend
Issue Type
Recherche
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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.
Meist klar
Voraussetzungen
C++HTTP/2
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30