}
}
+/// The minimum feerate we are allowed to send, as specify by LDK.
+const MIN_FEERATE: u32 = 253;
+
impl BitcoindClient {
pub async fn new(
host: String, port: u16, rpc_user: String, rpc_password: String,
let http_endpoint = HttpEndpoint::for_host(host.clone()).with_port(port);
let rpc_credentials =
base64::encode(format!("{}:{}", rpc_user.clone(), rpc_password.clone()));
- let bitcoind_rpc_client = RpcClient::new(&rpc_credentials, http_endpoint)?;
+ let mut bitcoind_rpc_client = RpcClient::new(&rpc_credentials, http_endpoint)?;
+ let _dummy = bitcoind_rpc_client
+ .call_method::<BlockchainInfo>("getblockchaininfo", &vec![])
+ .await
+ .map_err(|_| {
+ std::io::Error::new(std::io::ErrorKind::PermissionDenied,
+ "Failed to make initial call to bitcoind - please check your RPC user/password and access settings")
+ })?;
let mut fees: HashMap<Target, AtomicU32> = HashMap::new();
- fees.insert(Target::Background, AtomicU32::new(253));
+ fees.insert(Target::Background, AtomicU32::new(MIN_FEERATE));
fees.insert(Target::Normal, AtomicU32::new(2000));
fees.insert(Target::HighPriority, AtomicU32::new(5000));
let client = Self {
)
.await
.unwrap();
- match resp.feerate {
- Some(fee) => fee,
- None => 253,
+ match resp.feerate_sat_per_kw {
+ Some(feerate) => std::cmp::max(feerate, MIN_FEERATE),
+ None => MIN_FEERATE,
}
};
- // if background_estimate.
let normal_estimate = {
let mut rpc = rpc_client.lock().await;
)
.await
.unwrap();
- match resp.feerate {
- Some(fee) => fee,
+ match resp.feerate_sat_per_kw {
+ Some(feerate) => std::cmp::max(feerate, MIN_FEERATE),
None => 2000,
}
};
.await
.unwrap();
- match resp.feerate {
- Some(fee) => fee,
+ match resp.feerate_sat_per_kw {
+ Some(feerate) => std::cmp::max(feerate, MIN_FEERATE),
None => 5000,
}
};
fees.get(&Target::HighPriority)
.unwrap()
.store(high_prio_estimate, Ordering::Release);
- // match fees.get(Target::Background) {
- // Some(fee) => fee.store(background_estimate, Ordering::Release),
- // None =>
- // }
- // if let Some(fee) = background_estimate.feerate {
- // fees.get("background").unwrap().store(fee, Ordering::Release);
- // }
- // if let Some(fee) = normal_estimate.feerate {
- // fees.get("normal").unwrap().store(fee, Ordering::Release);
- // }
- // if let Some(fee) = high_prio_estimate.feerate {
- // fees.get("high_prio").unwrap().store(fee, Ordering::Release);
- // }
tokio::time::sleep(Duration::from_secs(60)).await;
}
});
self.fees.get(&Target::HighPriority).unwrap().load(Ordering::Acquire)
}
}
- // self.fees.g
- // 253
- // match confirmation_target {
- // ConfirmationTarget::Background =>
- // }
- // let mut rpc = self.bitcoind_rpc_client.lock().unwrap();
-
- // let (conf_target, estimate_mode, default) = match confirmation_target {
- // ConfirmationTarget::Background => (144, "ECONOMICAL", 253),
- // ConfirmationTarget::Normal => (18, "ECONOMICAL", 20000),
- // ConfirmationTarget::HighPriority => (6, "CONSERVATIVE", 50000),
- // };
-
- // // This function may be called from a tokio runtime, or not. So we need to check before
- // // making the call to avoid the error "cannot run a tokio runtime from within a tokio runtime".
- // let conf_target_json = serde_json::json!(conf_target);
- // let estimate_mode_json = serde_json::json!(estimate_mode);
- // let resp = match Handle::try_current() {
- // Ok(_) => tokio::task::block_in_place(|| {
- // runtime
- // .block_on(rpc.call_method::<FeeResponse>(
- // "estimatesmartfee",
- // &vec![conf_target_json, estimate_mode_json],
- // ))
- // .unwrap()
- // }),
- // _ => runtime
- // .block_on(rpc.call_method::<FeeResponse>(
- // "estimatesmartfee",
- // &vec![conf_target_json, estimate_mode_json],
- // ))
- // .unwrap(),
- // };
- // if resp.errored {
- // return default;
- // }
- // resp.feerate.unwrap()
}
}
let tx_serialized = serde_json::json!(encode::serialize_hex(tx));
tokio::spawn(async move {
let mut rpc = bitcoind_rpc_client.lock().await;
- rpc.call_method::<RawTx>("sendrawtransaction", &vec![tx_serialized]).await.unwrap();
+ // This may error due to RL calling `broadcast_transaction` with the same transaction
+ // multiple times, but the error is safe to ignore.
+ match rpc.call_method::<RawTx>("sendrawtransaction", &vec![tx_serialized]).await {
+ Ok(_) => {}
+ Err(e) => {
+ let err_str = e.get_ref().unwrap().to_string();
+ if !err_str.contains("Transaction already in block chain")
+ && !err_str.contains("Inputs missing or spent")
+ && !err_str.contains("bad-txns-inputs-missingorspent")
+ && !err_str.contains("non-BIP68-final")
+ {
+ panic!("{}", e);
+ }
+ }
+ }
});
- // let bitcoind_rpc_client = self.bitcoind_rpc_client.clone();
- // tokio::spawn(async move {
- // let rpc = bitcoind_rpc_client.lock().await;
- // rpc.call_method::<R>
- // });
- // let mut rpc = self.bitcoind_rpc_client.lock().unwrap();
- // let runtime = self.runtime.lock().unwrap();
-
- // let tx_serialized = serde_json::json!(encode::serialize_hex(tx));
- // // This function may be called from a tokio runtime, or not. So we need to check before
- // // making the call to avoid the error "cannot run a tokio runtime from within a tokio runtime".
- // match Handle::try_current() {
- // Ok(_) => {
- // tokio::task::block_in_place(|| {
- // runtime
- // .block_on(
- // rpc.call_method::<RawTx>("sendrawtransaction", &vec![tx_serialized]),
- // )
- // .unwrap();
- // });
- // }
- // _ => {
- // runtime
- // .block_on(rpc.call_method::<RawTx>("sendrawtransaction", &vec![tx_serialized]))
- // .unwrap();
- // }
- // }
}
}