From 07f172065e0980db2c5b96555ea04e2629139c83 Mon Sep 17 00:00:00 2001 From: bpb Date: Fri, 15 Jan 2016 16:31:10 +0300 Subject: [PATCH] fix for skipping 'd=' in packet --- .../corundumstudio/socketio/protocol/PacketDecoder.java | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/corundumstudio/socketio/protocol/PacketDecoder.java b/src/main/java/com/corundumstudio/socketio/protocol/PacketDecoder.java index a2a317b..25e6a13 100644 --- a/src/main/java/com/corundumstudio/socketio/protocol/PacketDecoder.java +++ b/src/main/java/com/corundumstudio/socketio/protocol/PacketDecoder.java @@ -52,11 +52,7 @@ public class PacketDecoder { public ByteBuf preprocessJson(Integer jsonIndex, ByteBuf content) throws IOException { String packet = URLDecoder.decode(content.toString(CharsetUtil.UTF_8), CharsetUtil.UTF_8.name()); - int startPos = 0; if (jsonIndex != null) { - // skip "d=" - startPos = 2; - /** * double escaping is required for escaped new lines because unescaping of new lines can be done safely on server-side * (c) socket.io.js @@ -64,6 +60,9 @@ public class PacketDecoder { * @see https://github.com/Automattic/socket.io-client/blob/1.3.3/socket.io.js#L2682 */ packet = packet.replace("\\\\n", "\\n"); + + // skip "d=" + packet = packet.substring(2); } packet = new String(packet.getBytes(CharsetUtil.ISO_8859_1), CharsetUtil.UTF_8);