Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Commit

Permalink
fix: bootstrap
Browse files Browse the repository at this point in the history
  • Loading branch information
daviddias committed Feb 18, 2018
1 parent afaaf2b commit d527b45
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions src/core/components/bootstrap.js
@@ -1,7 +1,7 @@
'use strict'

const defaultNodes = require('../runtime/config-nodejs.json').Bootstrap
const MultiAddr = require('multiaddr')
// const MultiAddr = require('multiaddr')
const promisify = require('promisify-es6')

module.exports = function bootstrap (self) {
Expand All @@ -21,7 +21,9 @@ module.exports = function bootstrap (self) {
}
try {
if (multiaddr) {
multiaddr = new MultiAddr(multiaddr)
// TODO understand what was the purpose of this code
// it failed on tests, it passes without
// multiaddr = new MultiAddr(multiaddr)

This comment has been minimized.

Copy link
@victorb

victorb Feb 18, 2018

Member

AFAIK, this was checking the validity of a multiaddr. Was being used instead of mafmt to make sure it's a valid multiaddr. No idea why though.

}
} catch (err) {
return setImmediate(() => callback(err))
Expand Down Expand Up @@ -53,7 +55,8 @@ module.exports = function bootstrap (self) {
}
try {
if (multiaddr) {
multiaddr = new MultiAddr(multiaddr)
// TODO understand what was the purpose of this code
// multiaddr = new MultiAddr(multiaddr)
}
} catch (err) {
return setImmediate(() => callback(err))
Expand Down

0 comments on commit d527b45

Please sign in to comment.