-
Notifications
You must be signed in to change notification settings - Fork 15
Make the Readme and example clearer, so that it's easier to get running #59
Comments
The readme example/instructions need to be updated. We do host a rendezvous server for demos and experimenting at The Running a rendezvous server, |
@jacobheun Note that |
@mkg20001 ah, that's good to know, thanks! We should make sure to switch the host on examples using local rendezvous servers so we're not referencing that anymore. The rendezvous repo will also need to get updated with this. |
I'm having trouble using js-ipfs in the browser to connect to a remote rendezvous server. Firstly, setting up IPFS like this:
throws an error: Then, even if I just write a simple WebSocket test script:
It tells me |
@aphelionz the error you're hitting is due to the Bootstrap address not being an ipfs address (default addresses for bootstrap are here). Anything in config: {
Addresses: {
Swarm: [
'/dns4/ws-star.discovery.libp2p.io/tcp/443/wss/p2p-websocket-star/'
]
}
} which will have the ipfs node listen to connections from that server. |
@jacobheun thanks for the clarification. Is there anything I can do on the server side so that it's not refusing connections? |
@aphelionz are you getting errors with the updated configuration? Or are you trying to hit it directly with WebSocket script you referenced above? If you're using the script, I'm not sure which library you're using or what the rest of the script looks like, but |
Thank you. So now I have:
That gives me this error in Chrome on Ubuntu: |
On the server side, I'm just running the rendezvous docker container with default settings |
@aphelionz I think that error is related to ipfs/js-ipfs#1620. Do you have other dependencies you're using like |
@jacobheun Should this work?
I receive this validation error:
|
@litzenberger you need to use ’dnsaddr’ instead of ’dns’ for it to work. Also ws-star-signal-1 has been replaced with ws-star.discovery.libp2p.io |
@mkg20001 Thanks. Looks like I'm still missing something. Now I get 'Error: There is already a transport with this key' |
@litzenberger what version of libp2p are you running? There was an issue with this in the browser but this should be fixed in libp2p@0.24.2 ipfs/js-ipfs#1699 (comment) |
@jacobheun thanks, I updated to the latest lib2p2. Does this work in the js-ipfs 0.33.1 version or does it have to run off the master branch? |
It will have to be off master, until 0.34 is released. Hopefully within the next week. |
Can this be closed? |
@mkg20001 not yet, the example in the readme is wrong and needs to be updated. Right now it's using an old libp2p constructor and is just dialing to itself which is going to fail. Ideally I think we should just create an examples folder and have a |
/dns4/ws-star-signal-1.servep2p.com/tcp/443/wss/p2p-websocket-star/
The text was updated successfully, but these errors were encountered: