envoyproxy/envoy

access log: add a new response flag to indicate request handled locally by Envoy

Open

#13 100 ouverte le 15 sept. 2020

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/access_logenhancementhelp 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: access log: add a new response flag to indicate request handled locally by Envoy

Description: This is started from the discussion in https://github.com/envoyproxy/envoy/pull/12912#discussion_r490613876 that we could have a more general way to indicate that the request is handled locally by Envoy (sendLocalReply) instead of by upstream.

The response flag could be RESPONDED_BY_FILTER and deprecates the other response_from_cache_filter. The RESPONSE_CODE_DETAIL can be used to learn more details.

cc @htuch

[optional Relevant Links:]

Any extra documentation required to understand the issue.

Guide contributeur

Direction de recherche
Comprenez le flux de sendLocalReply et comment les indicateurs de réponse sont actuellement définis. Ajoutez ensuite un nouvel indicateur 'RESPONDED BY FILTER' et dépréciez l'ancien 'response from cache filter'. Utilisez le détail du code de réponse pour plus d'informations.
Stack technique
cpp
Domaine
backendapi
Type d'issue
Fonctionnalité
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-2 jours
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
EnvoyC++access logging
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40