You've already forked node-redis
mirror of
https://github.com/redis/node-redis.git
synced 2025-08-06 02:15:48 +03:00
Fixed zero length multibulk. Passing null not an empty array
This commit is contained in:
2
index.js
2
index.js
@@ -118,7 +118,7 @@ RedisReplyParser.prototype.execute = function (incoming_buf) {
|
||||
this.multi_bulk_replies = [];
|
||||
this.state = "type";
|
||||
if (0 == this.multi_bulk_length) {
|
||||
this.send_reply([]);
|
||||
this.send_reply(null);
|
||||
}
|
||||
} else {
|
||||
this.emit("error", new Error("didn't see LF after NL reading multi bulk count"));
|
||||
|
2
test.js
2
test.js
@@ -218,7 +218,7 @@ tests.MULTIBULK_ZERO_LENGTH = function () {
|
||||
var name = "MULTIBULK_ZERO_LENGTH";
|
||||
client.KEYS(['users:*'], function(err, results){
|
||||
assert.strictEqual(null, err, 'error on empty multibulk reply');
|
||||
assert.strictEqual(0, results.length);
|
||||
assert.strictEqual(null, results);
|
||||
next(name);
|
||||
});
|
||||
};
|
||||
|
Reference in New Issue
Block a user