-
Notifications
You must be signed in to change notification settings - Fork 1.2k
IPNS Publish Does not work in JS-IPFS 0.54.2 #3584
Comments
Thank you for submitting your first issue to this repository! A maintainer will be here shortly to triage and review.
Finally, remember to use https://discuss.ipfs.io if you just need general support. |
In addition, did force |
Also, tried setting |
Hey @TJKoury |
I have not, it’s next in my list. I have tried everything to be able to
use it without actually changing any code; it’s safe to say that by
“experiment” you mean making code changes, I’m assuming to lib-p2p?
…On Tue, Mar 9, 2021 at 8:37 AM Vasco Santos ***@***.***> wrote:
Hey @TJKoury <https://github.com/TJKoury>
The js DHT implementation needs some work to be fully functional and it is
disabled by default for now as the reasoning. People can experiment with
it, but not recommended for production yet.
Did you try using IPNS over Pubsub instead?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3584 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFW4KA4UITGBLXGGIJOABLTCYQBJANCNFSM4Y2GNV7A>
.
|
You can have a look in this: https://github.com/ipfs/js-ipfs/tree/master/examples/browser-ipns-publish (it is in the browser though)
experiment I mean to try out stuff using the current implementation by manually enabling it like you mentioned. We need to get the DHT updated like go-libp2p DHT was last year and get it properly tested in a larger scale before recommending its usage and eventually have it enabled by default. Anyway, the insights I have from the go team is that IPNS is performing way better when using pubsub (they also have added a pubsub fallback that we need to add in JS) |
Yup, already checked that out and will try today. Good to hear about the
performance, makes sense when switching to a reactive model.
…On Tue, Mar 9, 2021 at 9:17 AM Vasco Santos ***@***.***> wrote:
I have not, it’s next in my list. I have tried everything to be able to
You can have a look in this:
https://github.com/ipfs/js-ipfs/tree/master/examples/browser-ipns-publish
(it is in the browser though)
“experiment” you mean making code changes
experiment I mean to try out stuff using the current implementation by
manually enabling it like you mentioned. We need to get the DHT updated
like go-libp2p DHT was last year and get it properly tested in a larger
scale before recommending its usage and eventually have it enabled by
default.
Anyway, the insights I have from the go team is that IPNS is performing
way better when using pubsub (they also have added a pubsub fallback that
we need to add in JS)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3584 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFW4KG2W45MPRWVACCVU63TCYUW3ANCNFSM4Y2GNV7A>
.
|
FYI, due to this issue, need to downgrade parcel-bundler to |
API connection works ok, websockets error |
Still not working, using the same settings in the example folder. Before I deconstruct the entire example for debugging, @vasco-santos anything obvious I'm missing? |
Test still failing on fresh install.
|
There is no transport available for that address mean the transports you have configured do not support tcp. Browser does not support TCP, so you will need to configure websockets on your "Addresses": {
"Swarm": [
"/ip4/0.0.0.0/tcp/4001",
"/ip4/0.0.0.0/tcp/4010/ws", This will mean that |
So does this mean the example is not supposed to run out of the box? If I get it working by changing the config in the test file using the |
The example has instructions for setting up the websocket transport on https://github.com/ipfs/js-ipfs/tree/master/examples/browser-ipns-publish#install-and-start-the-go-ipfs-node |
@vasco-santos Talking about the example in the folder that has this in the config:
Still failing with this config. |
Even putting in hard-coded ports instead of '0' gets the same error with those ports:
From what I can tell the example just does not work. |
Ok, so I imagine that when you start your go node now, you can see it is listening on websockets, is this correct? There is also a problem on the JS side. Apologies for not going directly into this particularity, but I did not realize we did not update this in this example. The websockets transport is not being used and the problem is related to the usage of a local address. Long story short, browser policies only allow dials when using WSS+DNS addresses, which is a pain because you need to configure that yourself on the go node. More context: protocol/web3-dev-team#70 With the above in mind, we blocked these types of dial by default in libp2p. For local experimenting, you can have this changed. You can check how to do this here. This basically allows localhost experimentation. You will probably need to use a localhost address for the If you are interested in updating the IPNS example with this configuration, it would be super helpful |
Thanks for all the tips! I got it working using all of the above plus referencing the go node (2nd step) with
since it seemed to be hanging with /wss. Results: |
Applying all the magic from #3584, updating the dependencies, adding tips (address example) for working with a local node Co-authored-by: Vasco Santos <vasco.santos@ua.pt> Co-authored-by: Alen Šiljak <code@alensiljak.eu.org> Co-authored-by: achingbrain <alex@achingbrain.net>
Publish doesn't work (
jsipfs name publish $(CID)
) using Ubuntu 18.04 LTS, Node 15, js-ipfs 0.54.2.Running two physical servers, both can see each other (
jsipfs swarm peers |grep $(peerID)
), both running RSA keys, both can resolve the IPFS CID for the content, but both reportrecord requested for $(IPNS_hash) was not found on the network.
Both can resolve their own IPNS hash.
Go-IPFS can also see the peers but fails after about 30 seconds with the go-equivalent error
Error: could not resolve name
.I have not tested any other version of js-ipfs.
Originally posted by @TJKoury in ipfs/kubo#7965 (comment)
The text was updated successfully, but these errors were encountered: