Merge #1782: fix electrum conftime_req filter for needs_block_height

20100aa ci: downgrade all workflows to ubuntu to 20.04 (Steve Myers)
53fb49f ci(clippy): fix disallow ref to static mut (Steve Myers)
f2e1632 ci: pin msrv dep version for ureq (Steve Myers)
fa53884 ci: pin msrv dep version for rustls and hashbrown (Steve Myers)
6a5b418 ci(clippy): fix missing docs errors for rust 1.83 (Steve Myers)
341b869 ci(clippy): fix updated ptr_arg check for rust 1.83 (Steve Myers)
24ec31b ci(clippy): fix new stricter needless_lifetime errors for rust 1.83 (Steve Myers)
4aa8fba ci: update examples/rpcwallet to use electrds/bitcoind_22_1 (Steve Myers)
8d190ba fix electrum conftime_req filter for needs_block_height (Zoe Faltibà)

Pull request description:

  <!-- You can erase any parts of this template not applicable to your Pull Request. -->

  ### Description

  <!-- Describe the purpose of this PR, what's being adding and/or fixed -->

  This PR fixes a bug introduced in 04994e4#diff-57bf66f87897e694c5ebfdfe0fd366774dda30b43eab93c1a0fdc802d0eb8c8dR171

  In that commit `block_times.get(height).is_none()` was converted to `block_times.contains_key(height)`, inverting the code logic.

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [x] I ran `cargo fmt` and `cargo clippy` before committing

ACKs for top commit:
  notmandatory:
    ACK 20100aa
  oleonardolima:
    utACK 20100aa
  nymius:
    ACK 20100aa

Tree-SHA512: 85b41b4f00273d0936172d7b845d28343b519cf698b4127dba85d45ab4030c69b6696f029bec38a58e3c2b3b17ff77620970d4c8fe20529980b13082299f5241

Author: notmandatory

.github/workflows/code_coverage.yml

@@ -14,7 +14,7 @@ jobs:
   Codecov:
     name: Code Coverage
     if: false # disabled
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     env:
       RUSTFLAGS: "-Cinstrument-coverage"
       RUSTDOCFLAGS: "-Cinstrument-coverage"

.github/workflows/cont_integration.yml

@@ -14,7 +14,7 @@ jobs:
 
   build-test:
     name: Build and test
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     strategy:
       matrix:
         rust:
@@ -64,8 +64,9 @@ jobs:
           cargo update -p regex --precise "1.7.3"
           cargo update -p security-framework-sys --precise "2.11.1"
           cargo update -p url --precise "2.5.0"
-          cargo update -p [email protected] --precise "0.23.17"
-          cargo update -p [email protected] --precise "0.15.0"
+          cargo update -p [email protected] --precise "0.23.19"
+          cargo update -p [email protected] --precise "0.15.0"
+          cargo update -p ureq --precise "2.10.1"
       - name: Build
         run: cargo build --features bitcoin/std,miniscript/std,${{ matrix.features }} --no-default-features
       - name: Clippy
@@ -76,7 +77,7 @@ jobs:
 
   test-readme-examples:
     name: Test README.md examples
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: checkout
         uses: actions/checkout@v4
@@ -166,7 +167,7 @@ jobs:
 
   fmt:
     name: Rust fmt
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: Checkout
         uses: actions/checkout@v4

.github/workflows/nightly_docs.yml

@@ -13,7 +13,7 @@ on:
 jobs:
   build_docs:
     name: Build docs
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: Checkout sources
         uses: actions/checkout@v4
@@ -28,7 +28,9 @@ jobs:
       - name: Install Rust toolchain
         uses: dtolnay/rust-toolchain@nightly
         with:
-          components: clippy
+          components: clippy, rustfmt
+      - name: Cargo update
+        run: cargo update
       - name: Build docs
         run: cargo rustdoc --verbose --features=compiler,electrum,esplora,use-esplora-blocking,compact_filters,rpc,key-value-db,sqlite,all-keys,verify,hardware-signer -- --cfg docsrs -Dwarnings
       - name: Upload artifact
@@ -41,7 +43,7 @@ jobs:
     name: 'Publish docs'
     if: github.ref == 'refs/heads/master'
     needs: [build_docs]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: Checkout `bitcoindevkit.org`
         uses: actions/checkout@v4

Cargo.toml

@@ -130,7 +130,7 @@ path = "examples/policy.rs"
 [[example]]
 name = "rpcwallet"
 path = "examples/rpcwallet.rs"
-required-features = ["keys-bip39", "key-value-db", "rpc", "electrsd/bitcoind_22_0"]
+required-features = ["keys-bip39", "key-value-db", "rpc", "electrsd/bitcoind_22_1"]
 
 [[example]]
 name = "psbt_signer"

