Open
Conversation
shawnburke
force-pushed
the
grpc-tunnel
branch
2 times, most recently
from
81bda7d to
63142b6
Compare
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… limits - C1: Add mutex-protected sendFunc for concurrent gRPC stream sends (client) - C2: Add 100MB request body size limit with HTTP 413 (server) - C3: Sort streams by ID for deterministic round-robin dispatch (server) - I1: Add heartbeat timeout detection via atomic + monitor goroutine (server) - I2: Wire FailStream on stream close to fail pending dispatch requests (server) - I3: Add requestAssembler for chunked request reassembly with unit tests (client) - I4: Check and log all send errors (client) - I5: Send ServerHello before Register to complete handshake first (server) - I6: Call ServerStopping before GracefulStop for clean shutdown (server) - I7: Add Token.String() with truncated hash for safe logging (server) - Increase E2E binary test to 1.5MB to force chunking through tunnel Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add buildDialOptions to create gRPC dial options with proxy support - Use passthrough:/// scheme to skip local DNS resolution when proxied - Build custom dialer that tunnels gRPC through HTTP CONNECT proxy - Add proto directory copy to Dockerfile for agent build - Both PROXY=0 and PROXY=1 relay_test.grpc.sh tests pass Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Refactor startAsync into getConnectionConfig for cleaner logic - If both env vars are set, connect directly without Cortex API registration - Add launch.json config for gRPC relay mode with direct connection Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add Valid() method to AcceptFileRuleWrapper to parse the "valid" field - Update MatchRule to check header requirements before matching a rule - Rules with "valid" requirements only match if request headers satisfy them - This fixes rules like the scaffolder rule being matched without x-cortex-service header - Add comprehensive unit tests for valid header matching Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Test plan
🤖 Generated with Claude Code