fix(grpc-transcode): correctly serialize empty repeated fields to [] #12391
+51
−705
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: Traky Deng <[email protected]>
…tion of each phase (apache#11793)
Co-authored-by: Ashish Tiwari <[email protected]>
Signed-off-by: xuruidong <[email protected]>
…t. (apache#11783) Co-authored-by: root <[email protected]>
* docs: improve real-ip plugin docs * update Chinese version * fix comments & lint * Update real-ip.md * Update real-ip.md * add notes back * fix typo
|
Hi @bytelazy, could you please merge the latest master branch code? |
membphis
previously approved these changes
Sep 10, 2025
sure, I have merged the latest master branch |
|
Hi @bytelazy, some failed tests need to be fixed. |
Got it. I'll fix the failed tests soon. |
dosubot
bot
added
size:XL
dosubot
bot
added
size:M
bytelazy
force-pushed
the
master
branch
2 times, most recently
from
11e3559 to
a0356cd
Compare
github-project-automation
bot
moved this from 👀 In review
to ✅ Done
in ⚡️ Apache APISIX Roadmap
dosubot
bot
added
size:XL
|
hi @Baoyuantop Sorry for the inconvenience, and please continue the review there 🙏 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
36 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
In the grpc-transcode plugin, when a repeated field in the gRPC response is empty, it is incorrectly encoded as an empty JSON object ({}) instead of an empty JSON array ([]). This behavior does not comply with the gRPC-JSON transcoding specification and can cause type-related errors on the client-side, which expects an array.
Fixes #11440
Checklist
Changes or the new features added to this PR
The main changes are in the response.lua file to correctly handle the serialization of repeated fields.
Identify repeated fields: A new function fetch_proto_array_names has been added. It recursively traverses the loaded Protobuf schema definition to identify and collect the names of all fields marked as repeated.
Ensure correct JSON array serialization:
A new function set_default_array has been introduced. After the gRPC response is decoded into a Lua table, this function traverses the table.
For any field that was identified as repeated, it sets a specific metatable (core.json.array_mt).
This metatable forces the core.json.encode function to serialize the corresponding Lua table as a JSON array ([]), even when it is empty.
This logic is applied just before the decoded response is encoded into the final JSON output, ensuring the structure is correct.