grpc-ecosystem/grpc-gateway

Support using FieldMasks with GET requests

Offen

#698 geöffnet am 10.07.2018

Auf GitHub ansehenContributor Guide
 (10 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (2.250 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (16.971 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Some GET requests (e. g. search) benefit greatly from being able to distinguish between default (e. g. 0) and non-defined values, since searching by "name='John' && age = 0" is not the same as "name='John' && age = undefined". Since FieldMasks are a generally recommended approach for optional fields in gRPC, it would be super helpful if grpc-gateway would support that.

Contributor Guide

Research-Richtung
Untersuche, wie FieldMasks derzeit im grpc gateway für andere HTTP Methoden (z. B. PATCH) behandelt werden, und implementiere die gleiche Parsing und Anwendungslogik für GET Anfragen. Sieh dir den vorhandenen Feldmasken Code im Runtime Paket an und verstehe, wie Feldmasken auf Abfrageparameter angewendet werden.
Tech Stack
go
Domain
backendapi
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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.
Klar
Voraussetzungen
GogRPC
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60