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

Connections #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Connections #2

wants to merge 4 commits into from

Conversation

laerling
Copy link
Contributor

@laerling laerling commented Oct 4, 2018

We don't not need tokio just yet, because we don't subscribe to properties for now.
As soon as properties come into play, synchronous messaging will be okay for prototyping.
That is, it will suffice at first to only check for incoming messages when expecting an answer for a sent message and then, if the incoming message is core.pub to handle that before handling the originally expected message.

- connection.rs: struct Connection and associated methods
- main.rs: Prototype for primitive core1.0 negotiation
Not only message streams, but also the I/O stream has to go through this struct,
because in mutliplexed mode it combines the both streams.
@laerling laerling mentioned this pull request Oct 21, 2018
2 tasks
@laerling
Copy link
Contributor Author

laerling commented Oct 21, 2018

I will probably delete this branch in favor of #3 and go make messaging asynchonous from there.

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