envoyproxy/envoy

Envoy should register invalid HTTP as an attempt to send a request

Open

#9 821 ouverte le 24 janv. 2020

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/httphelp 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

Title: Envoy should register invalid HTTP as an attempt to send a request

Description: If a malformed HTTP1 request is sent (e.g. No headers, just "a\r\n\r\n") the HTTP1.1 code will reject it (https://github.com/envoyproxy/envoy/blob/master/source/common/http/http1/codec_impl.cc#L497) and send a 400 without creating a ConnectionManagerImpl::ActiveStream. This means that the stats around total requests (downstream_rq_total_ & similar) will not be modified, as they are modified by ConnectionManagerImpl::ActiveStream. It seems like an invalid request should still be counted in those statistics.

Guide contributeur

Direction de recherche
Le problème est que les requêtes HTTP/1.1 invalides (par exemple, des en têtes malformés) sont rejetées dans source/common/http/http1/codec impl.cc vers la ligne 497 sans incrémenter downstream rq total et les statistiques associées. La correction doit impliquer l'ajout d'un incrément de statistiques à ce moment là, avant d'envoyer la réponse 400. Vérifiez la classe ConnectionManagerImpl::ActiveStream pour comprendre comment les statistiques sont normalement incrémentées pour les requêtes valides.
Stack technique
cpp
Domaine
backendobservability
Type d'issue
Bug
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++HTTP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55