xmr-btc-swap

Commit Graph

Author	SHA1	Message	Date
rishflab	fcec465bdb	Revert "No need to send Monero transfer proof from Alice to Bob"	4 years ago
Lucas Soriano del Pino	620216a596	Remove need to send Monero transfer proof from Alice to Bob	4 years ago
Lucas Soriano del Pino	0288e004c5	Make Alice watch for Monero lock transaction without transfer proof	4 years ago
Lucas Soriano del Pino	483c819e17	Correctly re-export types from bitcoin and monero modules	4 years ago
Lucas Soriano del Pino	4790d701e5	Update cross-curve-dleq dependency Making build times considerably faster. On my machine, after running `cargo clean`, `cargo build -p swap` takes 2min 19s. The updated dependency also comes with a critical fix to the `Scalar` type, which originally wrongly assumed that secp256k1 and ed25519 scalars had the same endianness. For this reason, we now have to reverse the bytes of recovered scalars if we are to use them on a different chain. Finally, there is no need to append `RUST_MIN_STACK=100000000` to avoid stack overflows in tests and when running the binary.	4 years ago
Philipp Hoenisch	306176b3e6	PR feedback: Removed network prefix and use the same for container and network. Add sentence explaining prefix	4 years ago
Philipp Hoenisch	0b9e8c145e	Update xmr-btc lib to use new monero-harness	4 years ago
Lucas Soriano del Pino	9384b0cf3c	[WIP] Swap app e2e test	4 years ago
Lucas Soriano del Pino	3f43581da7	Implement traits Receive{BitcoinRedeemEncsig, TransferProof} Unfortunately, I had to put the wrap the swarm in Alice's `Network` struct in an `Arc<Mutex<T>>` in order to be able to use `backoff` to control the retry mechanism. This is because the stream of events cannot be turned into a `SharedFuture` (unlike Bob's). It would be good to find an alternative solution.	4 years ago
Lucas Soriano del Pino	4ee82a5a2a	Execute Alice's on-chain protocol after handshake Co-authored-by: Tobin C. Harding <tobin@coblox.tech>	4 years ago
Lucas Soriano del Pino	24104d5d35	Test that Alice punishes if Bob is inactive after locking bitcoin	4 years ago
Lucas Soriano del Pino	cea1af1e1a	Take bitcoin_tx_lock_timeout as argument to action generators	4 years ago
Lucas Soriano del Pino	c86a82b315	Rename action_generator_alice to action_generator	4 years ago
Lucas Soriano del Pino	e84c56378c	Test that both parties refund if Alice does not redeem Also: - Move generator functions to `alice` and `bob` modules. This makes using `tracing` a lot easier, since the context of the file name let's us differentiate between Alice's and Bob's generator logs more clearly. - Accept 0 confirmations when watching for the Monero lock transaction. This should eventually be configured by the application, but in the tests it's making things unexpectedly slower.	4 years ago
Philipp Hoenisch	c0109d12cd	Use lib to ensure ports are free	4 years ago
Lucas Soriano del Pino	5395303a99	Test on-chain protocol happy path	4 years ago
Lucas Soriano del Pino	50ed74319f	Simplify xmr-btc/tests Monero wallet - Make it the same for Alice and Bob. - Make it contain a wallet client instead of the `Monero` struct. Also: Remove `Container` from inside `Monero` struct. The caller of `new` can simply ensure that `Container` is not dropped to keep the container alive. This makes the `Monero` struct easier to work with, as it just holds the data necessary to create the different clients created during `init`, and does not have any lifetime restrictions.	4 years ago
Lucas Soriano del Pino	55629838f4	Implement test function that calls action_generator_alice	4 years ago
Lucas Soriano del Pino	7c99415cff	Write action_generator_alice	4 years ago
Lucas Soriano del Pino	969ca63081	Split RefundBitcoin into CancelBitcoin and RefundBitcoin	4 years ago
Lucas Soriano del Pino	176b2195e3	Make ReceiveTransferProof async and take &mut self	4 years ago
Lucas Soriano del Pino	4723626fc0	Suppress compiler noise in tests folder By having two modules that declare and use the `harness` module we ran into a situation where not all parts of the `harness` module were being used by both of these other test modules. This was producing annoying warnings. For now this can be solved by marking the `harness` module declaration as public. This has no real effect since modules under `/tests` cannot be depended on elsewhere, but it satisfies the compiler. It does mean that we will not hear about unused functions, but this change should be temporary and it's a very minor problem given that it's just test code.	4 years ago
Lucas Soriano del Pino	5daa3ea9a8	[WIP] Generate actions for Bob's on-chain protocol Mimics what @thomaseizinger did here [1] and here [2]. This has the advantage that the consumer has more freedom to execute `Action`s without having to implement particular traits. The error handling required inside this protocol-executing function is also reduced. As discussed with Thomas, for this approach to work well, the trait functions such as `receive_transfer_proof` should be infallible, and the implementer should be forced to hide IO errors behind a retry mechanism. All of these asynchronous calls need to be "raced" against the abort condition (determined by the `refund_timelock`), which is missing in the current state of the implementation. The initial handshake of the protocol has not been included here, because it may not be easy to integrate this approach with libp2p, but a couple of messages still need to exchanged. I need @tcharding to tell me if it's feasible/good to do it like this. [1] https://github.com/comit-network/comit-rs/blob/move-nectar-swap-to-comit/nectar/src/swap/comit/herc20_hbit.rs#L57-L184. [2] `e584d2b14f/nectar/src/swap.rs (L716-L751)`.	4 years ago

23 Commits (53aae5cf5fa402068449a656426658c3d7e985b6)