envoyproxy/envoy

hcm: Allow filter to control on emitting trailers (e.g. to skip encoding trailers when empty)

Open

#12.374 geöffnet am 30. Juli 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/httpenhancementhelp wanted

Repository-Metriken

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

Beschreibung

Context: Some of the filters, e.g. gRPC Web requires to skip encoding trailers due to https://github.com/envoyproxy/envoy/issues/10514.

i.e. currently, we have the following "problematic" response frames.

[  5.036] recv DATA frame <length=28, flags=0x00, stream_id=13>
[  5.040] recv DATA frame <length=52, flags=0x00, stream_id=13>
[  5.053] recv HEADERS frame <length=0, flags=0x05, stream_id=13>
          ; END_STREAM | END_HEADERS

What we want is something like:

[  5.036] recv DATA frame <length=28, flags=0x00, stream_id=13>
[  5.040] recv DATA frame <length=52, flags=0x00, stream_id=13>
          ; END_STREAM

We have the current data frames since we always encode trailers. The proposed solution will be to have control from the filter side.

A couple of options:

  1. introduce a new return enum to FilterTrailersStatus, to indicate that trailers should not be emitted.
  2. add end_stream flags to addEncodedData, end stream there and encodeTrailers return StopIteration

From the discussion here, the second of the options above is preferable and when doing that we should do a parallel fix in addDecodedData too.

Contributor Guide

Research-Richtung
Überprüfe die zugehörige PR Diskussion (#12178), um den bevorzugten Ansatz zu verstehen (Option 2: end stream Flags zu addEncodedData hinzufügen). Implementiere dann die Änderungen im Filter Manager, um Filtern zu ermöglichen, zu steuern, ob Trailer gesendet werden, und stelle Konsistenz für codierte und decodierte Datenpfade sicher.
Tech Stack
c
Domain
backendinfrastructure
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
4
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.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Envoy filter pipeline
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
20