envoyproxy/envoy

Support for Request MetadataKind in InternalUpstreamTransport

Open

#30.674 aperta il 2 nov 2023

Vedi su GitHub
 (5 commenti) (1 reazione) (0 assegnatari)C++ (5373 fork)batch import
area/listenerenhancementhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

Title: Support for Request MetadataKind in InternalUpstreamTransport

Description:

I have a usecase where I am extracting an http header in a Lua filter, setting it in dynamicMetadata and passing it to an internal listener which has a TCPProxy filter configured with tunneling_config. The value that is set in dynamicMetadata needs to be referenced in the tunneling_config.

It seems there is support for passthrough_metadata for passing information from one listener to the other listener in this case. However, even though the docs seem to suggest that request metadataKind is supported ( which is needed for passing DynamicMetadata ), it seems the internal implementation is missing here and here

Is my understanding correct ? Happy to work on a PR if I can get any pointers!

[optional Relevant Links:]

Any extra documentation required to understand the issue.

Guida contributor

Direzione di ricerca
Esamina il file config.cc di InternalUpstreamTransport e aggiungi il supporto per il metadata kind Request estendendo il ciclo esistente per gestirlo in modo simile al metadata kind Connection.
Tech stack
cpplua
Dominio
backendinfrastructurenetworking
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40