From 98a3b3282a0ff59bbdf900adc765f6912d1975c1 Mon Sep 17 00:00:00 2001 From: rajarshimaitra Date: Thu, 16 Dec 2021 20:51:18 +0530 Subject: [PATCH] Remove sync verification The default sync verification is removed from wallet module. By default sync time verification only makes sense for `electrum` and `esplora` backend as they are usually untrusted 3rd party services. script verification for transaction is costly, so removing default script verification optimizes performance. --- src/wallet/mod.rs | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/src/wallet/mod.rs b/src/wallet/mod.rs index bf7993d4..8c3c968b 100644 --- a/src/wallet/mod.rs +++ b/src/wallet/mod.rs @@ -1537,23 +1537,6 @@ where .sync(self.database.borrow_mut().deref_mut(), progress_update,))?; } - #[cfg(feature = "verify")] - { - debug!("Verifying transactions..."); - for mut tx in self.database.borrow().iter_txs(true)? { - if !tx.verified { - verify::verify_tx( - tx.transaction.as_ref().ok_or(Error::TransactionNotFound)?, - self.database.borrow().deref(), - &self.client, - )?; - - tx.verified = true; - self.database.borrow_mut().set_tx(&tx)?; - } - } - } - let sync_time = SyncTime { block_time: BlockTime { height: maybe_await!(self.client.get_height())?,