Skip to content

Share HTTP client construction between gRPC calls and reflection#588

Merged
ryanfowler merged 1 commit into
mainfrom
fix-grpc-reflection-client-parity
May 26, 2026
Merged

Share HTTP client construction between gRPC calls and reflection#588
ryanfowler merged 1 commit into
mainfrom
fix-grpc-reflection-client-parity

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Extracted reqwest client setup into a shared builder used by both normal HTTP execution and gRPC reflection/discovery
  • Restored parity for reflection with request-path options: --min-tls, --max-tls, proxy handling, custom DNS, sessions, and timing connector support
  • Kept reflection on HTTP/2 prior knowledge while preserving request-mode-specific protocol selection for normal traffic
  • Added integration coverage for reflection TLS bounds, proxy rejection/allowance, and --dns-server

Testing

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

@ryanfowler ryanfowler force-pushed the fix-grpc-reflection-client-parity branch from a093cd4 to f5058fd Compare May 26, 2026 00:57
@ryanfowler ryanfowler merged commit 420a5d4 into main May 26, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-grpc-reflection-client-parity branch May 26, 2026 01:03
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