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

feat: add the ability to finish connection and channel handshakes for a specified connection/channel #1127

Open
jtieri opened this issue Mar 15, 2023 · 2 comments
Labels
C: CLI COMPONENT: Command-Line Interface T: Enhancement TYPE: Enhancement

Comments

@jtieri
Copy link
Member

jtieri commented Mar 15, 2023

Connection and Channel initialization in IBC is a 4 step handshake process involving an Init Try Ack and Confirm step. It is possible to experience a failure at any step in the process which leaves the handshake process unfinished. Currently in the relayer we assume the handshake is always starting at the Init step but as a user it is desirable to have a way to finish a partially completed handshake.

Ex: I start to open a connection between two IBC enabled chains. Due to network conditions my attempts to initialize the connection fail and the connection is stuck at Try, I don't want to create a whole new connection but would rather finish the handshake for this particular connection. I would like to specify the connection identifier and the current connection state i.e. Try so that the relayer will pick the handshake up from there and finish it.

One way to do this would be to accept an optional conn/channel identifier and if present the relayer will query the connection or channel state and then finish the handshake process. This would remove the need for the user to know which state the connection or channel is stuck in so they would just pass in the identifier and the relayer would figure the rest out.

@jtieri jtieri added T: Enhancement TYPE: Enhancement C: CLI COMPONENT: Command-Line Interface labels Mar 15, 2023
@jonathanpberger
Copy link

Good idea, but not feasible atm; related to this PR: #1147. Iceboxing for now.

@migueldingli1997
Copy link

This would be very helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: CLI COMPONENT: Command-Line Interface T: Enhancement TYPE: Enhancement
Projects
None yet
Development

No branches or pull requests

3 participants