Merge pull request #1226 from input-output-hk/damien/798/implement-anyhow-for-signable-builder

Implement anyhow context for signable_builder

Author: dlachaume

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "mithril-aggregator"
-version = "0.3.85"
+version = "0.3.86"
 description = "A Mithril Aggregator server"
 authors = { workspace = true }
 edition = { workspace = true }

mithril-aggregator/Cargo.toml

@@ -363,7 +363,8 @@ impl AggregatorRunnerTrait for AggregatorRunner {
             .dependencies
             .signable_builder_service
             .compute_protocol_message(signed_entity_type.to_owned())
-            .await?;
+            .await
+            .with_context(|| format!("Runner can not compute protocol message for signed entity type: '{signed_entity_type}'"))?;
 
         let multi_signer = self.dependencies.multi_signer.write().await;
         protocol_message.set_message_part(

mithril-aggregator/src/runtime/runner.rs

@@ -1,6 +1,6 @@
 [package]
 name = "mithril-common"
-version = "0.2.108"
+version = "0.2.109"
 authors = { workspace = true }
 edition = { workspace = true }
 documentation = { workspace = true }

mithril-common/Cargo.toml

@@ -1,3 +1,4 @@
+use anyhow::Context;
 use async_trait::async_trait;
 use std::sync::Arc;
 
@@ -47,16 +48,20 @@ impl SignableBuilderService for MithrilSignableBuilderService {
         signed_entity_type: SignedEntityType,
     ) -> StdResult<ProtocolMessage> {
         let protocol_message = match signed_entity_type {
-            SignedEntityType::MithrilStakeDistribution(e) => {
-                self.mithril_stake_distribution_builder
-                    .compute_protocol_message(e)
-                    .await?
-            }
-            SignedEntityType::CardanoImmutableFilesFull(beacon) => {
-                self.immutable_signable_builder
-                    .compute_protocol_message(beacon)
-                    .await?
-            }
+            SignedEntityType::MithrilStakeDistribution(e) => self
+                .mithril_stake_distribution_builder
+                .compute_protocol_message(e)
+                .await
+                .with_context(|| format!(
+                    "Signable builder service can not compute protocol message with epoch: '{e}'"
+                ))?,
+            SignedEntityType::CardanoImmutableFilesFull(beacon) => self
+                .immutable_signable_builder
+                .compute_protocol_message(beacon.clone())
+                .await
+                .with_context(|| format!(
+                    "Signable builder service can not compute protocol message with beacon: '{beacon}'"
+                ))?,
             SignedEntityType::CardanoStakeDistribution(_) => todo!(),
         };
 

mithril-common/src/signable_builder/signable_builder_service.rs

@@ -1,6 +1,6 @@
 [package]
 name = "mithril-signer"
-version = "0.2.75"
+version = "0.2.76"
 description = "A Mithril Signer"
 authors = { workspace = true }
 edition = { workspace = true }

mithril-signer/Cargo.toml

@@ -332,7 +332,8 @@ impl Runner for SignerRunner {
             .services
             .signable_builder_service
             .compute_protocol_message(signed_entity_type.to_owned())
-            .await?;
+            .await
+            .with_context(|| format!("Runner can not compute protocol message for signed entity type: '{signed_entity_type}'"))?;
 
         // 2 set the next signers keys and stakes in the message
         let epoch = signed_entity_type.get_epoch();