envoyproxy/envoy

Api: Move geoip_headers_to_add from provider level to geoip filter

Open

#31.001 aperta il 21 nov 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/geoiparea/http_filterenhancementhelp wanted

Metriche repository

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

Descrizione

Title: Move geoip_headers_to_add from provider level to geoip filter

Description: Currently geoip_headers_to_add are configured in geoip provider. From logical standpoint it makes more sense to have them in geoip filter config instead, as this is not property of provider (from which source the data is being looked up) but filter (what kind of data to get from provider).

Relevant Links Discussion in past PR

Guida contributor

Direzione di ricerca
Identifica la posizione attuale di geoip headers to add nel proto del provider e nella configurazione del filtro. Sposta la definizione del campo nel proto del filtro, aggiorna l'implementazione del filtro per leggere dalla nuova posizione e assicura la retrocompatibilità o un piano di migrazione.
Tech stack
cpp
Dominio
backendinfrastructure
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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
GitC++Protobuf
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30