envoyproxy/envoy

Use response_headers_to_add for Envoy-generated error responses

Open

#8 944 ouverte le 8 nov. 2019

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

On events like a request timeout, Envoy sends a locally generated response (the sendLocalReply function). But it doesn't use the response_headers_to_add config option. Which could be useful when response_headers_to_add has some diagnostic information (e.g. the %REQ(x-request-id)% which was added in #8683)

We can fix this by adding finalizeResponseHeaders calls before sendLocalReply calls, in functions like onUpstreamAbort.

Guide contributeur

Direction de recherche
Examinez la base de code d'Envoy pour trouver les fonctions qui appellent `sendLocalReply` pour générer des réponses d'erreur (par exemple, `onUpstreamAbort`). Étudiez la fonction `finalizeResponseHeaders` pour comprendre comment elle applique `response headers to add`. Ajoutez des appels à `finalizeResponseHeaders` avant `sendLocalReply` dans ces fonctions. Consultez la PR associée #8683 pour le contexte sur la façon dont `response headers to add` est utilisé pour les en têtes de requête.
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.
1-3 heures
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++Envoy
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60