refactor: move is_local to transport

crates/transports/src/client.rs

@@ -75,14 +75,12 @@ where
 
 pub struct ClientBuilder<L> {
     builder: ServiceBuilder<L>,
-    is_local: bool,
 }
 
 impl<L> ClientBuilder<L> {
     pub fn layer<M>(self, layer: M) -> ClientBuilder<tower::layer::util::Stack<M, L>> {
         ClientBuilder {
             builder: self.builder.layer(layer),
-            is_local: self.is_local,
         }
     }
 
@@ -93,7 +91,8 @@ impl<L> ClientBuilder<L> {
         L::Service: Transport + Clone,
         <L::Service as tower::Service<Box<RawValue>>>::Future: Send,
     {
-        RpcClient::new(self.builder.service(transport), self.is_local)
+        let is_local = transport.is_local();
+        RpcClient::new(self.builder.service(transport), is_local)
     }
 }
 

crates/transports/src/transports/http.rs

@@ -3,7 +3,7 @@ use serde_json::value::RawValue;
 use std::{future::Future, pin::Pin, str::FromStr, sync::atomic::AtomicU64, task};
 use tower::Service;
 
-use crate::{client::RpcClient, error::TransportError};
+use crate::{client::RpcClient, error::TransportError, Transport};
 
 impl<T> RpcClient<Http<T>>
 where
@@ -88,3 +88,9 @@ impl Service<Box<RawValue>> for Http<reqwest::Client> {
         })
     }
 }
+
+impl Transport for Http<reqwest::Client> {
+    fn is_local(&self) -> bool {
+        self.is_local()
+    }
+}

crates/transports/src/transports/mod.rs

@@ -107,20 +107,7 @@ pub trait Transport:
     + Send
     + 'static
 {
-}
-
-impl<T> Transport for T
-where
-    T: Service<
-            Box<RawValue>,
-            Response = Box<RawValue>,
-            Error = TransportError,
-            Future = Pin<Box<dyn Future<Output = Result<Box<RawValue>, TransportError>> + Send>>,
-        > + Clone
-        + Send
-        + 'static,
-    T::Future: Send + 'static,
-{
+    fn is_local(&self) -> bool;
 }
 
 #[must_use = "futures do nothing unless you `.await` or poll them"]