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

Move to broadcast_tx_sync() for IBC transactions and keep track of account sequence #986

Closed
5 tasks
ancazamfir opened this issue May 25, 2021 · 0 comments · Fixed by #1091
Closed
5 tasks
Assignees
Milestone

Comments

@ancazamfir
Copy link
Collaborator

ancazamfir commented May 25, 2021

Crate

relayer

Summary

Move to broadcast_tx_sync() for IBC transactions

Problem Definition

hermes uses broadcast_tx_commit() to send IBC messages. While it bulks multiple messages in a transaction, it can only include one Tx per block for a given chain.
We should move to using broadcast_tx_sync() instead and keep track of account sequence.

Issues that may be addressed:
#919 - account sequence mismatch
#930, #938 - out of gas issues, incorrect bulking
#977 - broadcast_tx_commit timeout
#978 - account sequence mismatch

Proposal

Acceptance Criteria


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant