Skip to content

Comments

synchronous connections#3

Open
laerling wants to merge 6 commits intomasterfrom
connections_synchronous
Open

synchronous connections#3
laerling wants to merge 6 commits intomasterfrom
connections_synchronous

Conversation

@laerling
Copy link
Contributor

@laerling laerling commented Oct 21, 2018

Like stated in #2, we don't need asynchronous messaging for now.
Though, there are still a few todos for synchronous messaging:

  • Use [u8] instead of [u8; 1024]
  • If the incoming message is core.pub, handle it before the originally expected message

- Use [u8; 1024] for now
- Better error reporting when socket not found
- Use correct vt6 path (changed when vt6tokio was introduced)
@laerling laerling mentioned this pull request Oct 21, 2018
@laerling laerling force-pushed the connections_synchronous branch from d230864 to a75f0bf Compare October 30, 2018 18:08
@laerling laerling force-pushed the connections_synchronous branch from 09817a8 to 3c91681 Compare October 30, 2018 18:49
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 this pull request may close these issues.

1 participant