envoyproxy/envoy

ext_proc: knob to disallow immediate_response

Open

#28.698 aperta il 28 lug 2023

Vedi su GitHub
 (8 commenti) (0 reazioni) (2 assegnatari)C++ (5373 fork)batch import
area/ext_procenhancementhelp wanted

Metriche repository

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

Descrizione

Title: ext_proc: knob to disallow immediate_response

Description: Just like the way we have a knob disallow_clear_route_cache that will ignore a clear_route_cache setting in the ProcessingResponse, we need a knob on the filter (disallow_immediate_response) that will ignore an immediate_response value in the ProcessingResponse. The behavior should be to close the ext_proc stream but NOT do the immediate response. Instead, the request should be forwarded as through no changes were done by ext_proc.

Knob is false by default.

Guida contributor

Direzione di ricerca
Inizia studiando l'implementazione esistente dell'interruttore `disallow clear route cache` in `source/extensions/filters/http/ext proc/config.cc` e `ext proc.proto`. Comprendi come `immediate response` è attualmente gestito nella logica del filtro. Il nuovo interruttore dovrebbe comportarsi in modo simile: aggiungi un campo booleano al proto di configurazione del filtro, controllalo nel codice di elaborazione della risposta e chiudi lo stream senza inviare la risposta immediata. Rivedi i test pertinenti in `test/extensions/filters/http/ext proc/` per garantire il comportamento corretto.
Tech stack
cpp
Dominio
infrastructureapi
Tipo issue
Funzionalità
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
C++Envoy
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50