-
Notifications
You must be signed in to change notification settings - Fork 929
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
protocols/rendezvous: Improve examples #2229
Conversation
* Add support for the `Identify` protocol to the server, such that the `register_with_identify` example works as intended * Add discovery loop to the `discovery` example and demonstrate cookie usage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
is mxinden still the only one with write access to the repo? maybe it's time to add @thomaseizinger |
Thanks for the nomination! There is an ongoing discussion about becoming a maintainer, should be sorted once @mxinden comes back from vacation! |
|
||
loop { | ||
tokio::select! { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
futures::select
would be more generic, presumably this is just a reexport
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not, the implementation is in fact very different: tokio-rs/tokio@8cf98d6
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's in my toolbox right now. Didn't bother trying to be runtime agnostic, as the example already used stuff from tokio.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @wngr!
Identify
protocol to the server, such that theregister_with_identify
example works as intendeddiscovery
example and demonstrate cookieusage