envoyproxy/envoy

Make wrapped / native scalar types consistent in v4 API

Open

#8.128 geöffnet am 3. Sept. 2019

Auf GitHub ansehen
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
api/v4help wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Das Issue erfordert, dass verpackte (z. B. google.protobuf.BoolValue) und native skalare Typen in der v3 API konsistent gemacht werden. Überprüfen Sie die Issue Kommentare unter https://github.com/envoyproxy/envoy/issues/8128 auf etwaige Richtliniendiskussionen. Durchsuchen Sie den Envoy Quellcode (insbesondere .proto Dateien) nach allen Verwendungen verpackter skalarer Typen und vergleichen Sie diese mit den nativen Gegenstücken. Schlagen Sie eine konsistente Richtlinie vor und identifizieren Sie Dateien, die Änderungen benötigen. Das Issue ist umfangreich und erfordert ein tiefes Verständnis von Protobuf und dem API Design von Envoy.
Tech Stack
cpp
Domain
apibackend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Recherche nötig
Voraussetzungen
GitC++Protobuf
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
45