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

Questions and issues around integration into an asynchronous environment #180

Open
drewstone opened this issue Sep 1, 2022 · 1 comment

Comments

@drewstone
Copy link

Issue

One of the main issues we are having in our environment seems to stem from the fact that peers in our network start the protocol at different times. Since they start at different times, they send the initial messages at different times. It just so happens that we think messages are received by other peers before they have initialized the relevant structs for handling those messages and, as such, those messages go unprocessed. We are considering adding a queue-based cache for each participant, but we are curious if there are more ideas/recommendations @DmytroTym @survived?

Questions

  1. Does the system process out-of-order messages at all or does it expect protocol to work sequentially.
  2. Does the library provide replication over messages that have that been ACK'd effectively? For example, a peer has sent a message and has not been able to proceed in the round so it can send that same message again?
@leontiad
Copy link

leontiad commented Dec 9, 2022

Great comments @drewstone. The crate does not provide networking stack it is only the cryptographic codebase. So it is up to the user to implement it. Sees those comments are more relevant for the rounds-based-protocol or the Gotham city. In general assuming there is a message to be signed it will simplify a lot verifying at the beginning that there are those n peers live and active that will collaborate to sign the message. There should be some coordination that all those peers are up and running. I guess those two questions are more relevant with those lacking issues ZenGo-X/round-based-protocol#3

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

No branches or pull requests

2 participants