-
Notifications
You must be signed in to change notification settings - Fork 999
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
Public gossipsub api #1677
Public gossipsub api #1677
Conversation
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. I only suggested some smaller grammatical changes.
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.
Looks good to me other than the suggestions from Toralf.
1982242
to
507df6b
Compare
687b4f2
to
9d70260
Compare
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.
Would you mind adding an entry to the gossipsub CHANGELOG
file before we merge? Sorry for all the back-and-forth.
9d70260
to
7738745
Compare
…bsub api Basically, querying current topics and mesh for each topic.
Co-authored-by: Toralf Wittner <[email protected]>
Co-authored-by: Max Inden <[email protected]>
d4f4de0
to
fcfbbec
Compare
@rklaehn would you mind either merging current master or allowing us to commit to your branch? |
Merged. Sorry for the slow responses - have been on a road trip through eastern Europe the last two weeks, only checking occasionally. |
Add some public api to allow implementing the ipfs pubsub api
Basically
-pubsub ls https://docs.ipfs.io/reference/http/api/#api-v0-pubsub-ls
lists topics that we are currently listening to
-pubsub peers https://docs.ipfs.io/reference/http/api/#api-v0-pubsub-peers
lists peers we are peered to for a single topic, or overall