Merge pull request fedimint#6301 from joschisan/safe_new_ci_variants

feat: make introduction of new ci variants consensus safe
Kodylow · Nov 11, 2024 · 2c7d93c · 2c7d93c
2 parents d4902ad + 9877168
commit 2c7d93c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/fedimint-server/src/consensus/engine.rs b/fedimint-server/src/consensus/engine.rs
@@ -768,7 +768,8 @@ impl ConsensusEngine {
                     target: LOG_CONSENSUS,
                     "Minor consensus version mismatch: unexpected consensus item type: {variant}"
                 );
-                bail!("Unexpected consensus item type: {variant}")
+
+                panic!("Unexpected consensus item type: {variant}")
             }
         }
     }

diff --git a/modules/fedimint-wallet-server/src/lib.rs b/modules/fedimint-wallet-server/src/lib.rs
@@ -497,7 +497,7 @@ impl ServerModule for Wallet {
                 }
             }
             WalletConsensusItem::Default { variant, .. } => {
-                bail!("Received wallet consensus item with unknown variant {variant}");
+                panic!("Received wallet consensus item with unknown variant {variant}");
             }
         }
-Original file line number
+Diff line change
@@ Expand Up / @@ -497,7 +497,7 @@ impl ServerModule for Wallet { @@
                     }
                 }
                 WalletConsensusItem::Default { variant, .. } => {
-                    bail!("Received wallet consensus item with unknown variant {variant}");
+                    panic!("Received wallet consensus item with unknown variant {variant}");
                 }
             }
@@ Expand Down @@