当客户端断开连接时,node.js websocket崩溃

use*_*728 4 websocket node.js

我很新的NodeJSWebsockets,但我想用它玩.

我所做的是从串口读取传入数据,然后使用websocket将这些数据发送到网页.从这里一切正常.

  • 我使用node-static来为我的网页提供服务
  • 我使用ws作为websocket

问题是当客户端关闭他的浏览器,然后我的NodeJS websocket服务器崩溃时出现以下错误:

root@WS-SERVER-2:~/app# node socketserver.js
open serial communication
Client disconnected.

/root/node-v0.10.29/lib/node_modules/ws/lib/WebSocket.js:187
    else throw new Error('not opened');
               ^
Error: not opened
    at WebSocket.send (/root/node-v0.10.29/lib/node_modules/ws/lib/WebSocket.js:187:16)
    at sendAll (/root/app/socketserver.js:30:16)
    at SerialPort.<anonymous> (/root/app/socketserver.js:58:8)
    at SerialPort.emit (events.js:95:17)
    at Object.module.exports.raw [as parser] (/root/node-v0.10.29/bin/node_modules/serialport/parsers.js:8:13)
    at Object.SerialPort.options.dataCallback (/root/node-v0.10.29/bin/node_modules/serialport/serialport.js:143:15)
    at SerialPortFactory.SerialPort._emitData (/root/node-v0.10.29/bin/node_modules/serialport/serialport.js:312:20)
    at afterRead (/root/node-v0.10.29/bin/node_modules/serialport/serialport.js:290:18)
    at /root/node-v0.10.29/bin/node_modules/serialport/serialport.js:304:9
    at Object.wrapper [as oncomplete] (fs.js:459:17)
Run Code Online (Sandbox Code Playgroud)

这是我的websocket/serialport代码:

var WebSocketServer = require('../node-v0.10.29/lib/node_modules/ws').Server;
var SerialPort = require('../node-v0.10.29/bin/node_modules/serialport').SerialPort;

var serialPort;
var portName = '/dev/ttyACM0';
var sendData = "";
var wss = new WebSocketServer({port: 8080});

var CLIENTS=[];

wss.on('connection', function(ws) {
       CLIENTS.push(ws);
    ws.on('message', function(message) {
        console.log('received: %s', message);
        sendAll(message);
    });
    ws.on('close', function() {
        console.log('Client disconnected.');
    });
    ws.on('error', function() {
        console.log('ERROR');
    });
    ws.send("");
});

function sendAll(message)
{
for(var i=0;i<CLIENTS.length;i++)
    {
    CLIENTS[i].send(message);
    }
}

serialListener();

function serialListener(debug)
{
    var receivedData = "";
    serialPort = new SerialPort(portName, {
        baudrate: 9600,
         dataBits: 8,
         parity: 'none',
         stopBits: 1,
         flowControl: false
    });

    serialPort.on("open", function () {
      console.log('open serial communication');
            // Listens to incoming data
        serialPort.on('data', function(data) {
             receivedData += data.toString();
          if (receivedData .indexOf('E') >= 0 && receivedData .indexOf('B') >= 0) {
           sendData = receivedData .substring(receivedData .indexOf('B') + 1, receivedData .indexOf('E'));
           receivedData = '';
         }
         // send the incoming data to browser with websockets.
       sendAll(sendData);
      });
    });
}
Run Code Online (Sandbox Code Playgroud)

有人可以帮我弄清楚这里有什么问题吗?

rku*_*usa 9

我想,您应该CLIENTS在两个closeerror事件上从数组中删除套接字.否则,它会尝试将消息发送到已关闭的套接字.

  • 也许`CLIENTS.splice(CLIENTS.indexOf(ws), 1)`? (2认同)

jbl*_*bll 5

我遇到了同样的问题。结果我试图将事件发送到处于“关闭”状态的套接字。在广播消息之前检查每个套接字是否专门打开为我修复它:

    function sendAll(data){
        for(var i = 0; i < clients.length; i++){
            if(this.clients[i].readyState != this.clients[0].OPEN){
                console.error('Client state is ' + this.clients[i].readyState);
            }
            else{
                this.clients[i].send(data);
            }
        }
    }
Run Code Online (Sandbox Code Playgroud)

  • 官方文档像这样测试 readyState:`if (this.clients[i].readyState === WebSocket.OPEN)` 我认为这比针对第一个数组元素的属性进行测试更可取 (4认同)