Browse Source

Merge branch 'master' of github.com:mrniko/netty-socketio

master
Nikita 10 years ago
parent
commit
7d3cb580ab
  1. 2
      README.md
  2. 7
      src/main/java/com/corundumstudio/socketio/protocol/PacketDecoder.java

2
README.md

@ -199,7 +199,7 @@ Include the following to your dependency list:
<dependency>
<groupId>com.corundumstudio.socketio</groupId>
<artifactId>netty-socketio</artifactId>
<version>1.7.6</version>
<version>1.7.8</version>
</dependency>
### Supported by

7
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);

Loading…
Cancel
Save