diff --git a/CHANGELOG.md b/CHANGELOG.md
index 54c82d5..88d0412 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,12 @@
+## [3.4.3](https://github.com/socketio/socket.io-parser/compare/3.4.2...3.4.3) (2023-05-22)
+
+
+### Bug Fixes
+
+* check the format of the event name ([2dc3c92](https://github.com/socketio/socket.io-parser/commit/2dc3c92622dad113b8676be06f23b1ed46b02ced))
+
+
+
## [3.4.2](https://github.com/socketio/socket.io-parser/compare/3.4.1...3.4.2) (2022-11-09)
diff --git a/index.js b/index.js
index 245a800..50ff90e 100644
--- a/index.js
+++ b/index.js
@@ -329,11 +329,10 @@ function decodeString(str) {
// look up json data
if (str.charAt(++i)) {
var payload = tryParse(str.substr(i));
- var isPayloadValid = payload !== false && (p.type === exports.ERROR || isArray(payload));
- if (isPayloadValid) {
+ if (isPayloadValid(p.type, payload)) {
p.data = payload;
} else {
- return error('invalid payload');
+ throw new Error("invalid payload");
}
}
@@ -349,6 +348,26 @@ function tryParse(str) {
}
}
+function isPayloadValid(type, payload) {
+ switch (type) {
+ case 0: // CONNECT
+ return typeof payload === "object";
+ case 1: // DISCONNECT
+ return payload === undefined;
+ case 4: // ERROR
+ return typeof payload === "string" || typeof payload === "object";
+ case 2: // EVENT
+ case 5: // BINARY_EVENT
+ return (
+ isArray(payload) &&
+ (typeof payload[0] === "string" || typeof payload[0] === "number")
+ );
+ case 3: // ACK
+ case 6: // BINARY_ACK
+ return isArray(payload);
+ }
+}
+
/**
* Deallocates a parser's resources
*
diff --git a/package.json b/package.json
index f82999a..5202c76 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
{
"name": "socket.io-parser",
- "version": "3.4.2",
+ "version": "3.4.3",
"description": "socket.io protocol parser",
"repository": {
"type": "git",
diff --git a/test/arraybuffer.js b/test/arraybuffer.js
index c8295f1..f2948a5 100644
--- a/test/arraybuffer.js
+++ b/test/arraybuffer.js
@@ -50,7 +50,7 @@ describe('parser', function() {
it('cleans itself up on close', function() {
var packet = {
type: parser.BINARY_EVENT,
- data: [new ArrayBuffer(2), new ArrayBuffer(3)],
+ data: ["foo", new ArrayBuffer(2), new ArrayBuffer(3)],
id: 0,
nsp: '/'
};
diff --git a/test/parser.js b/test/parser.js
index 652aa7f..7d560bf 100644
--- a/test/parser.js
+++ b/test/parser.js
@@ -86,12 +86,20 @@ describe('parser', function(){
}
});
- it('returns an error packet on parsing error', function(done){
- var decoder = new parser.Decoder();
- decoder.on('decoded', function(packet) {
- expect(packet).to.eql({ type: 4, data: 'parser error: invalid payload' });
- done();
- });
- decoder.add('442["some","data"');
+ it('returns an error packet on parsing error', function(){
+ function isInvalidPayload (str) {
+ expect(function () {
+ new parser.Decoder().add(str)
+ }).to.throwException(/^invalid payload$/);
+ }
+
+ isInvalidPayload('442["some","data"');
+ isInvalidPayload('0/admin,"invalid"');
+ isInvalidPayload("1/admin,{}");
+ isInvalidPayload('2/admin,"invalid');
+ isInvalidPayload("2/admin,{}");
+ isInvalidPayload('2[{"toString":"foo"}]');
+ isInvalidPayload('2[true,"foo"]');
+ isInvalidPayload('2[null,"bar"]');
});
});
pFad - Phonifier reborn
Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.
Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.
Alternative Proxies:
Alternative Proxy
pFad Proxy
pFad v3 Proxy
pFad v4 Proxy