7 Commits (7df93faa4ba410eee3d644f847687eef2a5b2084)

Author	SHA1	Message	Date
bors[bot]	d144405182	Merge #376 ... 376: ASB resumes unfinished swaps after startup r=da-kami a=da-kami Fixes #374 - [x] Save Bob peer-id in database for Alice - [x] Alice: Wait for `10` Monero confirmations in `BtcRefunded` instead of `XmrLocked` (requires extending the RPC to distinguish locked / unlocked balance) - [x] Save Alice peer-id in database for Bob ~~(+ multiaddress and remove params from resume)~~ - [ ] ~~Refactor Bob in test setup (handle event-loop in test setup similar to Alice)~~ I decided against refactoring Bob in the test setup, because eventually we might still want to add concurrent swap tests with multiple Bobs. The refactoring I had in mind would not allow such kind of tests. Generally, the current state of the changes already contains enough added value to open the PR :) Follow ups out of scope - [ ] Parametrize database with role (Alice / Bob) and remove all the (currently useless) mapping between DB and protocol types. - [ ] Alice: Wait for transfer proof ack before transitioning to new `XmrLocked` Co-authored-by: Daniel Karzel <daniel@comit.network>	3 years ago
Daniel Karzel	2135a6e53e	Alice resumes swaps	3 years ago
Daniel Karzel	d90496931b	Save Alice's peer-id in the db for Bob ... This allows loading the seller-peer-id from the database upon resuming a swap. Thus, the parameters `--seller-peer-id` is removed for the `resume` command. Other than the peer-id the multi address of a seller can change and thus is still a parameter. This parameter might become optional once we add DHT support.	3 years ago
Thomas Eizinger	e27c153c4a	Re-word changelog entry for bad readiness check of monero-wallet-rpc	3 years ago
Thomas Eizinger	52b9a78de2	Alice to validate Bob's PSBT for correctness ... In order for the re-construction of TxLock to be meaningful, we limit `Message2` to the PSBT instead of the full struct. This is a breaking change in the network layer. The PSBT is valid if: - It has at most two outputs (we allow a change output) - One of the outputs pays the agreed upon amount to a shared output script Resolves #260.	3 years ago
rishflab	bc902ea63a	Use language agnostic heuristic to check if monero_wallet_rpc is ready ... Our strategy of searching for a english string to determine if monero_wallet_rpc is ready is not compatible with languages other than english. Instead we assume the monero rpc is ready if it has stopped writing to stdout. We make a json rpc request to confirm this. A better solution would have been to configure the monero_wallet_rpc to always output in english but there is not command line argument to configure the language. Closes #353.	3 years ago
Thomas Eizinger	b25d21282d	Add CHANGELOG file	3 years ago