fix: add a max timeout for outgoing requests

hirosystems · Sep 10, 2024 · 89bbaac · 89bbaac
1 parent 20cfa57
commit 89bbaac
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/components/chainhook-sdk/src/chainhooks/bitcoin/mod.rs b/components/chainhook-sdk/src/chainhooks/bitcoin/mod.rs
@@ -21,7 +21,7 @@ use serde::{de, Deserialize, Deserializer};
 use serde_json::Value as JsonValue;
 use std::{
     collections::{BTreeMap, HashMap, HashSet},
-    str::FromStr,
+    str::FromStr, time::Duration,
 };
 
 use reqwest::RequestBuilder;
@@ -766,6 +766,7 @@ pub fn handle_bitcoin_hook_action<'a>(
     match &trigger.chainhook.action {
         HookAction::HttpPost(http) => {
             let client = Client::builder()
+                .timeout(Duration::from_secs(30))
                 .build()
                 .map_err(|e| format!("unable to build http client: {}", e.to_string()))?;
             let host = format!("{}", http.url);

diff --git a/components/chainhook-sdk/src/chainhooks/stacks/mod.rs b/components/chainhook-sdk/src/chainhooks/stacks/mod.rs
@@ -22,6 +22,7 @@ use schemars::JsonSchema;
 use serde_json::Value as JsonValue;
 use std::collections::{BTreeMap, HashMap};
 use std::io::Cursor;
+use std::time::Duration;
 
 use reqwest::RequestBuilder;
 
@@ -1337,6 +1338,7 @@ pub fn handle_stacks_hook_action<'a>(
     match &trigger.chainhook.action {
         HookAction::HttpPost(http) => {
             let client = Client::builder()
+                .timeout(Duration::from_secs(30))
                 .build()
                 .map_err(|e| format!("unable to build http client: {}", e.to_string()))?;
             let host = format!("{}", http.url);