Skip to content
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

Failed Cosmos txs are still sent to the confirm queue #5310

Open
Tracked by #3321
tkporter opened this issue Jan 28, 2025 · 0 comments
Open
Tracked by #3321

Failed Cosmos txs are still sent to the confirm queue #5310

tkporter opened this issue Jan 28, 2025 · 0 comments
Labels
agent alt-VM non-EVM alternative execution environment cosmos solana move cardano polkadot bug Something isn't working cosmos

Comments

@tkporter
Copy link
Collaborator

tkporter commented Jan 28, 2025

Was poking around the explorer and noticed that some Neutron destination messages weren't immediately processed.

Taking a look at the logs for https://explorer.hyperlane.xyz/message/0x5eff3922f7cae967815a5ac19f934c5d7dde07136f613c4569c0affd26414d47:

  1. The first time preparing, it fails because of an account mismatch https://cloudlogging.app.goo.gl/GE2RfeUgMi1UPZ3R8
  2. Now in the submit queue with retry count 1, the tx is submitted: https://cloudlogging.app.goo.gl/iEGwjyaGyM2ptnLV7. However, you can see we immediately get back TxResponse { height: 0, txhash: "E0CF5F673A5793E504D6554D278DCB59B359DFA4DC2CD8EF71AEBFAB91242BBF", codespace: "sdk", code: 32, data: "", raw_log: "account sequence mismatch, expected 752482, got 752481: incorrect account sequence", logs: [], info: "", gas_wanted: 0, gas_used: 0, tx: None, timestamp: "", events: [] }
  3. We don't realize that this is supposed to be an error, and instead send the message straight to the confirm queue, where it sits for a number of minutes until we realize that the message was in fact not processed, and we send it back to the prep queue

Some other thoughts in the wake of this:

  • wonder if the confirm queue should be more generous with its RPC usage, as it could theoretically poll e.g. every minute or something to find out if the delivery didn't land quicker than the current period now. It'd still only consider a tx finalized if after the whole period, it's continuously been marked as processed

related: #5311

@tkporter tkporter added agent alt-VM non-EVM alternative execution environment cosmos solana move cardano polkadot bug Something isn't working cosmos labels Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agent alt-VM non-EVM alternative execution environment cosmos solana move cardano polkadot bug Something isn't working cosmos
Projects
Status: No status
Development

No branches or pull requests

1 participant