README.md

@@ -215,6 +215,7 @@ cargo update -p home --precise "0.5.5"
 cargo update -p regex --precise "1.7.3"
 cargo update -p security-framework-sys --precise "2.11.1"
 cargo update -p url --precise "2.5.0"
-cargo update -p [email protected] --precise "0.23.17"
-cargo update -p [email protected] --precise "0.15.0"
+cargo update -p [email protected] --precise "0.23.19"
+cargo update -p [email protected] --precise "0.15.0"
+cargo update -p ureq --precise "2.10.1"
 ```

examples/compiler.rs

@@ -35,7 +35,6 @@ use bdk::{KeychainKind, Wallet};
 /// can be derived from the policy.
 ///
 /// This example demonstrates the interaction between a bdk wallet and miniscript policy.
-
 fn main() -> Result<(), Box<dyn Error>> {
     env_logger::init_from_env(
         env_logger::Env::default().filter_or(env_logger::DEFAULT_FILTER_ENV, "info"),

examples/policy.rs

@@ -27,7 +27,6 @@ use bdk::wallet::signer::SignersContainer;
 ///
 /// This example demos a Policy output for a 2of2 multisig between between 2 parties, where the wallet holds
 /// one of the Extend Private key.
-
 fn main() -> Result<(), Box<dyn Error>> {
     env_logger::init_from_env(
         env_logger::Env::default().filter_or(env_logger::DEFAULT_FILTER_ENV, "info"),

src/blockchain/electrum.rs

@@ -168,7 +168,7 @@ impl WalletSync for ElectrumBlockchain {
                     let needs_block_height = conftime_req
                         .request()
                         .filter_map(|txid| txid_to_height.get(txid).cloned())
-                        .filter(|height| block_times.contains_key(height))
+                        .filter(|height| !block_times.contains_key(height))
                         .take(chunk_size)
                         .collect::<HashSet<u32>>();
 

src/database/keyvalue.rs

@@ -405,12 +405,13 @@ impl BatchDatabase for Tree {
 #[cfg(test)]
 mod test {
     use lazy_static::lazy_static;
+    use std::sync::atomic::{AtomicUsize, Ordering};
     use std::sync::{Arc, Condvar, Mutex, Once};
     use std::time::{SystemTime, UNIX_EPOCH};
 
     use sled::{Db, Tree};
 
-    static mut COUNT: usize = 0;
+    static COUNT: AtomicUsize = AtomicUsize::new(0);
 
     lazy_static! {
         static ref DB: Arc<(Mutex<Option<Db>>, Condvar)> =
@@ -419,33 +420,31 @@ mod test {
     }
 
     fn get_tree() -> Tree {
-        unsafe {
-            let cloned = DB.clone();
-            let (mutex, cvar) = &*cloned;
+        let cloned = DB.clone();
+        let (mutex, cvar) = &*cloned;
 
-            INIT.call_once(|| {
-                let mut db = mutex.lock().unwrap();
+        INIT.call_once(|| {
+            let mut db = mutex.lock().unwrap();
 
-                let time = SystemTime::now().duration_since(UNIX_EPOCH).unwrap();
-                let mut dir = std::env::temp_dir();
-                dir.push(format!("mbw_{}", time.as_nanos()));
+            let time = SystemTime::now().duration_since(UNIX_EPOCH).unwrap();
+            let mut dir = std::env::temp_dir();
+            dir.push(format!("mbw_{}", time.as_nanos()));
 
-                *db = Some(sled::open(dir).unwrap());
-                cvar.notify_all();
-            });
+            *db = Some(sled::open(dir).unwrap());
+            cvar.notify_all();
+        });
 
-            let mut db = mutex.lock().unwrap();
-            while !db.is_some() {
-                db = cvar.wait(db).unwrap();
-            }
+        let mut db = mutex.lock().unwrap();
+        while !db.is_some() {
+            db = cvar.wait(db).unwrap();
+        }
 
-            COUNT += 1;
+        COUNT.fetch_add(1, Ordering::Relaxed);
 
-            db.as_ref()
-                .unwrap()
-                .open_tree(format!("tree_{}", COUNT))
-                .unwrap()
-        }
+        db.as_ref()
+            .unwrap()
+            .open_tree(format!("tree_{}", COUNT.load(Ordering::Relaxed)))
+            .unwrap()
     }
 
     #[test]

src/database/mod.rs

@@ -214,6 +214,7 @@ pub(crate) trait DatabaseUtils: Database {
 impl<T: Database> DatabaseUtils for T {}
 
 #[cfg(test)]
+#[allow(missing_docs)]
 pub mod test {
     use bitcoin::consensus::encode::deserialize;
     use bitcoin::consensus::serialize;