Skip to content
This repository was archived by the owner on Aug 2, 2021. It is now read-only.

add notes about http-api ux #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add notes about http-api ux #4

wants to merge 1 commit into from

Conversation

daviddias
Copy link
Member

No description provided.

@daviddias
Copy link
Member Author

@whyrusleeping

@whyrusleeping
Copy link
Contributor

👍 needs @jbenet's review though

@jbenet
Copy link
Contributor

jbenet commented Sep 14, 2016

What does this look like in CLI UX? they need to match.

Some comments up-front:

  • subscribe should not be able to be left open. it should always cancel without a socket ramining open, or continued requests from the user
  • We can add route as a subcommand that tells the node to add a subscription and route the messages, but does not count as a "client's intent to subscribe", only that the node should behave as a router in the network. This state should be persistent and durable (i.e. changes config to disk, works across reboots)
  • We are getting pubsub ls subcommand ipfs/kubo#3223 and pubsub peers subcommand ipfs/kubo#3224 -- does that satisfy some of these needs ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants