envoyproxy/envoy

Investigate is it okay to kill the connection when the path includes `#`

Open

#10.033 geöffnet am 12. Feb. 2020

Auf GitHub ansehen
 (0 Kommentare) (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

Issue Template

Title: Investigate is it okay to kill the connection when the path includes #

Description: Quote from the comment in https://github.com/envoyproxy/envoy/pull/10010#discussion_r378329481:

Note: # should be never sent on the wire (at least in theory), so we might want to kill such connections... but this would require investigation to make sure that this wouldn't kill some popular but broken clients.

Contributor Guide

Research-Richtung
Überprüfen Sie die Diskussion in PR #10010, insbesondere den Kommentar zu '#'. Überprüfen Sie, ob einige Clients Pfade mit '#' senden, und bewerten Sie die Auswirkungen, wenn solche Verbindungen getrennt werden. Sehen Sie sich den vorhandenen Verbindungshandling Code in Envoy an, um zu bestimmen, wo die Prüfung hinzugefügt werden soll.
Tech Stack
cpp
Domain
backendsecurity
Issue Type
Recherche
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.
Meist klar
Voraussetzungen
C++HTTPproxy internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30