Drop immediate-retry logic

It's not super clear that it achieves much in the face of a rate-limited
Esplora server, and having a custom sleep there is just awkward. So we
drop it and hope we still get a chance to sync our on-chain wallet now
and then.

Author: tnull

src/wallet.rs

@@ -36,7 +36,6 @@ use bitcoin::{ScriptBuf, Transaction, TxOut, Txid};
 use std::mem;
 use std::ops::{Deref, DerefMut};
 use std::sync::{Arc, Mutex, RwLock};
-use std::time::Duration;
 
 enum WalletSyncStatus {
 	Completed,
@@ -113,21 +112,12 @@ where
 				Err(e) => match e {
 					bdk::Error::Esplora(ref be) => match **be {
 						bdk::blockchain::esplora::EsploraError::Reqwest(_) => {
-							// Drop lock, sleep for a second, retry.
-							drop(wallet_lock);
-							tokio::time::sleep(Duration::from_secs(1)).await;
 							log_error!(
 								self.logger,
-								"Sync failed due to HTTP connection error, retrying: {}",
+								"Sync failed due to HTTP connection error: {}",
 								e
 							);
-							let sync_options = SyncOptions { progress: None };
-							self.inner
-								.lock()
-								.unwrap()
-								.sync(&self.blockchain, sync_options)
-								.await
-								.map_err(|e| From::from(e))
+							Err(From::from(e))
 						},
 						_ => {
 							log_error!(self.logger, "Sync failed due to Esplora error: {}", e);