upgrade(swap): Upgrade bdk library #180
Conversation
The new dependencies are part of the bdk upgrade and include the improved wallet code. They, too, depend on sqlite3. However, they use a newer version than we currently use via sqlx. This necessitated the sqlx upgrade. This entailed trivial changes (use Pool directly instead of pool.acquire()). We might have to fix the CI as well, I kept getting compile errors from the macro until I ran swap/sqlx_dev_setup.sh.
|
Oh boy. It looks like we'll have to upgrade |
That's a bummer but I'd assume it's mostly a few API changes that propogate through the codebase. The Bitcoin protocol hasn't changed. |
|
Probably. The changelog isn't very clear on that, sadly. |
|
We also have to consider that this upgrade could be a breaking network upgrade if the serde serialization structures have changed. |
Einliterflasche
force-pushed
the
swap/upgrade-bdk
branch
from
6e6eb9d to
52e52f1
Compare
…nd for readibility
|
I haven't looked too much what you are doing here but fyi you do not really need to do any backwards compatability stuff. Our descriptor is derived from our |
|
The docs recommended that we get the revelation indices of the wallet to speed up the migration, so that's why I kept some of the old wallet code around |
|
I fixed almost all compile errors now. Only left are |
…ring re-scan This allows us to compute an updated Bitcoin balance without requiring a re-scan
…ing amount of callback calls
Fixed first (and maybe last) issue. Was a rounding issue in how we were parsing the fee returned by the Electrum server |
…traits for converting to Arc<Mutex<_>>, move sync_ext into own moid
|
The docker tests pass locally. Not sure why they don't in the CI. Another issue is that if the user interrupt the initial full scan, we will not try again.. (I'm fixing this right now) |
…ull scan, transmit assumed_total for full scan to tauri, add some icons to progress displays
Fixed |
|
Integration tests are failing. I've manually done a successful swap. Some issue with the Electrum server I believe... |
|
I tried doing a mainnet swap with this as a client and an old asb as a maker. |
This closes #98.