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

Support for optimistic channel handshake initiation from the state machine #2910

Closed
1 of 5 tasks
ancazamfir opened this issue Nov 29, 2022 · 0 comments · Fixed by #2919
Closed
1 of 5 tasks

Support for optimistic channel handshake initiation from the state machine #2910

ancazamfir opened this issue Nov 29, 2022 · 0 comments · Fixed by #2919
Assignees
Labels
O: new-feature Objective: cause to add a new feature or support
Milestone

Comments

@ancazamfir
Copy link
Collaborator

ancazamfir commented Nov 29, 2022

Summary

From Aditya's message:

[...] want to flag a potential issue that may arise in the medium-term future.
One thing i see emerging is the handshake INIT messages are more and more being sent by the state machine itself (e.g. ICS). afaik, the relayers are equipped to handle this when its a channel handshake or connection handshake

but I am not sure if this works if the state machine is starting both handshakes at once (this is allowed)

in this case, the relayer should complete the connection handshake before it sends the ChanOpenTry which will expect the underlying connection to be OPEN

Problem Definition

This combination needs to be tested as we do not have official support for it. Any problems need to be fixed.

Proposal

tbd

Acceptance Criteria

tbd


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@ancazamfir ancazamfir self-assigned this Nov 29, 2022
@adizere adizere added this to the v1.3 milestone Dec 6, 2022
@adizere adizere added the P-high label Dec 6, 2022
@seanchen1991 seanchen1991 added the O: new-feature Objective: cause to add a new feature or support label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O: new-feature Objective: cause to add a new feature or support
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

3 participants