pomelo 框架一个session bug的修复

最新推荐文章于 2023-10-10 12:46:53 发布

原创

最新推荐文章于 2023-10-10 12:46:53 发布 · 2.8k 阅读

1 ·

CC 4.0 BY-SA版权

文章标签：

#调试 #nodejs #bug #pomelo #session

在客户端调试状态下，当socket连接未保持心跳但未完全断开时，服务端尝试发送数据到session会导致'Error: not opened'异常。错误发生在WebSocket.send()方法。修复方法是在session的send()方法中添加对socket状态的检查，如果socket失效则关闭它。修复代码位于D:projectgamegame-server ode_modulespomelolibcommonservicesessionService.js的517行。

有一种情况发生会导致如下的错误发生。这种情况发生于客户端调试状态，socket还保持连接，但是没有维持心跳包，然后客户端突然断开，而服务端还不断的往这个session发送网络数据，导致发生异常

websocket send binary data failed: "Error: not opened\n at WebSocket.send (D:\\project\\gameBilliards\\g
ame-server\\node_modules\\pomelo\\node_modules\\ws\\lib\\WebSocket.js:217:38)\n at Socket.sendRaw (D:\\project\\gameB
illiards\\game-server\\node_modules\\pomelo\\lib\\connectors\\hybridsocket.js:65:15)\n at Socket.send (D:\\project\\g
ameBilliards\\game-server\\node_modules\\pomelo\\lib\\connectors\\hybridsocket.js:84:8)\n at Session.send (D:\\projec
t\\gameBilliards\\game-server\\node_modules\\pomelo\\lib\\common\\service\\sessionService.js:517:19)\n at send (D:\\p
roject\\gameBilliards\\game-server\\node_modules\\pomelo\\lib\\common\\service\\sessionService.js:415:11)\n at Sessio
nService.sendMessage (D:\\project\\gameBilliards\\game-ser

最低0.47元/天解锁文章

2 条评论

cwk8866 2017.04.04
根据您的启发,我重新修改了下,sessionService: if(this.__socket__.state !== ST_WORKING) { console.log('sessionService.js 519 socket error,',this.__socket__.state); this.closed('socket lost'); return; } hybridsocket: if(this.state !== ST_WORKING) { return; } if(this.socket.state !== ST_WORKING && this.socket.state !== ST_WAIT_ACK) { console.log('hybridsocket.js 64. sk state:%s, state:%s',this.socket.state,this.state); return; } this.socket.send(msg, {binary: true}, function(err) { if(!!err) { logger.error('websocket send binary data failed: %j', err.stack); return; } });

ningcaicheng 2017.01.21
先用调试器调试跟踪，看问题出在什么地方然后解决
- cwk8866回复ningcaicheng 2017.04.04
  [reply]ningcaicheng[/reply] 现在更新到2.2.5的版本了,还是时不时会抱这种错,看了下源码修改了下 if(this.__socket__!= ST_INITED) { console.log('519 socket error'); this.closed('socket lost'); return; }

cwk8866 2017.01.13
前辈,为什么我这里加了这两段代码打印出来的状态都是undefined,我这边也会时不时报一些hybridsocket.js这一类的错误,一直没有搞太清楚

ningcaicheng 2016.09.22
l同样的提示另一处要修复的bug是路径：D:\project\gameBilliards\game-server\node_modules\pomelo\lib\connectors\hybridsocket.js line：59 Socket.prototype.sendRaw = function(msg) var WebSocket_OPEN=1; if(this.socket.readyState !== WebSocket_OPEN) { logger.error("websocket is closed"); return; }