envoyproxy/envoy

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

Open

#13.100 geöffnet am 15. Sept. 2020

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/access_logenhancementhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Verstehen Sie den sendLocalReply Ablauf und wie aktuell Antwortflags gesetzt werden. Fügen Sie dann ein neues Flag 'RESPONDED BY FILTER' hinzu und verwerfen Sie das alte 'response from cache filter'. Verwenden Sie das Antwortcode Detail für zusätzliche Informationen.
Tech Stack
cpp
Domain
backendapi
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
EnvoyC++access logging
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40