Merge pull request #214 from kaleido-io/fix-concurrent-map-close

Fix concurrent map panic in webSocketServer.Close()

Author: peterbroadhurst

pkg/wsserver/wsserver.go

@@ -127,7 +127,13 @@ func (s *webSocketServer) connectionClosed(c *webSocketConnection) {
 }
 
 func (s *webSocketServer) Close() {
+	s.mux.Lock()
+	conns := make([]*webSocketConnection, 0, len(s.connections))
 	for _, c := range s.connections {
+		conns = append(conns, c)
+	}
+	s.mux.Unlock()
+	for _, c := range conns {
 		c.close()
 	}
 }