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

Commit

Permalink
fix: create HTTP servers in series
Browse files Browse the repository at this point in the history
In Windows this consistently fails because of port collisions when using ethemeral ports.

License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
  • Loading branch information
Alan Shaw committed Aug 22, 2019
1 parent 76e7a51 commit 4e750c6
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/http/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,20 +23,20 @@ function hapiInfoToMultiaddr (info) {
return toMultiaddr(uri)
}

function serverCreator (serverAddrs, createServer, ipfs) {
async function serverCreator (serverAddrs, createServer, ipfs) {
serverAddrs = serverAddrs || []
// just in case the address is just string
serverAddrs = Array.isArray(serverAddrs) ? serverAddrs : [serverAddrs]

const processServer = async address => {
const servers = []
for (const address of serverAddrs) {
const addrParts = address.split('/')
const server = await createServer(addrParts[2], addrParts[4], ipfs)
await server.start()
server.info.ma = hapiInfoToMultiaddr(server.info)
return server
servers.push(server)
}

return Promise.all(serverAddrs.map(processServer))
return servers
}

class HttpApi {
Expand Down

0 comments on commit 4e750c6

Please sign in to comment.