forked from comit-network/xmr-btc-swap
-
Notifications
You must be signed in to change notification settings - Fork 13
upgrade(swap): Concurrent syncing, bdk upgrade, refactors #180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
128 commits
Select commit
Hold shift + click to select a range
cc760f0
upgrade sqlx to 0.8, add bdk_wallet and bdk_electrum
Einliterflasche 52e52f1
move old wallet code to extra module
Einliterflasche 988143c
fix fee estimation for old client
Einliterflasche e556bc3
bump bitcoin crate, add new wallet constructor
Einliterflasche 61e45c7
remove unused old Client, move code around for better readibility
Einliterflasche dc1e323
make Wallet generic over Persister (database) and move more code arou…
Einliterflasche c7e8db3
add script history, start reimplementing client methods
Einliterflasche b30cc91
update some imports
Einliterflasche 47bb2ff
cargo fmt
Einliterflasche a5c4261
Add comments, fix fee estimation, address generation and status_of_sc…
Einliterflasche 32430db
redo state update and wallet sync
Einliterflasche e634cc8
fix bitcoin address validation and more imports, use Amount everywhere
Einliterflasche 1d2543a
fix tx cancel, lock, punish, redeem, refund
Einliterflasche 11c9aa9
fix bitcoin::Address de-/serialisation
Einliterflasche bde3b91
fix more address validation
Einliterflasche 3907593
fix more address parsing and validation, also some more imports
Einliterflasche 533bd30
cargo fmt
Einliterflasche 5e15c66
fix wallet initialization, start wallet migration
Einliterflasche 69269c4
fail test instead of ignoring it
Einliterflasche 5e52e0a
perform full scan on creation, load from db if it exists
Einliterflasche c353538
add more wallet info, fix wallet initialization
Einliterflasche 1302840
fix: default to null in config
binarybaron ec361a6
migrate from old wallet if needed
Einliterflasche 4e908e7
change something
Einliterflasche 65eed53
fix some tests
Einliterflasche 3b92235
Merge remote-tracking branch 'origin/master' into swap/upgrade-bdk
Einliterflasche 0129170
temporarily patch bdk_wallet and bdk_electrum
Einliterflasche f421c55
fix more tests
Einliterflasche a3f4317
fix missing rustls
Einliterflasche 556d161
asb: only start tor client if register_hidden_service=true in the config
Einliterflasche b048569
fix: use p2wsh_signature_hash instead of p2wpkh_signature_hash
Einliterflasche e82e2e6
fix some bitcoin address parsing and fee rate parsing
Einliterflasche 54dc27d
dprint fmt
Einliterflasche a863271
add bitcoin-harness to this project and update to the new bitcoin ver…
Einliterflasche ffbd53a
fix max_givible again
Einliterflasche e38d78d
create electrum client separately from wallet, clean up some code
Einliterflasche 4777322
Merge branch 'master' into swap/upgrade-bdk
Einliterflasche 995ff2c
add comment
Einliterflasche 455e83b
Merge branch 'master' into swap/upgrade-bdk
Einliterflasche 26b43d0
ignore .env.development
Einliterflasche c0828fe
log config file path on ./asb config
Einliterflasche 14a5b4c
feat(monero-sys): Initial commit. Regtest integration test. Wrapper a…
binarybaron 455cd5b
Merge master into swap/upgrade-bdk branch
binarybaron cf5269e
Revert "feat(monero-sys): Initial commit. Regtest integration test. W…
binarybaron 2654402
upgrade to rust toolchain 1.81
binarybaron c41cc38
Use new bdk update for code from master
binarybaron 82603a7
fix
binarybaron 49df038
remove
binarybaron e534a55
fix: add empty .gitmodules file to fix Docker build
binarybaron 90ec757
fix: clean up submodule references
binarybaron 8ff0003
fix: properly declare monero submodule with ignore flag
binarybaron 08af515
fix(wallet, bdk): only reveal new address if absolutely necessary
binarybaron 5aaac32
fix: private keys not loaded into bdk wallet
binarybaron 2102314
refactor: sync wallet progress log
binarybaron 4454a3f
dprint fmt
binarybaron 7f6a51d
refactor: move bitcoin-harness to outside repo
binarybaron 1696640
refactor: remove redundant log message
binarybaron 8a23ba6
Display sync progress
binarybaron d028afe
Remove redundant arg to swap/tests/harness/mod.rs function
binarybaron 74d6c6d
fix: call rustls::crypto::ring::default_provider()
binarybaron 7305164
dprint fmt
binarybaron 1fab238
refactor: remove debug code
binarybaron d09d3ff
refactor: move old bdk wallet export to own function, clear log messages
binarybaron a0279de
remove old migr for testnets (checksum mismatch), remove balance and …
binarybaron 571608e
use revalidate_network function, remove redundant drop
binarybaron cf5c17f
Display progress of background tasks, TauriBackgroundProgressHandle s…
binarybaron 6bfc159
Merge branch 'master' into swap/upgrade-bdk
binarybaron aca3e57
fix: almost satisfy clippy
binarybaron a89e1e1
fix: gen-bindings error
binarybaron e072261
feat: add BackgroundRefund background type
binarybaron 0591c64
feat: use builder pattern for constructing Bitcoin wallet
binarybaron b9c4f2d
dprint ftm
binarybaron 86042d7
sync electrum in seperate thread
binarybaron fc16f63
do not allow user to start sync while sync is in progress
binarybaron 50eb315
remove redundant log message
binarybaron 6759ae8
display random buffer in AlertWithLinearProgress progress
binarybaron d696269
fix: use TauriContextStatusEvent.Available), dont show syncing wallet…
binarybaron 25d217d
differentiate between TestWalletBuilder and WalletBuilder
binarybaron e64f543
satisfy clippy
binarybaron de9afc1
remove custom BackgroundRefund event, move into background process ar…
binarybaron ee7970a
refactor
binarybaron 9ad6d34
dprint fmt
binarybaron 53c2caa
progress: get unit tests compiling
binarybaron c695c2e
fix: bitcoin unit tests specify const values like sync_interval
binarybaron dfc67c5
fix: get unit tests passing
binarybaron 1364290
make clippy happy
binarybaron ab74b29
feat: display full sync progress, fix unit test import issues
binarybaron 2cca39d
dprint fmt
binarybaron f5c0c18
make clippy happy, use u32 for target_block and not usize
binarybaron 4301e91
always spawn tor for asb
binarybaron 3d57cd1
refactor: remove gen_background_progress_id and just use Uuid::new_v4()
binarybaron 958f6b4
refactor(hooks.ts): clarify comment on useConservativeBitcoinSyncProg…
binarybaron 2ac0bef
fix typo
binarybaron abd7fb6
refactor: do not let WalletBuilder take entire env struct
binarybaron 5250a29
dprint fmt
binarybaron 5758eff
refactor: remove default feature from workspace patch of bdk
binarybaron b061015
first try for concurrent syncing
binarybaron ab8f486
refactor: concurrent syncing
binarybaron 48cbd5c
fix(wallet.rs): Safely convert FeeRate from btc / kb to sats / kwu
binarybaron b6d80d4
feat(wallet.rs): persist published Bitcoin transactions without requi…
binarybaron 8a0354d
refactor(wallet.rs): use just 5 concurrent sync requests
binarybaron 42f33d5
refactor: display snackbar error when Wallet refresh fails
binarybaron 527a287
fix: add missing space
binarybaron dc0cd76
dprint fmt
binarybaron 70112d6
refactor: fancy traits for the CumulativeProgress struct, allow limit…
binarybaron 15a6ef4
make clippy happy
binarybaron b902c02
dprint fmt
binarybaron 6f4a8fa
refactor: clearly differntiate between SyncMutex and TokioMutex, use …
binarybaron 5f0b136
fix: skip syncing if no spks in wallet
binarybaron 1eeb10c
fix: update bdk.sh to test migration from old wallet (pre 1.0.0 bdk) …
binarybaron c36c420
fix: increase bitcoin_lock_confirmed_timeout in RegTest env to 5 minutes
binarybaron 90cede6
refactor: avoid usize where possible, create persistence only after f…
binarybaron 22041ce
make clippy happy
binarybaron ca4ba55
fix(ci): change rust toolchain 1.81
binarybaron 483768b
fix(cross compilation arm): use ring instead of aws-lc-rs
binarybaron 44898f9
fmt
binarybaron 3d7909d
Merge branch 'master' into swap/upgrade-bdk
binarybaron 470784a
ignore failing rendezvous tests
binarybaron 67bec79
fix printing_status_change_doesnt_spam_on_same_status
binarybaron b406883
fix: given_bitcoin_address_network_mismatch_then_error test
binarybaron 3a346ef
ignore list_sellers_should_report_all_registered_asbs_with_a_quote test
binarybaron 3ffa61a
feat: add tor icon
binarybaron b2ce5c0
refactor(wallet.rs): reorder struct by abstraction level
binarybaron ee7be94
refactor(bitcoin wallet): chunk size for syncing
binarybaron f0410f3
fix(integration tests): decrease sync interval to 3s
binarybaron 8d204a5
fix: resolve merge conflict in background.ts
binarybaron 2a33b9d
fix(integration tests): parse_rpc_err method to take new bdk error, n…
binarybaron 493e430
add changelog entry
binarybaron File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,7 +19,7 @@ jobs: | |
|
||
- uses: dtolnay/rust-toolchain@master | ||
with: | ||
toolchain: "1.80" | ||
toolchain: "1.81" | ||
components: clippy,rustfmt | ||
|
||
- uses: Swatinem/[email protected] | ||
|
@@ -127,7 +127,7 @@ jobs: | |
|
||
- uses: dtolnay/rust-toolchain@master | ||
with: | ||
toolchain: "1.80" | ||
toolchain: "1.81" | ||
targets: armv7-unknown-linux-gnueabihf | ||
|
||
- name: Install dependencies required by Tauri v2 (ubuntu only) | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,4 @@ | ||
target | ||
.vscode | ||
.vscode | ||
.claude/settings.local.json | ||
.DS_Store |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
# Explicitly declare submodules that no longer exist | ||
# This prevents Docker build errors when trying to clone missing submodules | ||
[submodule "monero-sys/monero"] | ||
path = monero-sys/monero | ||
url = https://github.com/monero-project/monero.git | ||
ignore = all | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.