-
Notifications
You must be signed in to change notification settings - Fork 36
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
Expose the Peerstore #89
Comments
I am not too excited about that. |
Sorry but i can't see any use-cases for |
The protocols are discovered and added to the Peer Store during Identify, there's just currently no trivial way for us to get those known protocols from the Daemon. Right now the only access we have to the Peer Store is LIST_PEERS. I'd like to be able to get this information from the Daemon about a given peer.
@vyzo is it the usage in LIST_PEERS, or exposing the protocols period? |
You can use identify directly on a peer. |
Unless I am missing something, the only way to currently use identify directly on a peer would be to implement the identify protocol in a daemon client and perform that on a connected peer.
|
It's trivial, you just open a stream and read the identify response. Having said that, we probably want to expose the Peerstore where such information can be obtained. |
Exposing the Peerstore would be ideal. I can create a PR for the proto and Spec additions. |
Expose the Peerstore to allow daemon clients to access that information, such as known protocols a peer supports.
I'm proposing an addition of protos to the PeerInfo protobuf. This would enable us to expose the known protocols of a peer to consuming clients during the LIST_PEERS request.The text was updated successfully, but these errors were encountered: