mailbox: change Websocket close status

Javascript errors are thrown on websocket connection close if the
StatusGoingAway code is used. Things work if the StatusNormalClosure
code is used instead.

Author: ellemouton

mailbox/client_conn.go

@@ -537,17 +537,25 @@ func (c *ClientConn) Close() error {
 		if c.receiveSocket != nil {
 			log.Debugf("sending bye on receive socket")
 			returnErr = c.receiveSocket.Close(
-				websocket.StatusGoingAway, "bye",
+				websocket.StatusNormalClosure, "bye",
 			)
+			if returnErr != nil {
+				log.Errorf("Error closing receive socket: %v",
+					returnErr)
+			}
 		}
 		c.receiveStreamMu.Unlock()
 
 		c.sendStreamMu.Lock()
 		if c.sendSocket != nil {
 			log.Debugf("sending bye on send socket")
 			returnErr = c.sendSocket.Close(
-				websocket.StatusGoingAway, "bye",
+				websocket.StatusNormalClosure, "bye",
 			)
+			if returnErr != nil {
+				log.Errorf("Error closing send socket: %v",
+					returnErr)
+			}
 		}
 		c.sendStreamMu.Unlock()