Skip to content

Commit 91e60d4

Browse files
thomaseizingerjacobheun
authored andcommitted
feat: prepare for new randomWalk config parameters
As per: libp2p/js-libp2p-kad-dht#76 fix: pass whole dht config into DHT constructor
1 parent 679d446 commit 91e60d4

File tree

5 files changed

+25
-10
lines changed

5 files changed

+25
-10
lines changed

src/config.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,12 @@ const OptionsSchema = Joi.object({
3434
dht: Joi.object().keys({
3535
kBucketSize: Joi.number().default(20),
3636
enabled: Joi.boolean().default(true),
37-
enabledDiscovery: Joi.boolean().default(true),
37+
randomWalk: Joi.object().keys({
38+
enabled: Joi.boolean().default(true),
39+
queriesPerPeriod: Joi.number().default(1),
40+
interval: Joi.number().default(30000),
41+
timeout: Joi.number().default(10000)
42+
}).default(),
3843
validators: Joi.object().allow(null),
3944
selectors: Joi.object().allow(null)
4045
}).default(),

src/index.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -105,14 +105,10 @@ class Node extends EventEmitter {
105105
// dht provided components (peerRouting, contentRouting, dht)
106106
if (this._config.dht.enabled) {
107107
const DHT = this._modules.dht
108-
const enabledDiscovery = this._config.dht.enabledDiscovery !== false
109108

110109
this._dht = new DHT(this._switch, {
111-
kBucketSize: this._config.dht.kBucketSize,
112-
enabledDiscovery,
113110
datastore: this.datastore,
114-
validators: this._config.dht.validators,
115-
selectors: this._config.dht.selectors
111+
...this._config.dht
116112
})
117113
}
118114

test/config.spec.js

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,12 @@ describe('configuration', () => {
9696
dht: {
9797
kBucketSize: 20,
9898
enabled: true,
99-
enabledDiscovery: true
99+
randomWalk: {
100+
enabled: true,
101+
queriesPerPeriod: 1,
102+
interval: 30000,
103+
timeout: 10000
104+
}
100105
},
101106
relay: {
102107
enabled: true
@@ -185,7 +190,12 @@ describe('configuration', () => {
185190
dht: {
186191
kBucketSize: 20,
187192
enabled: true,
188-
enabledDiscovery: true,
193+
randomWalk: {
194+
enabled: true,
195+
queriesPerPeriod: 1,
196+
interval: 30000,
197+
timeout: 10000
198+
},
189199
selectors,
190200
validators
191201
}

test/utils/bundle-browser.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,9 @@ class Node extends libp2p {
8080
},
8181
dht: {
8282
kBucketSize: 20,
83-
enabledDiscovery: true,
83+
randomWalk: {
84+
enabled: true
85+
},
8486
enabled: false
8587
},
8688
EXPERIMENTAL: {

test/utils/bundle-nodejs.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,9 @@ class Node extends libp2p {
7373
},
7474
dht: {
7575
kBucketSize: 20,
76-
enabledDiscovery: true,
76+
randomWalk: {
77+
enabled: true
78+
},
7779
enabled: true
7880
},
7981
EXPERIMENTAL: {

0 commit comments

Comments
 (0)