fix: return on call to nextMuxer

alanshaw · Jacob Heun · commit fef2d1185037 · 2018-05-30T19:49:03.000+02:00
When the call to multistream.Dialer.select is unsuccessful, call nextMuxer to try select the next one in the list but do not continue executing callback afterwards.

License: MIT
Signed-off-by: Alan Shaw &lt;alan@tableflip.io&gt;
diff --git a/src/dial.js b/src/dial.js
@@ -208,7 +208,7 @@ class Dialer {
             return callback(new Error('could not upgrade to stream muxing'))
           }
 
-          nextMuxer(muxers.shift())
+          return nextMuxer(muxers.shift())
         }
 
         const muxedConn = this.switch.muxers[key].dialer(conn)

Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,7 @@ class Dialer {`
`208`	`208`	`return callback(new Error('could not upgrade to stream muxing'))`
`209`	`209`	`}`
`210`	`210`
`211`		`- nextMuxer(muxers.shift())`
	`211`	`+ return nextMuxer(muxers.shift())`
`212`	`212`	`}`
`213`	`213`
`214`	`214`	`const muxedConn = this.switch.muxers[key].dialer(conn)`