envoyproxy/envoy

Use StringMatcher for route path and header matching

Ouverte

#8 392 ouverte le 26 sept. 2019

Voir sur GitHubGuide contributeur
 (7 commentaires) (0 réaction) (0 personne assignée)C++ (5 373 forks)batch import
api/v3help wanted

Métriques du dépôt

Stars
 (27 997 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

As per #next-major-version comments in https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/route/route.proto. We should ideally do this in v2 with new fields and then deprecate the existing matchers, allowing for auto-deletion in v3.

Guide contributeur

Direction de recherche
Examinez le fichier route.proto mentionné dans le problème, identifiez les correspondances de chaînes regex existantes et ajoutez de nouveaux champs StringMatcher pour la correspondance de chemin et d'en tête. Dépréciez les anciens champs. Assurez la compatibilité avec v2 et planifiez la suppression dans v3. Testez les modifications.
Stack technique
cpp
Domaine
backendnetworking
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.
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++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65