-
Notifications
You must be signed in to change notification settings - Fork 273
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
fix: rendezvous spec #284
fix: rendezvous spec #284
Conversation
Thoughts @vyzo @jacobheun ? |
Yeah, let's just use signed peer records. |
rendezvous/README.md
Outdated
@@ -204,6 +204,7 @@ message Message { | |||
optional string ns = 1; | |||
optional PeerInfo peer = 2; | |||
optional int64 ttl = 3; // in seconds | |||
optional bytes signedPeerRecord = 4; |
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.
Yeah, drop PeerInfo for signedPeerRecord. Also, this spec needs to point to the SignedPeerRecord spec to clarify what bytes signedPeerRecord
actually is.
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.
Yeah, should we get the latest recommendations in and get the signed peer record PR merged? #217
This PR adds the SignedPeerRecord to the Register message.
Since this is still a Working Draft spec, there are a few things that I would like to suggest:
PeerInfo
message and just use theSignedPeerRecord