From 684bb1bf36d02162d9970571194e3b3b183bd3a1 Mon Sep 17 00:00:00 2001 From: Vlad Frangu Date: Mon, 13 Jan 2020 19:54:15 +0200 Subject: [PATCH] src: Remove `_trace` from different places in the WS (#3679) * src: Remove `ws._trace` from READY * src: Remove `ws._trace` from RESUME * lint: Fix lint by removing unused packet (#7) Co-authored-by: bdistin --- src/client/websocket/packets/handlers/Ready.js | 3 +-- src/client/websocket/packets/handlers/Resumed.js | 6 ++---- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/src/client/websocket/packets/handlers/Ready.js b/src/client/websocket/packets/handlers/Ready.js index 7fea48c8d..b015b924b 100644 --- a/src/client/websocket/packets/handlers/Ready.js +++ b/src/client/websocket/packets/handlers/Ready.js @@ -74,8 +74,7 @@ class ReadyHandler extends AbstractHandler { const ws = this.packetManager.ws; ws.sessionID = data.session_id; - ws._trace = data._trace; - client.emit('debug', `READY ${ws._trace.join(' -> ')} ${ws.sessionID}`); + client.emit('debug', `READY ${ws.sessionID}`); ws.checkIfReady(); } } diff --git a/src/client/websocket/packets/handlers/Resumed.js b/src/client/websocket/packets/handlers/Resumed.js index 0c796053b..a0a51c0c1 100644 --- a/src/client/websocket/packets/handlers/Resumed.js +++ b/src/client/websocket/packets/handlers/Resumed.js @@ -2,18 +2,16 @@ const AbstractHandler = require('./AbstractHandler'); const Constants = require('../../../../util/Constants'); class ResumedHandler extends AbstractHandler { - handle(packet) { + handle() { const client = this.packetManager.client; const ws = client.ws.connection; - ws._trace = packet.d._trace; - ws.status = Constants.Status.READY; this.packetManager.handleQueue(); const replayed = ws.sequence - ws.closeSequence; - ws.debug(`RESUMED ${ws._trace.join(' -> ')} | replayed ${replayed} events.`); + ws.debug(`RESUMED | replayed ${replayed} events.`); client.emit(Constants.Events.RESUME, replayed); ws.heartbeat(); }