move readLoop to goroutine to avoid "too many open files"

build-all.sh

@@ -107,7 +107,7 @@ do
   done
 done
 
-# Need to rebuild lthe inux-rethink binary without stripping debug info.
+# Need to rebuild the linux-rethink binary without stripping debug info.
 echo "Building the binary for the demo at api.tinode.co"
 
 rm -f $GOPATH/bin/tinode

loadtest/tinode.erl

@@ -1,6 +1,8 @@
 %% Support module for Tinode load testing with Tsung.
 %% Compile using erlc then copy resulting .beam to
 %% /usr/local/lib/erlang/lib/tsung-1.7.0/ebin/
+%% Alternatively you can just leave it in the current
+%% directory.
 
 -module(tinode).
 -export([rand_user_secret/1]).

server/hdl_websock.go

@@ -54,7 +54,9 @@ func (sess *Session) readLoop() {
 		// Read a ClientComMessage
 		_, raw, err := sess.ws.ReadMessage()
 		if err != nil {
-			log.Println("sess.readLoop: " + err.Error())
+			if websocket.IsUnexpectedCloseError(err, websocket.CloseGoingAway, websocket.CloseAbnormalClosure) {
+				log.Println("sess.readLoop: " + err.Error())
+			}
 			return
 		}
 
@@ -148,6 +150,9 @@ func serveWebSocket(wrt http.ResponseWriter, req *http.Request) {
 
 	sess := globals.sessionStore.Create(ws, "")
 
+	// Do work in goroutines to return from serveWebSocket() to release file pointers.
+	// Otherwise "too many open files" will happen.
+
 	go sess.writeLoop()
-	sess.readLoop()
+	go sess.readLoop()
 }