envoyproxy/envoy

Router: Audit ASSERTs

Open

#14.961 geöffnet am 5. Feb. 2021

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/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 router filter and recommendations to change some: https://gist.github.com/asraa/859a41a8f5f252d35da58c9ef7502230

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

Contributor Guide

Research-Richtung
Überprüfe den Gist unter https://gist.github.com/asraa/859a41a8f5f252d35da58c9ef7502230, der das ASSERT Audit beschreibt. Die Hauptdatei, die geändert werden muss, ist source/common/router/router.cc, mit den spezifischen Zeilen: 276, 566 und 653. Die Aufgabe umfasst das Hinzufügen einer Prüfung für Scheme Header während der Filterverarbeitung und das Auditieren anderer ASSERTs, wie markiert. Untersuche den Code um diese Zeilen, um die aktuelle Assert Logik zu verstehen, und wende die empfohlenen Änderungen an. Es gibt keine verknüpften PRs, daher ist die Implementierung unabhängig.
Tech Stack
cpp
Domain
backendinfrastructure
Issue Type
Refactoring
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-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.
Klar
Voraussetzungen
C++Envoy internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40