Skip to content

Enable gRPC reflection for schema-less response decoding#606

Merged
ryanfowler merged 1 commit into
mainfrom
fix-grpc-reflection-for-responses
May 26, 2026
Merged

Enable gRPC reflection for schema-less response decoding#606
ryanfowler merged 1 commit into
mainfrom
fix-grpc-reflection-for-responses

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Attempt gRPC reflection whenever --grpc is used without a local method descriptor, instead of only when the request body needs a schema.
  • Keep raw-call compatibility by ignoring reflection failures unless the request body actually requires schema-driven JSON-to-protobuf conversion.
  • Add an integration test covering a schema-less gRPC call that now decodes reflected response output.

Testing

  • cargo fmt
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --all-features --test integration grpc_reflection_h2c_go_cases -- --test-threads=1
  • cargo test --all-features

@ryanfowler ryanfowler enabled auto-merge May 26, 2026 22:57
@ryanfowler ryanfowler merged commit b6d38c1 into main May 26, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-grpc-reflection-for-responses branch May 26, 2026 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant