apache/apisix

bug: grpc-transcode cant transcode empty array list to [] , but it did to {}

Geschlossen

#11.440 geöffnet am 26.07.2024

Auf GitHub ansehenContributor Guide
 (13 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Lua (2.860 Forks)batch import
buggood first issue

Repository-Metriken

Stars
 (16.597 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 8h) (109 gemergte PRs in 30 T)

Beschreibung

Current Behavior

when back to front server, grpc transcode plugin cant transcode empty list (array) to [] , but to {} . use grpc example demo : image

when list has content , it did well . image

Expected Behavior

response empty list expect to get [] . not {}

Error Logs

No response

Steps to Reproduce

  1. apisix 3.9 latest
  2. grpc server golang
  3. postman

Environment

  1. apisix 3.9 latest
  2. grpc server golang
  3. postman

Contributor Guide

Research-Richtung
Untersuchen Sie das gRPC Transkodierungs Plugin in Apache APISIX (wahrscheinlich in der Datei 'plugins/grpc transcode.lua'). Das Problem ist, dass wenn eine gRPC Antwort ein leeres wiederholtes Feld enthält, es als JSON Objekt '{}' anstelle eines leeren Arrays '[]' serialisiert wird. Überprüfen Sie die Protobuf zu JSON Konvertierungslogik, insbesondere die Kodierung wiederholter Felder, und beziehen Sie sich auf die gRPC Transkodierungsspezifikation, um die Behandlung leerer Arrays zu korrigieren.
Tech Stack
luago
Domain
backendapi
Issue Type
Fehler
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
gRPCLua
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70