envoyproxy/envoy

ext_proc: knob to disallow immediate_response

Open

#28.698 geöffnet am 28. Juli 2023

Auf GitHub ansehen
 (8 Kommentare) (0 Reaktionen) (2 zugewiesene Personen)C++ (5.373 Forks)batch import
area/ext_procenhancementhelp wanted

Repository-Metriken

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

Beschreibung

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.

Contributor Guide

Research-Richtung
Beginnen Sie mit dem Studium der vorhandenen Implementierung des Schalters `disallow clear route cache` in `source/extensions/filters/http/ext proc/config.cc` und `ext proc.proto`. Verstehen Sie, wie `immediate response` aktuell in der Filterlogik behandelt wird. Der neue Schalter sollte sich ähnlich verhalten: Fügen Sie dem Filterkonfigurations Proto ein boolesches Feld hinzu, prüfen Sie es im Antwortverarbeitungscode und schließen Sie den Stream, ohne die sofortige Antwort zu senden. Überprüfen Sie die relevanten Tests in `test/extensions/filters/http/ext proc/`, um korrektes Verhalten sicherzustellen.
Tech Stack
cpp
Domain
infrastructureapi
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.
Klar
Voraussetzungen
C++Envoy
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50