Commit graph

1281 commits

Author SHA1 Message Date
patrini32
0d40179a65 remove changes from state.rs 2024-06-25 17:23:15 +03:00
patrini32
f5f8af92a0 check for punish timelock in fn refund, not yet expired timelock in cancel and remove breaking database changes 2024-06-25 17:23:15 +03:00
patrini32
0caf678192 better comments 2024-06-25 17:23:15 +03:00
patrini32
f2973bf5cc remove missleading comment 2024-06-25 17:23:15 +03:00
patrini32
f66c16a381 provide better comments 2024-06-25 17:23:15 +03:00
patrini32
08a97ba1b7 provide better comments 2024-06-25 17:23:15 +03:00
patrini32
2c37e6e4f4 Revert "Reimplement tx_punish check without breaking database changes"
This reverts commit ab33bcde8f.
2024-06-25 17:23:15 +03:00
patrini32
1f9ac64166 Revert "fix formatting"
This reverts commit 990206596b.
2024-06-25 17:23:15 +03:00
patrini32
a133321255 fix formatting 2024-06-25 17:23:15 +03:00
patrini32
756f817daa Reimplement tx_punish check without breaking database changes 2024-06-25 17:23:15 +03:00
patrini32
8866e60323 Use transaction created by check_for_tx_cancel and delete construct_tx_cancel 2024-06-25 17:23:15 +03:00
patrini32
4b444140c0 Remove redundant comment 2024-06-25 17:23:15 +03:00
patrini32
6c2c205dd8 Remove redundant debug print 2024-06-25 17:23:15 +03:00
patrini32
5f392e3b7b return accidentally removed code 2024-06-25 17:23:15 +03:00
patrini32
1d2223c54d Return accidentally deleted check for RpcVerifyAlreadyInChain 2024-06-25 17:23:15 +03:00
patrini32
8ac04e0d16 Check if Alice already published tx_cancel/tx_punish if we get an error when trying to cancel/refund. Test: use cli::cancel and cli::refund to check if cli::cancel sets state to BtcCancelled 2024-06-25 17:23:15 +03:00
patrini32
5dc717d3e8 formalize description 2024-06-25 17:23:15 +03:00
patrini32
7f80966b86 better description 2024-06-25 17:23:15 +03:00
patrini32
dc84437f63 better description 2024-06-25 17:23:15 +03:00
patrini32
25ff29d633 Refactor and describe 'subscription' deletion 2024-06-25 17:23:15 +03:00
patrini32
aabff429bc fix formatting: delete whitespaces 2024-06-25 17:23:15 +03:00
patrini32
25ebcf731d fix formatting 2024-06-25 17:23:15 +03:00
patrini32
70220d1581 Add comments about logic 2024-06-25 17:23:15 +03:00
patrini32
8b9f1b04c9 refactor and print successful refund message only if it succeeded 2024-06-25 17:23:15 +03:00
patrini32
1c6e757f02 Use assert_bob_punished 2024-06-25 17:23:15 +03:00
patrini32
4635d0aae9 use original approach and add timelock check 2024-06-25 17:23:15 +03:00
patrini32
4421db7aad Revert "introduce better approach for checking for error"
This reverts commit 7844ad1483.
2024-06-25 17:23:15 +03:00
patrini32
ca8a9cbc0b Revert "Fix multiple things"
This reverts commit d00293de35.
2024-06-25 17:23:15 +03:00
patrini32
c9e3c40987 Fix multiple things 2024-06-25 17:23:15 +03:00
patrini32
e2b6538199 introduce better approach for checking for error 2024-06-25 17:23:15 +03:00
patrini32
a9a1724ad3 implement refund and lock check 2024-06-25 17:23:15 +03:00
patrini32
957c33bffe Fix stupid errors 2024-06-25 17:23:15 +03:00
patrini32
7e5c1ae45a fix formatting and typo 2024-06-25 17:23:15 +03:00
patrini32
8e4fd2cc3b Add test and rustify code 2024-06-25 17:23:15 +03:00
patrini32
4f8907a4b0 prettify error, when bob tries to cancel and refund after disconnecting 2024-06-25 17:23:15 +03:00
dependabot[bot]
dabe1abb21
build(deps): bump uuid from 1.8.0 to 1.9.0
Bumps [uuid](https://github.com/uuid-rs/uuid) from 1.8.0 to 1.9.0.
- [Release notes](https://github.com/uuid-rs/uuid/releases)
- [Commits](https://github.com/uuid-rs/uuid/compare/1.8.0...1.9.0)

---
updated-dependencies:
- dependency-name: uuid
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-06-24 11:10:02 +00:00
dependabot[bot]
4d557d1ef9
build(deps): bump mockito from 1.3.0 to 1.4.0 (#1567)
* build(deps): bump mockito from 1.3.0 to 1.3.1

Bumps [mockito](https://github.com/lipanski/mockito) from 1.3.0 to 1.3.1.
- [Release notes](https://github.com/lipanski/mockito/releases)
- [Commits](https://github.com/lipanski/mockito/compare/1.3.0...1.3.1)

---
updated-dependencies:
- dependency-name: mockito
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: breaking change in mockito async

* bump mockito to 1.4

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Byron Hambly <byron@hambly.dev>
2024-06-19 15:35:05 +02:00
dependabot[bot]
543f2748b5
build(deps): bump serial_test from 3.0.0 to 3.1.1
Bumps [serial_test](https://github.com/palfrey/serial_test) from 3.0.0 to 3.1.1.
- [Release notes](https://github.com/palfrey/serial_test/releases)
- [Commits](https://github.com/palfrey/serial_test/compare/v3.0.0...v3.1.1)

---
updated-dependencies:
- dependency-name: serial_test
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-06-19 11:35:30 +00:00
COMIT Botty McBotface
a3b1e772b2
Prepare release 0.13.1 (#1675) 2024-06-10 20:23:50 +02:00
Byron Hambly
90494ba4a5
fix: monero wallet refresh (#1596)
This PR changes the following behaviour in the refresh functionality of the monero wallet
- Allows for multiple retries because in some cases users have experienced an issue where the wallet rpc returns `no connection to daemon` even though the daemon is available. I'm not 100% sure why this happens but retrying often fixes the issue
- Print the current sync height after each failed attempt at syncing to see how far we've come
- The `monero-wallet-rpc` is started with the `--no-initial-sync` flag which ensures that as soon as it's started, it's ready to respond to requests
- The `monero-wallet-rpc` was upgraded to `v0.18.3.1` because this PR https://github.com/monero-project/monero/pull/8941 has improved some of the issues mentioned above


This PR is part of a larger effort to fix this issue https://github.com/comit-network/xmr-btc-swap/issues/1432
2024-06-10 18:53:52 +02:00
binarybaron
55b67d31d4 feat (Bob): Log extensive information about deposit requirements 2024-06-06 15:21:48 +02:00
chengehe
49a7f7eed6
chore: remove repeat word (#1671)
Signed-off-by: chengehe <hechenge@yeah.net>
2024-06-06 12:10:19 +02:00
pokkst
9635c0b551
fix (Bob): Check if Bitcoin redeem transaction was published before transitioning to CancelTimelockExpired (#1427)
* fix (Bob): Check if Bitcoin redeem transaction was published before transitioning to CancelTimelockExpired

---------

Co-authored-by: binarybaron <86064887+binarybaron@users.noreply.github.com>
Co-authored-by: Byron Hambly <bhambly@blockstream.com>
2024-06-04 12:49:15 +02:00
dependabot[bot]
1930540c1f
build(deps): bump reqwest from 0.11.27 to 0.12.4 (#1588)
* build(deps): bump reqwest from 0.11.27 to 0.12.0

Bumps [reqwest](https://github.com/seanmonstar/reqwest) from 0.11.27 to 0.12.0.
- [Release notes](https://github.com/seanmonstar/reqwest/releases)
- [Changelog](https://github.com/seanmonstar/reqwest/blob/master/CHANGELOG.md)
- [Commits](https://github.com/seanmonstar/reqwest/compare/v0.11.27...v0.12.0)

---
updated-dependencies:
- dependency-name: reqwest
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* wip

* wip

* ci: lock sqlx-cli install

* bump reqwest to 0.12.2

* deps: reqwest to 0.12.4

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Byron Hambly <byron@hambly.dev>
2024-05-30 10:27:48 +02:00
COMIT Botty McBotface
2932abc9ec
Prepare release 0.13.0 (#1659) 2024-05-29 16:00:17 +02:00
Byron Hambly
796863359f
upgrade secp256kfun (#1466)
* ci: add cargo check on rust stable

* refactor: upgrade secp256kfun and fix resulting issues

* build(deps): update sigma_fun and ecdsa_fun to a52142cf7f

 #1520
 #1521

* chore: fix clippy issue

* update to 91112f80b24

* bump to 294de1721add

* chore(deps): remove spectral

spectral fails to compile on rust stable 1.76 due to dep on deprecated
rustc-serialize

* secp256kfun: update to 7da9d277 and set rev in manifest

* update to 6fdc5d8

* switch to crates.io versions of ecdsa_fun and sigma_fun

* ci: update toolchain to 1.74 and fix draft action

* clippy fixes

---------

Co-authored-by: binarybaron <86064887+binarybaron@users.noreply.github.com>
2024-05-27 11:03:20 +02:00
binarybaron
32ca0b1a4a CLI: Upgrade monero-wallet-rpc to 0.18.3.1, clarify a few log. messages 2024-05-25 18:22:17 +02:00
binarybaron
c433bd2389
CLI: Initiate tracing earlier to avoid lost logs (#1646) 2024-05-25 17:59:12 +02:00
binarybaron
0ca98cd0b7
Make tracing file appender blocking (#1643) 2024-05-25 14:43:22 +02:00
binarybaron
0c4b7d50c2
Lower DEFAULT_BITCOIN_CONFIRMATION_TARGET to 1 to ensure timely confirmation of bitcoin transactions (#1640) 2024-05-24 18:02:29 +02:00