envoyproxy/envoy

upgrade http1.0 to http1.1 in response

Open

#6 537 ouverte le 10 avr. 2019

Voir sur GitHub
 (7 commentaires) (0 réactions) (1 assigné)C++ (5 373 forks)batch import
area/httpenhancementhelp 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: HTTP 1.0 / 1.1 : implementation of RFC 2145

Description: According to nginx behaviour and RFC 2145. See this link for a TLDR. One receiving an http1.0 request should reply with either a 1.0 or a 1.1 response but the 1.1 would be prefered.

By creating an HTTP connection manager and allowing the http1.0 and 0.9 protocols as specified in the documentation. If I do a request using http1.0 protocol, a response using the 1.1 protocol should be prefered. Nevertheless I am getting an http1.0 response.

Is this behaviour normal and am i not understanding the RFC correctly or is this a bug ?

Guide contributeur

Direction de recherche
Étudiez le code du gestionnaire de connexions HTTP d'Envoy pour comprendre comment la version HTTP de la réponse est déterminée. Vérifiez le fichier source `source/common/http/conn manager impl.cc` pour le traitement de la version de réponse. Examinez le RFC 2145 et le comportement décrit dans le lien serverfault. Consultez les éventuels commentaires des mainteneurs dans l'issue pour obtenir des conseils sur le fait qu'il s'agisse d'un bug ou d'un comportement intentionnel.
Stack technique
cpp
Domaine
backendinfrastructure
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.
Plutôt claire
Prérequis
C++HTTP protocol knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50