keep-network / tbtc-dapp

Deposit BTC and redeem TBTC
http://dapp.test.tbtc.network/
MIT License
33 stars 31 forks source link

3 Redeems sending testBTC all in 1 single transaction and giving same link #348

Closed Ostrovmaria80 closed 3 years ago

Ostrovmaria80 commented 3 years ago

Description:

This issue happens when trying simultaneously to redeem 3 different TDTs. Please see screenshots: all 3 redeem transactions display 1 single testBTC transaction. Which probably also causes loss of testBTC for users

Additional context:

https://blockstream.info/testnet/tx/758e88b6bf2421f2dd7075b92555ecc5ce3578ac601a97cb62a5c0674e102a4d

Screenshots

1in3 1 1in3 2 1in3 3

Shadowfiend commented 3 years ago

Great catch! This is a known issue in tbtc.js, due to the fact that it doesn't currently disambiguate well between different transactions to the same receiving address during redemption (embodied in this FIXME, for the curious). It's on my snipe list, thanks for the report!