Skip to content

fix(grpc-web): missing trailers when empty resp body#12490

Merged
SkyeYoung merged 26 commits into
apache:masterfrom
bzp2010:bzp/fix-grpc-web-missing-trailers
Aug 12, 2025
Merged

fix(grpc-web): missing trailers when empty resp body#12490
SkyeYoung merged 26 commits into
apache:masterfrom
bzp2010:bzp/fix-grpc-web-missing-trailers

Conversation

@bzp2010

@bzp2010 bzp2010 commented Aug 5, 2025

Copy link
Copy Markdown
Contributor

Description

Fixes #12202

The problem that this PR solves is fully described in the comments of the code.

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@bzp2010 bzp2010 changed the title Bzp/fix grpc web missing trailers fix(grpc-web): missing trailers when empty resp body Aug 5, 2025
@bzp2010
bzp2010 marked this pull request as ready for review August 5, 2025 12:59
@dosubot dosubot Bot added size:XXL This PR changes 1000+ lines, ignoring generated files. bug Something isn't working labels Aug 5, 2025
Comment thread apisix/plugins/grpc-web.lua Outdated
@bzp2010
bzp2010 force-pushed the bzp/fix-grpc-web-missing-trailers branch from 237c47a to 1a5a156 Compare August 5, 2025 16:43
@SkyeYoung

Copy link
Copy Markdown
Member

Need to fix the CI.

@Revolyssup

Copy link
Copy Markdown
Contributor

waiting on CI pass for review

@SkyeYoung SkyeYoung left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the detailed comments, LGTM

@SkyeYoung
SkyeYoung merged commit 1b0bb84 into apache:master Aug 12, 2025
23 checks passed
jizhuozhi pushed a commit to jizhuozhi/apisix that referenced this pull request Oct 18, 2025
shreemaan-abhishek pushed a commit to shreemaan-abhishek/apisix that referenced this pull request Jan 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: grpc-web unexpected error "upstream grpc status not received"

4 participants