envoyproxy/envoy

Make wrapped / native scalar types consistent in v4 API

Open

#8 128 ouverte le 3 sept. 2019

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
api/v4help 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

In v2, we started of by wrapping everything, but then settled on a policy of only wrapping scalar fields that have non-proto3 defaults. There is a legacy of this across the API that should be cleaned up to be more consistent, v3 is such an opportunity.

Guide contributeur

Direction de recherche
Le problème nécessite de rendre cohérents les types scalaires encapsulés (par exemple google.protobuf.BoolValue) et natifs dans l'API v3. Consultez les commentaires du problème sur https://github.com/envoyproxy/envoy/issues/8128 pour toute discussion sur les politiques. Recherchez dans le code source d'Envoy (en particulier les fichiers .proto) toutes les utilisations de types scalaires encapsulés et comparez les avec leurs équivalents natifs. Proposez une politique cohérente et identifiez les fichiers nécessitant des modifications. Le problème est vaste et nécessite une compréhension approfondie de Protobuf et de la conception de l'API d'Envoy.
Stack technique
cpp
Domaine
apibackend
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.
1-2 jours
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.
Investigation requise
Prérequis
GitC++Protobuf
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45