envoyproxy/envoy

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

Open

#9 525 ouverte le 30 déc. 2019

Voir sur GitHub
 (5 commentaires) (1 réaction) (0 assignés)C++ (5 373 forks)batch import
area/httphelp wanted

Métriques du dépôt

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

Description

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.

Guide contributeur

Direction de recherche
Examinez la définition protobuf de core.HeaderValueOption et la logique d'ajout d'en têtes dans source/common/router/header parser.cc. Ajoutez un champ booléen skip if present avec une valeur par défaut false, puis implémentez la logique dans le chemin d'ajout d'en têtes de requête/réponse pour ignorer l'ajout si l'en tête existe déjà et que skip if present est true.
Stack technique
cpp
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
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
C++ProtobufBazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65