envoyproxy/envoy

Path normalization controls

Offen

#6.589 geöffnet am 15.04.2019

Auf GitHub ansehenContributor Guide
 (17 Kommentare) (1 Reaktion) (2 zugewiesene Personen)C++ (5.373 Forks)batch import
area/securityhelp wantedpriority/hightech debt

Repository-Metriken

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

Beschreibung

The fix for CVE-2019-9900, provided a coarse grained ability to opt-in/out of path normalization. That is, you could either normalize, in which case both normalization was used for matching and transforming the path to the upstream, or not normalize.

Ideally we provide finer grained controls similar to Nginx, where users can opt to normalize for match independent of transforming the path to the upstream.

CC @PiotrSikora

Action item for CVE-2019-9901

Contributor Guide

Research-Richtung
Untersuche die aktuelle Pfadnormalisierungsimplementierung in Envoy (source/common/http/path utility.h/cc). Verstehe die CVE 2019-9900 Korrektur und die gewünschte Nginx ähnliche Steuerung. Schlage ein Design vor, das die Normalisierungsoptionen für Matching und Transformation trennt. Implementiere und teste die Änderungen.
Tech Stack
cpp
Domain
backendnetworkingsecurity
Issue Type
Funktion
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.
Klar
Voraussetzungen
C++Envoy proxy internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30