rpc/client: Fail fast on any non-temporary dial error

Previously, RPC client unconditionally blocked by context passed into
`WithContext` (e.g. for 15s). In particular, it definitely stuck on
non-temporary error encounter. An example of such errors could be an
incorrect network address or connection refusal.

Now client instantly fails with any irreparable error in which it is
pointless to wait for a change in the connection state. This is achieved
with `grpcstd.FailOnNonTempDialError(true)` option.

Signed-off-by: Leonard Lyubich <[email protected]>

rpc/client/connect.go

@@ -51,15 +51,20 @@ func (c *Client) openGRPCConn(ctx context.Context, extraDialOpts ...grpcstd.Dial
 	dialCtx, cancel := context.WithTimeout(ctx, c.dialTimeout)
 	var err error
 
-	dialOpts := []grpcstd.DialOption{
-		grpcstd.WithTransportCredentials(creds),
-		grpcstd.WithReturnConnectionError(),
-	}
 	if extraDialOpts != nil {
-		dialOpts = append(dialOpts, extraDialOpts...)
+		extraDialOpts = append([]grpcstd.DialOption{
+			grpcstd.WithTransportCredentials(creds),
+			grpcstd.WithReturnConnectionError(),
+			grpcstd.FailOnNonTempDialError(true),
+		}, extraDialOpts...)
+	} else {
+		extraDialOpts = []grpcstd.DialOption{
+			grpcstd.WithTransportCredentials(creds),
+			grpcstd.WithReturnConnectionError(),
+		}
 	}
 
-	c.conn, err = grpcstd.DialContext(dialCtx, c.addr, dialOpts...)
+	c.conn, err = grpcstd.DialContext(dialCtx, c.addr, extraDialOpts...)
 
 	cancel()