envoyproxy/envoy

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

Open

#10 033 ouverte le 12 févr. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/httparea/securityhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez la discussion dans le PR #10010, en particulier le commentaire sur '#'. Verifiez si certains clients envoient des chemins avec '#' et evaluez l'impact de la fermeture de ces connexions. Regardez le code de gestion des connexions existant dans Envoy pour determiner ou ajouter la verification.
Stack technique
cpp
Domaine
backendsecurity
Type d'issue
Recherche
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++HTTPproxy internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30