envoyproxy/envoy

cleanup: add proto definition for test extensions

Fermée

#20 476 ouverte le 23 mars 2022

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)C++ (5 373 forks)batch import
area/protobufbeginnerbughelp wantedtech debt

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

Currently, lookup-by-name for extensions is used for internal extensions. This is inconsistent with actual extensions which exclusively use lookup-by-type-URL. The cleanup is to add protos for all internal extensions, and remove the runtime guard usage in the tests.

Guide contributeur

Direction de recherche
Investigate the existing internal extensions in 'source/extensions/' directory. Examine proto definitions for external extensions (e.g., 'api/envoy/extensions/') to understand the pattern. Find all uses of lookup by name and runtime guards in test files. The goal is to add proto definitions for each internal extension and refactor tests to use lookup by type URL. Refer to any comments or linked PRs in issue #20476 for additional context.
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.
3-5 jours
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
C++ knowledgeProtocol BuffersEnvoy internalsTesting experience
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30