envoyproxy/envoy

Add support of additional %UPSTREAM% variables to custom headers

Open

#13.897 geöffnet am 4. Nov. 2020

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

Repository-Metriken

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

Beschreibung

One of our customers is using the Envoy tap filter to retrieve transport information and add them to their request/response headers, as documented here : https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_conn_man/headers.html#custom-request-response-headers

At this point, very few %UPSTREAM% variable names are supported : %UPSTREAM_METADATA([“namespace”, “key”, …])% %UPSTREAM_REMOTE_ADDRESS%

They wish additional %UPSTREAM% variables were supported, such as %UPSTREAM_LOCAL_ADDRESS%.

Thanks !

Contributor Guide

Research-Richtung
Untersuchen Sie den vorhandenen Code zur Variablenerweiterung in Envoy (wahrscheinlich in source/common/http/header formatter.cc oder ähnlich), um zu verstehen, wie die aktuellen %UPSTREAM ...% Variablen behandelt werden, und fügen Sie dann Unterstützung für neue Variablen wie %UPSTREAM LOCAL ADDRESS% nach dem gleichen Muster hinzu.
Tech Stack
Keine
Domain
backendinfrastructure
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65