Improve ConsensusConfig.protocol_version conversion

matter-labs · Nov 28, 2023 · 6ff7532 · 6ff7532
1 parent d0f85d9
commit 6ff7532
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/node/actors/executor/src/config/mod.rs b/node/actors/executor/src/config/mod.rs
@@ -31,11 +31,12 @@ impl ProtoFmt for ConsensusConfig {
     type Proto = proto::ConsensusConfig;
 
     fn read(proto: &Self::Proto) -> anyhow::Result<Self> {
-        let protocol_version = proto.protocol_version.context("protocol_version")?;
         Ok(Self {
             key: read_required_text(&proto.key).context("key")?,
             public_addr: read_required_text(&proto.public_addr).context("public_addr")?,
-            protocol_version: validator::ProtocolVersion::try_from(protocol_version)
+            protocol_version: required(&proto.protocol_version)
+                .copied()
+                .and_then(validator::ProtocolVersion::try_from)
                 .context("protocol_version")?,
         })
     }