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

Tracking Issue: Migrate transport level logic into backroll-transport #9

Open
5 tasks
james7132 opened this issue Jun 1, 2021 · 0 comments
Open
5 tasks
Labels
breaking change This will require at least a minor version bump

Comments

@james7132
Copy link
Member

There are a notable amount of the application level protocol that belongs in a lower level implementation. The heartbeat, the initial synchronization, the magic number management, sequence number, and disconnect timeout are all to establish a unreliable but ordered connection, something Laminar and other reliable UDP implementations already support. This should exist at the backroll-transport layer. A good amount of this is already established in TimonPost/laminar#290, and this change be considered a step towards implementing backroll-transport's connection model in Laminar.

  • Create a wrapper around Peer for ordered unreliable sends. OrderedPeer?
  • Add a way to notify the main game of incoming connections.
  • Alter Peer to notify either end of a network interruption.
  • Move "Synchronizing, Running, Interrupted, Disconnected" state machine to the transport layer.
  • Move heartbeat task into OrderedPeer
@james7132 james7132 added the breaking change This will require at least a minor version bump label Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change This will require at least a minor version bump
Projects
None yet
Development

No branches or pull requests

1 participant