envoyproxy/envoy

feature request: `skip_if_present` for request/response headers to add

Open

#9.525 geöffnet am 30. Dez. 2019

Auf GitHub ansehen
 (5 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/httphelp wanted

Repository-Metriken

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

Beschreibung

Title: skip_if_present for request/response headers to add

Description: When using request/response_headers_to_add, the only options are either replace the header or append to it. Sometimes it may be preferable to simply ignore if the header is already set. This would add a boolean skip_if_present option to core.HeaderValueOption that defaults to false.

Contributor Guide

Research-Richtung
Untersuchen Sie die Protobuf Definition von core.HeaderValueOption und die Header Additionslogik in source/common/router/header parser.cc. Fügen Sie ein Bool Feld skip if present mit Standardwert false hinzu und implementieren Sie dann die Logik im Request/Response Header Additionspfad, um das Hinzufügen zu überspringen, wenn der Header bereits existiert und skip if present true ist.
Tech Stack
cpp
Domain
backend
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.
Halber Tag
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++ProtobufBazel
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65