refactor(blocking): propagate hex-conservative errors instead of

oleonardolima · oleonardolima · commit ea9b5d1c0425 · 2026-01-08T00:22:09.000-03:00
`.unwrap()`.
diff --git a/src/blocking.rs b/src/blocking.rs
@@ -144,7 +144,7 @@ impl BlockingClient {
             }
             Ok(resp) => {
                 let hex_str = resp.as_str().map_err(Error::Minreq)?;
-                let hex_vec = Vec::from_hex(hex_str).unwrap();
+                let hex_vec = Vec::from_hex(hex_str)?;
                 deserialize::<T>(&hex_vec)
                     .map_err(Error::BitcoinEncoding)
                     .map(|r| Some(r))
@@ -162,7 +162,7 @@ impl BlockingClient {
             }
             Ok(resp) => {
                 let hex_str = resp.as_str().map_err(Error::Minreq)?;
-                let hex_vec = Vec::from_hex(hex_str).unwrap();
+                let hex_vec = Vec::from_hex(hex_str)?;
                 deserialize::<T>(&hex_vec).map_err(Error::BitcoinEncoding)
             }
             Err(e) => Err(e),

Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,7 @@ impl BlockingClient {`
`144`	`144`	`}`
`145`	`145`	`Ok(resp) => {`
`146`	`146`	`let hex_str = resp.as_str().map_err(Error::Minreq)?;`
`147`		`- let hex_vec = Vec::from_hex(hex_str).unwrap();`
	`147`	`+ let hex_vec = Vec::from_hex(hex_str)?;`
`148`	`148`	`deserialize::<T>(&hex_vec)`
`149`	`149`	`.map_err(Error::BitcoinEncoding)`
`150`	`150`	`.map(\|r\| Some(r))`
`@@ -162,7 +162,7 @@ impl BlockingClient {`
`162`	`162`	`}`
`163`	`163`	`Ok(resp) => {`
`164`	`164`	`let hex_str = resp.as_str().map_err(Error::Minreq)?;`
`165`		`- let hex_vec = Vec::from_hex(hex_str).unwrap();`
	`165`	`+ let hex_vec = Vec::from_hex(hex_str)?;`
`166`	`166`	`deserialize::<T>(&hex_vec).map_err(Error::BitcoinEncoding)`
`167`	`167`	`}`
`168`	`168`	`Err(e) => Err(e),`