fix sharedpeererror to propagate tracing context

Cargo.lock

@@ -2344,6 +2344,7 @@ dependencies = [
  "tower",
  "tower-load",
  "tracing",
+ "tracing-error",
  "tracing-futures",
  "zebra-chain",
 ]

zebra-network/Cargo.toml

@@ -36,3 +36,4 @@ tower-load = "0.3"
 metrics = "0.12"
 
 zebra-chain = { path = "../zebra-chain" }
+tracing-error = { version = "0.1.2", features = ["traced-error"] }

zebra-network/src/peer/error.rs

@@ -2,19 +2,20 @@ use std::sync::{Arc, Mutex};
 
 use thiserror::Error;
 
+use tracing_error::TracedError;
 use zebra_chain::serialization::SerializationError;
 
 /// A wrapper around `Arc<PeerError>` that implements `Error`.
 #[derive(Error, Debug, Clone)]
-#[error("{0}")]
+#[error(transparent)]
-pub struct SharedPeerError(Arc<PeerError>);
+pub struct SharedPeerError(Arc<TracedError<PeerError>>);
 
 impl<E> From<E> for SharedPeerError
 where
     PeerError: From<E>,
 {
     fn from(source: E) -> Self {
-        Self(Arc::new(PeerError::from(source)))
+        Self(Arc::new(TracedError::from(PeerError::from(source))))
     }
 }
 

zebrad/src/commands/start.rs

@@ -211,8 +211,14 @@ where
 
     async fn drain_requests(&mut self, request_goal: usize) -> Result<(), Report> {
         while self.block_requests.len() > request_goal {
-            match self.block_requests.next().await {
+            match self
-                Some(Ok(zebra_network::Response::Blocks(blocks))) => {
+                .block_requests
+                .next()
+                .await
+                .expect("expected: block_requests is never empty")
+                .map_err::<Report, _>(|e| eyre!(e))
+            {
+                Ok(zebra_network::Response::Blocks(blocks)) => {
                     for block in blocks {
                         self.downloaded_block_heights
                             .insert(block.coinbase_height().unwrap());
@@ -225,12 +231,9 @@ where
                             .map_err(|e| eyre!(e))?;
                     }
                 }
-                Some(Ok(_)) => continue,
+                Ok(_) => continue,
-                Some(Err(e)) => {
+                Err(e) => {
-                    error!(%e);
+                    error!("{:?}", e);
-                }
-                None => {
-                    unreachable!("None is never encountered due to the condition on the while loop")
                 }
             }
         }