search

sygmaprotocol / sygma-relayer

GNU Lesser General Public License v3.0
20 stars 18 forks source link

Failed Execution on BTC is considered resolved if retried from EVM side #325

Open LyonSsS opened 5 days ago

LyonSsS commented 5 days ago

Context: If an Execution fails on destination side ( BTC) by any reasons, we can execute RETRY from EVM side ( old behavior). On this one, the Relayer considers this as an already resolved transaction.

Deposit: https://sepolia.etherscan.io/tx/0x39c9c4ec59d8b8aa416034c179e0d549b625a651980b65eea57712034684e052 Retry: https://sepolia.etherscan.io/tx/0x6fcb0dc29febe164013604bd1918e643ec41f14bdab5f037e11ebdc8ecad8182

Relayer log initial deposit execution here image

Retry logs here image

mpetrun5 commented 5 days ago

There is a PR for this based on audit issues: https://github.com/sygmaprotocol/sygma-relayer/pull/319 Though there are some caveats to it described in the audit issue comments.

LyonSsS commented 5 days ago

Going to keep this as cloned issue after https://github.com/sygmaprotocol/sygma-relayer/pull/319 to remind to retest this. Thanks for pointing it out @mpetrun5