Skip to content

Commit 2a222de

Browse files
authored
Merge pull request #601 from Dragoi-Stefan/master
Add null checks on connection commands array
2 parents b03e3c4 + ec5208e commit 2a222de

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

lib/connections.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -308,8 +308,10 @@ class ConnectionWrapper {
308308
// redisConnection.options.port);
309309
const extraROCmds = require('config').get('redis.extraAllowedReadOnlyCommands');
310310
redisConnection.options.commandList = {
311-
all: p[0].value.map((item) => (item[0].toLowerCase())),
312-
ro: p[0].value.filter((item) => (item[2].indexOf('readonly') >= 0 || extraROCmds.indexOf(item[0]) >= 0))
311+
all: p[0].value.filter(item => item && Array.isArray(item) && item.length > 0)
312+
.map((item) => (item[0].toLowerCase())),
313+
ro: p[0].value.filter((item) => item && Array.isArray(item) && item.length > 2 &&
314+
(item[2].indexOf('readonly') >= 0 || extraROCmds.indexOf(item[0]) >= 0))
313315
.map((item) => (item[0].toLowerCase()))
314316
};
315317
}

0 commit comments

Comments
 (0)