envoyproxy/envoy

Api: Move geoip_headers_to_add from provider level to geoip filter

Ouverte

#31 001 ouverte le 21 nov. 2023

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C++ (5 373 forks)batch import
area/geoiparea/http_filterenhancementhelp wanted

Métriques du dépôt

Stars
 (27 997 étoiles)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Identifiez l'emplacement actuel de geoip headers to add dans le proto du fournisseur et dans la configuration du filtre. Déplacez la définition du champ dans le proto du filtre, mettez à jour l'implémentation du filtre pour lire depuis le nouvel emplacement et assurez la rétrocompatibilité ou un plan de migration.
Stack technique
cpp
Domaine
backendinfrastructure
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++Protobuf
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30