Update Wallet to 2.3.0 and other deps

[tvpeter]

Description

This PR updates the Wallet api to v2.3.0 and other dependencies.

Fixes #243

Changelog notice

• Update bdk_wallet to v2.3.0
• Add WalletEvents to the project
• Update bdk_bitcoind_rpc to v0.22.0
• Update bdk_electrum to v0.23.2
• Update bdk_kyoto to v0.15.4

Checklists

All Submissions:

• I've signed all my commits
• I followed the contribution guidelines
• I ran cargo fmt and cargo clippy before committing

[tvpeter]

@notmandatory, I’m wondering if we should update our signing approach since the current method is deprecated. If we choose to proceed with the update, we will need to add miniscript as a separate dependency, as the version exported from the Wallet (v2.3.0) is 12.3.1. We require at least version 13.0.0. Or should we postpone this update until a later version?

[notmandatory]

@notmandatory, I’m wondering if we should update our signing approach since the current method is deprecated. If we choose to proceed with the update, we will need to add miniscript as a separate dependency, as the version exported from the Wallet (v2.3.0) is 12.3.1. We require at least version 13.0.0. Or should we postpone this update until a later version?

Good question. I've created bitcoindevkit/bdk_wallet#405 to revert the deprecating of the signer mod. The new PSBT signer probably won't be available until bdk_wallet 3.1.

[va-an]

Related #243.

[va-an]

Blocked by #252.

[va-an]

Suggested change

	let update = client.full_scan(request, _stop_gap, batch_size, false)?;
	let update = client.full_scan(request, _stop_gap, *batch_size, false)?;

[va-an]

Suggested change

	let update = client.sync(request, batch_size, false)?;
	let update = client.sync(request, *batch_size, false)?;

[tvpeter]

Blocked by #252.

Hi @va-an, could you please verify that you tagged the correct PR? Thank you!

[va-an]

Blocked by #252.

Hi @va-an, could you please verify that you tagged the correct PR? Thank you!

All correct. I've also reproduced error from PR's CI build in my local run just pre-push:

error[E0061]: this method takes 1 argument but 2 arguments were supplied
    --> src/payjoin/mod.rs:605:30
     |
 605 |   ...                   .check_payment(
     |                          ^^^^^^^^^^^^^
...
 618 | / ...                       |outpoint| {
 619 | | ...                           let utxo = self.wallet.get_utxo(outpoint);
 620 | | ...                           match utxo {
 621 | | ...                               Some(_) => Ok(false),
...    |
 624 | | ...                       }
     | |___________________________- unexpected argument #2 of type `{closure@src/payjoin/mod.rs:618:33: 618:43}`
     |
note: method defined here
    --> /home/vaan/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/payjoin-1.0.0-rc.2/src/core/receive/v2/mod.rs:1262:12
     |
1262 |     pub fn check_payment(
     |            ^^^^^^^^^^^^^
help: remove the extra argument
     |
 617 -                                 },
 618 -                                 |outpoint| {
 619 -                                     let utxo = self.wallet.get_utxo(outpoint);
 620 -                                     match utxo {
 621 -                                         Some(_) => Ok(false),
 622 -                                         None => Ok(true),
 623 -                                     }
     |

I've applied the fix from #252 to my local code to verify that it helps.