envoyproxy/envoy

Make wrapped / native scalar types consistent in v4 API

Open

#8128 aperta il 3 set 2019

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
api/v4help wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema richiede di rendere coerenti i tipi scalari avvolti (ad es. google.protobuf.BoolValue) e nativi nell'API v3. Rivedere i commenti del problema su https://github.com/envoyproxy/envoy/issues/8128 per eventuali discussioni sulle policy. Cercare nel codice sorgente di Envoy (specialmente file .proto) tutti gli usi dei tipi scalari avvolti e confrontarli con le controparti native. Proporre una policy coerente e identificare i file che necessitano di modifiche. Il problema è ampio e richiede una profonda comprensione di Protobuf e del design delle API di Envoy.
Tech stack
cpp
Dominio
apibackend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Richiede indagine
Prerequisiti
GitC++Protobuf
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45