-
Notifications
You must be signed in to change notification settings - Fork 106
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
eth: Manual refunds #1311
Comments
In the past I had made a couple quick and dirty tools to do refunds and redeems without dexc.
These are not well generalized at all currently, only working with DCR and BTC, but that can change. In general, I think we need to make sure sufficient info is logged. Longer term, we should consider if and how the swaps can be made deterministic w.r.t. the secret keys so that more recovery options are available given the dex Core app seed. It's not clear if there is such a solution though. |
Those look nice.. I was thinking about something similar to the With ETH they are already deterministic as long as we only use a single swap contract. Definitely would be harder with DCR/BTC. I guess we would need the wallet to keep track of all the unredeemed swaps in the UTXO set. |
Address reuse and the central contract with ETH certainly makes things easier, even if crazier. A complementary idea that @buck54321 had suggested was a secondary record of the essential swap data, which would be redundant with both the DB and the log file, but potentially provide a more concise and easily parseable record of the critical on-chain swap data.
It's all already in the log file as I've demonstrated, but this would still be valuable IMO. |
Resolved by #1455, which permits manual refunds to be performed. The refund of course still pays to the initiator's account, which would be the dex ETH wallet, so actually claiming those funds requires the app or asset seed, but it can at least be done without the dexc.db. |
For BTC and DCR, we log a signed refund transaction that the user can manually submit to the blockchain in case the dex fails to properly refund a swap. For ETH this is not possible, because the nonce might have been incremented by the time the user would need to do a refund. We need an different way to do manual refunds for ETH.
The text was updated successfully, but these errors were encountered: