Skip to content

Commit 6b434ae

Browse files
committed
chore: update websockets (#806)
* chore: update websockets
1 parent 44f0703 commit 6b434ae

File tree

5 files changed

+68
-7
lines changed

5 files changed

+68
-7
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@
109109
"libp2p-secio": "^0.13.1",
110110
"libp2p-tcp": "^0.15.1",
111111
"libp2p-webrtc-star": "^0.20.0",
112-
"libp2p-websockets": "^0.14.0",
112+
"libp2p-websockets": "^0.15.0",
113113
"multihashes": "^3.0.1",
114114
"nock": "^13.0.3",
115115
"p-defer": "^3.0.0",

test/dialing/direct.spec.js

Lines changed: 52 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ const pDefer = require('p-defer')
77
const pWaitFor = require('p-wait-for')
88
const delay = require('delay')
99
const Transport = require('libp2p-websockets')
10+
const filters = require('libp2p-websockets/src/filters')
1011
const Muxer = require('libp2p-mplex')
1112
const { NOISE: Crypto } = require('libp2p-noise')
1213
const multiaddr = require('multiaddr')
@@ -41,7 +42,7 @@ describe('Dialing (direct, WebSockets)', () => {
4142
upgrader: mockUpgrader,
4243
onConnection: () => {}
4344
})
44-
localTM.add(Transport.prototype[Symbol.toStringTag], Transport)
45+
localTM.add(Transport.prototype[Symbol.toStringTag], Transport, { filter: filters.all })
4546
})
4647

4748
afterEach(() => {
@@ -292,6 +293,7 @@ describe('Dialing (direct, WebSockets)', () => {
292293
})
293294

294295
describe('libp2p.dialer', () => {
296+
const transportKey = Transport.prototype[Symbol.toStringTag]
295297
let libp2p
296298

297299
afterEach(async () => {
@@ -307,6 +309,13 @@ describe('Dialing (direct, WebSockets)', () => {
307309
transport: [Transport],
308310
streamMuxer: [Muxer],
309311
connEncryption: [Crypto]
312+
},
313+
config: {
314+
transport: {
315+
[transportKey]: {
316+
filter: filters.all
317+
}
318+
}
310319
}
311320
})
312321

@@ -330,6 +339,13 @@ describe('Dialing (direct, WebSockets)', () => {
330339
maxParallelDials: 10,
331340
maxDialsPerPeer: 1,
332341
dialTimeout: 1e3 // 30 second dial timeout per peer
342+
},
343+
config: {
344+
transport: {
345+
[transportKey]: {
346+
filter: filters.all
347+
}
348+
}
333349
}
334350
}
335351
libp2p = await Libp2p.create(config)
@@ -347,6 +363,13 @@ describe('Dialing (direct, WebSockets)', () => {
347363
transport: [Transport],
348364
streamMuxer: [Muxer],
349365
connEncryption: [Crypto]
366+
},
367+
config: {
368+
transport: {
369+
[transportKey]: {
370+
filter: filters.all
371+
}
372+
}
350373
}
351374
})
352375

@@ -370,6 +393,13 @@ describe('Dialing (direct, WebSockets)', () => {
370393
transport: [Transport],
371394
streamMuxer: [Muxer],
372395
connEncryption: [Crypto]
396+
},
397+
config: {
398+
transport: {
399+
[transportKey]: {
400+
filter: filters.all
401+
}
402+
}
373403
}
374404
})
375405

@@ -397,6 +427,13 @@ describe('Dialing (direct, WebSockets)', () => {
397427
transport: [Transport],
398428
streamMuxer: [Muxer],
399429
connEncryption: [Crypto]
430+
},
431+
config: {
432+
transport: {
433+
[transportKey]: {
434+
filter: filters.all
435+
}
436+
}
400437
}
401438
})
402439

@@ -414,6 +451,13 @@ describe('Dialing (direct, WebSockets)', () => {
414451
transport: [Transport],
415452
streamMuxer: [Muxer],
416453
connEncryption: [Crypto]
454+
},
455+
config: {
456+
transport: {
457+
[transportKey]: {
458+
filter: filters.all
459+
}
460+
}
417461
}
418462
})
419463

@@ -427,6 +471,13 @@ describe('Dialing (direct, WebSockets)', () => {
427471
transport: [Transport],
428472
streamMuxer: [Muxer],
429473
connEncryption: [Crypto]
474+
},
475+
config: {
476+
transport: {
477+
[transportKey]: {
478+
filter: filters.all
479+
}
480+
}
430481
}
431482
})
432483

test/dialing/resolver.spec.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,12 @@ describe('Dialing (resolvable addresses)', () => {
3737
[libp2p, remoteLibp2p] = await peerUtils.createPeer({
3838
number: 2,
3939
config: {
40-
modules: baseOptions.modules,
40+
...baseOptions,
4141
addresses: {
4242
listen: [multiaddr(`${relayAddr}/p2p-circuit`)]
4343
},
4444
config: {
45+
...baseOptions.config,
4546
peerDiscovery: {
4647
autoDial: false
4748
}

test/transports/transport-manager.spec.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ const sinon = require('sinon')
66

77
const multiaddr = require('multiaddr')
88
const Transport = require('libp2p-websockets')
9+
const filters = require('libp2p-websockets/src/filters')
910
const { NOISE: Crypto } = require('libp2p-noise')
1011
const AddressManager = require('../../src/address-manager')
1112
const TransportManager = require('../../src/transport-manager')
@@ -39,7 +40,7 @@ describe('Transport Manager (WebSockets)', () => {
3940
})
4041

4142
it('should be able to add and remove a transport', async () => {
42-
tm.add(Transport.prototype[Symbol.toStringTag], Transport)
43+
tm.add(Transport.prototype[Symbol.toStringTag], Transport, { filter: filters.all })
4344
expect(tm._transports.size).to.equal(1)
4445
await tm.remove(Transport.prototype[Symbol.toStringTag])
4546
})
@@ -66,23 +67,23 @@ describe('Transport Manager (WebSockets)', () => {
6667
})
6768

6869
it('should be able to dial', async () => {
69-
tm.add(Transport.prototype[Symbol.toStringTag], Transport)
70+
tm.add(Transport.prototype[Symbol.toStringTag], Transport, { filter: filters.all })
7071
const addr = MULTIADDRS_WEBSOCKETS[0]
7172
const connection = await tm.dial(addr)
7273
expect(connection).to.exist()
7374
await connection.close()
7475
})
7576

7677
it('should fail to dial an unsupported address', async () => {
77-
tm.add(Transport.prototype[Symbol.toStringTag], Transport)
78+
tm.add(Transport.prototype[Symbol.toStringTag], Transport, { filter: filters.all })
7879
const addr = multiaddr('/ip4/127.0.0.1/tcp/0')
7980
await expect(tm.dial(addr))
8081
.to.eventually.be.rejected()
8182
.and.to.have.property('code', ErrorCodes.ERR_TRANSPORT_UNAVAILABLE)
8283
})
8384

8485
it('should fail to listen with no valid address', async () => {
85-
tm.add(Transport.prototype[Symbol.toStringTag], Transport)
86+
tm.add(Transport.prototype[Symbol.toStringTag], Transport, { filter: filters.all })
8687

8788
await expect(tm.listen([listenAddr]))
8889
.to.eventually.be.rejected()

test/utils/base-options.browser.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,12 @@
11
'use strict'
22

33
const Transport = require('libp2p-websockets')
4+
const filters = require('libp2p-websockets/src/filters')
45
const Muxer = require('libp2p-mplex')
56
const { NOISE: Crypto } = require('libp2p-noise')
67

8+
const transportKey = Transport.prototype[Symbol.toStringTag]
9+
710
module.exports = {
811
modules: {
912
transport: [Transport],
@@ -16,6 +19,11 @@ module.exports = {
1619
hop: {
1720
enabled: false
1821
}
22+
},
23+
transport: {
24+
[transportKey]: {
25+
filter: filters.all
26+
}
1927
}
2028
}
2129
}

0 commit comments

Comments
 (0)