chore: throw error no swarm on multiaddrs using websocket-star

packages/ipfs/src/core/components/start.js

@@ -11,6 +11,8 @@ const Components = require('./')
 const createMfsPreload = require('../mfs-preload')
 const { withTimeoutOption } = require('../utils')
 
+const WEBSOCKET_STAR_PROTO_CODE = 479
+
 module.exports = ({
   apiManager,
   options: constructorOptions,
@@ -40,6 +42,12 @@ module.exports = ({
       config.Addresses.Swarm.forEach(addr => {
         let ma = multiaddr(addr)
 
+        // Temporary error for users migrating using websocket-star multiaddrs for listenning on libp2p
+        // websocket-star support was removed from ipfs and libp2p
+        if (ma.protoCodes().includes(WEBSOCKET_STAR_PROTO_CODE)) {
+          throw new Error('websocket-star swarm addresses are not supported. See https://github.com/ipfs/js-ipfs/issues/2779')
+        }
+
         // multiaddrs that go via a signalling server or other intermediary (e.g. stardust,
         // webrtc-star) can have the intermediary's peer ID in the address, so append our
         // peer ID to the end of it
@@ -130,7 +138,6 @@ module.exports = ({
     apiManager.update(api, () => undefined)
   } catch (err) {
     cancel()
-    startPromise.reject(err)
     throw err
   }
 

packages/ipfs/test/core/create-node.spec.js

@@ -250,4 +250,28 @@ describe('create node', function () {
 
     await node.stop()
   })
+
+  it('should error when receiving websocket-star swarm addresses', async () => {
+    const node = await IPFS.create({
+      repo: tempRepo,
+      init: { bits: 512 },
+      start: false,
+      config: {
+        Addresses: {
+          Swarm: ['/ip4/127.0.0.1/tcp/13579/wss/p2p-websocket-star']
+        },
+        Bootstrap: []
+      },
+      preload: { enabled: false }
+    })
+
+    try {
+      await node.start()
+    } catch (err) {
+      expect(err).to.exist()
+      return
+    }
+
+    throw new Error('should error when receiving websocket-star swarm addresses')
+  })
 })