Skip to content

Commit

Permalink
fix tests: using passthrough is required for in-memory bufconn since …
Browse files Browse the repository at this point in the history
…grpc-go v1.63.0
  • Loading branch information
FZambia committed Apr 7, 2024
1 parent d2bb6e0 commit 98870be
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion internal/proxy/connect_handler_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,9 @@ type grpcConnHandleTestCase struct {

func getTestGrpcProxy(commonProxyTestCase *tools.CommonGRPCProxyTestCase) Config {
return Config{
Endpoint: commonProxyTestCase.Listener.Addr().String(),
// Using passthrough is required for in-memory bufconn since grpc-go v1.63.0.
// See https://github.com/grpc/grpc-go/issues/7091.
Endpoint: "passthrough:///" + commonProxyTestCase.Listener.Addr().String(),
Timeout: tools.Duration(5 * time.Second),
testGrpcDialer: func(ctx context.Context, s string) (net.Conn, error) {
return commonProxyTestCase.Listener.Dial()
Expand Down

0 comments on commit 98870be

Please sign in to comment.