Decrease request_timeout in builders to avoid panics (#4267)

CHANGELOG.md

@@ -84,6 +84,9 @@
 - [#4177](https://github.com/ChainSafe/forest/pull/4177) Fixed a bug where the
   environment variable `IPFS_GATEWAY` was not used to change the IPFS gateway.
 
+- [#4267](https://github.com/ChainSafe/forest/pull/4267) Fixed potential panics
+  in `forest-tool api compare`.
+
 ## Forest 0.17.2 "Dovakhin"
 
 This is a **mandatory** release for all mainnet node operators. It changes the

src/rpc/client.rs

@@ -153,7 +153,7 @@ impl Debug for UrlClient {
 
 impl UrlClient {
     async fn new(url: Url, token: impl Into<Option<String>>) -> Result<Self, ClientError> {
-        let timeout = Duration::MAX; // we handle timeouts ourselves.
+        const ONE_DAY: Duration = Duration::from_secs(24 * 3600); // we handle timeouts ourselves.
         let headers = match token.into() {
             Some(token) => HeaderMap::from_iter([(
                 header::AUTHORIZATION,
@@ -172,9 +172,9 @@ impl UrlClient {
             "ws" | "wss" => OneClientInner::Ws(
                 jsonrpsee::ws_client::WsClientBuilder::new()
                     .set_headers(headers)
-                    .request_timeout(timeout)
                     .max_request_size(MAX_REQUEST_BODY_SIZE)
                     .max_response_size(MAX_RESPONSE_BODY_SIZE)
+                    .request_timeout(ONE_DAY)
                     .build(&url)
                     .await?,
             ),
@@ -183,7 +183,7 @@ impl UrlClient {
                     .set_headers(headers)
                     .max_request_size(MAX_REQUEST_BODY_SIZE)
                     .max_response_size(MAX_RESPONSE_BODY_SIZE)
-                    .request_timeout(timeout)
+                    .request_timeout(ONE_DAY)
                     .build(&url)?,
             ),
             it => {