envoyproxy/envoy

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

Open

#9525 aperta il 30 dic 2019

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

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la definizione protobuf di core.HeaderValueOption e la logica di aggiunta delle intestazioni in source/common/router/header parser.cc. Aggiungi un campo bool skip if present con valore predefinito false, quindi implementa la logica nel percorso di aggiunta delle intestazioni di richiesta/risposta per saltare l'aggiunta se l'intestazione esiste già e skip if present è true.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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++ProtobufBazel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65