Skip to content

Commit 9356d9d

Browse files
feat: Switch state grpc client to NewClient rather than DialContext (#2176)
We were able to validate grpc.NewClient works as expected after the upstream fix so there's nothing blocking us from upgrading it.
1 parent 86ea011 commit 9356d9d

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

state/client.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,9 +59,7 @@ func NewConnectedClientWithOptions(ctx context.Context, backendOpts *plugin.Back
5959
return &NoOpClient{}, nil
6060
}
6161

62-
// TODO: Remove once there's a documented migration path per https://github.com/grpc/grpc-go/issues/7244
63-
// nolint:staticcheck
64-
backendConn, err := grpc.DialContext(ctx, backendOpts.Connection,
62+
backendConn, err := grpc.NewClient(backendOpts.Connection,
6563
grpc.WithTransportCredentials(insecure.NewCredentials()),
6664
grpc.WithDefaultCallOptions(
6765
grpc.MaxCallRecvMsgSize(connOpts.MaxMsgSizeInBytes),

0 commit comments

Comments
 (0)