-
Notifications
You must be signed in to change notification settings - Fork 48
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.
This looks good to me! Just a small detail regarding the delegated router that should be used.
and nice work with the API improvements!
There are several ways to give in-browser IPFS nodes a better experience on the network, one of those is [Delegate Nodes](https://blog.ipfs.io/2019-08-06-js-ipfs-0-37/#delegated-peer-and-content-routing). A Delegate Node is a network peer that performs certain actions on behalf of other nodes on the network. In this case it will make [DHT][] queries on our behalf so we can find more peers and more content than ever before. | ||
|
||
`js-IPFS@0.48.0` enables delegate nodes in the configuration by default. That means you should see far more peers that you have previously and be able to find content faster and more reliably than ever before. | ||
|
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.
We should mention that the default config uses public delegate nodes, but in a production environment other delegate nodes should be setup to guarantee availability and avoid congestion. I have in my TODO list to create some documentation around libp2p infrastructure/deployment guidelines libp2p/js-libp2p#696 that will be useful for this. But meanwhile, I think we can just say that people should configure their own as we already have some information on how to do that, but it is spread across multiple channels
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.
Done!
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
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.
Added some language corrections and a recommendation to clarify js dht performance expectations.
Co-authored-by: Jacob Heun <jacobheun@gmail.com>
Adds post for the js-IPFS release today