envoyproxy/envoy

Route actions prefix_rewrite and regex_rewrite should be a oneof

Open

#10 182 ouverte le 26 févr. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
api/v4help wantedtech debt

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: Route actions prefix_rewrite and regex_rewrite should be a oneof

Description:

In the discussion for pull request #10050, it was suggested that the prefix_rewrite and regex_rewrite route actions be put under a oneof in the next api version, v4. However, it's too early to mark the proto files with that annotation now, so the code change for #10050 didn't add the annotation. This issue is to remember that this should be considered when the time is appropriate.

Guide contributeur

Direction de recherche
Localisez le fichier de définition proto où prefix rewrite et regex rewrite sont définis et enveloppez les dans un bloc oneof pour l'API v4.
Stack technique
Aucun
Domaine
backend
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.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Protobuf
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80