Merge pull request fedimint#6376 from joschisan/insert_new_entry

fix: upgrade warn to a panic
Kodylow · Nov 17, 2024 · e493ea4 · e493ea4
2 parents 03a7a94 + 4c643aa
commit e493ea4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/fedimint-core/src/db/mod.rs b/fedimint-core/src/db/mod.rs
@@ -1124,12 +1124,7 @@ where
         K::Value: MaybeSend + MaybeSync,
     {
         if let Some(prev) = self.insert_entry(key, value).await {
-            debug_assert!(
-                false,
-                "Database overwriting element when expecting insertion of new entry. Key: {key:?} Prev Value: {prev:?}"
-            );
-            warn!(
-                target: LOG_DB,
+            panic!(
                 "Database overwriting element when expecting insertion of new entry. Key: {key:?} Prev Value: {prev:?}"
             );
         }

diff --git a/modules/fedimint-lnv2-server/src/lib.rs b/modules/fedimint-lnv2-server/src/lib.rs
@@ -403,7 +403,7 @@ impl ServerModule for Lightning {
                             return Err(LightningInputError::InvalidPreimage);
                         }
 
-                        dbtx.insert_new_entry(&PreimageKey(*contract_id), preimage)
+                        dbtx.insert_entry(&PreimageKey(*contract_id), preimage)
                             .await;
 
                         contract.claim_pk