Updating pinning server, swarm and bootstrap nodes #91
Conversation
| '/dns4/wss0.bootstrap.libp2p.io/tcp/443/wss/ipfs/QmZMxNdpMkewiVZLMRxaNxUeZpDUb34pWjZ1kZvsd16Zic', | ||
| '/dns4/wss1.bootstrap.libp2p.io/tcp/443/wss/ipfs/Qmbut9Ywz9YEDrz8ySBSgWyJk41Uvm2QJPhwDJzJyGFsD6', | ||
| '/dns4/bootstrap.osliki.net/tcp/443/wss/ipfs/QmfJB77qXfiEdJkaSxpZgiMh9kAPiDBj3ga7TxF72QdWtf', | ||
| '/p2p-circuit/ipfs/QmfJB77qXfiEdJkaSxpZgiMh9kAPiDBj3ga7TxF72QdWtf' |
There was a problem hiding this comment.
Do we need the bootstrap nodes? Why did you add them here?
There was a problem hiding this comment.
I preferred to whitelist them because one of them was throwing connection error (nyc-2 I believe)
There was a problem hiding this comment.
Oh, right. By default there are bootstrap nodes, but if we set this to an empty array we should not connect to any I think. see #82
| Addresses: { | ||
| Swarm: [ | ||
| '/dns4/wrtc-star.discovery.libp2p.io/tcp/443/wss/p2p-webrtc-star', | ||
| '/dns4/ws-star.discovery.libp2p.io/tcp/443/wss/p2p-websocket-star' |
There was a problem hiding this comment.
What are these? Looks like some webrtc enablers? Do you know what they do?
There was a problem hiding this comment.
Only the first one seems to be a webrtc enabler. I think it's safe to remove them after testing they work. But I don't find harmful them to be there.
There was a problem hiding this comment.
let's keep for now, we can test in #82
| @@ -11,8 +11,27 @@ const OrbitdbKeyAdapter = require('./orbitdbKeyAdapter') | |||
| const utils = require('./utils') | |||
|
|
|||
| const ADDRESS_SERVER_URL = 'https://beta.3box.io/address-server' | |||
| const PINNING_SERVER = '/ip4/18.236.233.237/tcp/4003/ws/ipfs/QmbgpyTLCYBy84E1HGwei6niQHiLQmRpfN6SQtfJiyNMUd' | |||
| const PINNING_SERVER = '/dnsaddr/ipfs.3box.io/tcp/443/wss/ipfs/QmbgpyTLCYBy84E1HGwei6niQHiLQmRpfN6SQtfJiyNMU' | |||
There was a problem hiding this comment.
How come we are using /dnsaddr rather than /dns4 as the entries below?
There was a problem hiding this comment.
tried with DNS4 and the multiaddr (with wss instead of ws) was invalid. This was the only way I could make it work
There was a problem hiding this comment.
Ok, strange that it works for the other bootstrap nodes 🤔
No description provided.