Skip to content

Commit 6934270

Browse files
committed
fix: always copy subscribe unsubscribe arguments
1 parent 4271a70 commit 6934270

File tree

2 files changed

+15
-10
lines changed

2 files changed

+15
-10
lines changed

changelog.md

+7-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
1-
Changelog
2-
=========
1+
# Changelog
2+
3+
## v.2.7.2 - 14 Mar, 2017
4+
5+
Bugfixes
6+
7+
- Fixed not always copying subscribe unsubscribe arguments
38

49
## v.2.7.1 - 14 Mar, 2017
510

lib/individualCommands.js

+8-8
Original file line numberDiff line numberDiff line change
@@ -398,7 +398,7 @@ RedisClient.prototype.subscribe = RedisClient.prototype.SUBSCRIBE = function sub
398398
callback,
399399
i = 0;
400400
if (Array.isArray(arguments[0])) {
401-
arr = arguments[0];
401+
arr = arguments[0].slice(0);
402402
callback = arguments[1];
403403
} else {
404404
len = arguments.length;
@@ -425,7 +425,7 @@ Multi.prototype.subscribe = Multi.prototype.SUBSCRIBE = function subscribe () {
425425
callback,
426426
i = 0;
427427
if (Array.isArray(arguments[0])) {
428-
arr = arguments[0];
428+
arr = arguments[0].slice(0);
429429
callback = arguments[1];
430430
} else {
431431
len = arguments.length;
@@ -453,7 +453,7 @@ RedisClient.prototype.unsubscribe = RedisClient.prototype.UNSUBSCRIBE = function
453453
callback,
454454
i = 0;
455455
if (Array.isArray(arguments[0])) {
456-
arr = arguments[0];
456+
arr = arguments[0].slice(0);
457457
callback = arguments[1];
458458
} else {
459459
len = arguments.length;
@@ -481,7 +481,7 @@ Multi.prototype.unsubscribe = Multi.prototype.UNSUBSCRIBE = function unsubscribe
481481
callback,
482482
i = 0;
483483
if (Array.isArray(arguments[0])) {
484-
arr = arguments[0];
484+
arr = arguments[0].slice(0);
485485
callback = arguments[1];
486486
} else {
487487
len = arguments.length;
@@ -510,7 +510,7 @@ RedisClient.prototype.psubscribe = RedisClient.prototype.PSUBSCRIBE = function p
510510
callback,
511511
i = 0;
512512
if (Array.isArray(arguments[0])) {
513-
arr = arguments[0];
513+
arr = arguments[0].slice(0);
514514
callback = arguments[1];
515515
} else {
516516
len = arguments.length;
@@ -537,7 +537,7 @@ Multi.prototype.psubscribe = Multi.prototype.PSUBSCRIBE = function psubscribe ()
537537
callback,
538538
i = 0;
539539
if (Array.isArray(arguments[0])) {
540-
arr = arguments[0];
540+
arr = arguments[0].slice(0);
541541
callback = arguments[1];
542542
} else {
543543
len = arguments.length;
@@ -565,7 +565,7 @@ RedisClient.prototype.punsubscribe = RedisClient.prototype.PUNSUBSCRIBE = functi
565565
callback,
566566
i = 0;
567567
if (Array.isArray(arguments[0])) {
568-
arr = arguments[0];
568+
arr = arguments[0].slice(0);
569569
callback = arguments[1];
570570
} else {
571571
len = arguments.length;
@@ -593,7 +593,7 @@ Multi.prototype.punsubscribe = Multi.prototype.PUNSUBSCRIBE = function punsubscr
593593
callback,
594594
i = 0;
595595
if (Array.isArray(arguments[0])) {
596-
arr = arguments[0];
596+
arr = arguments[0].slice(0);
597597
callback = arguments[1];
598598
} else {
599599
len = arguments.length;

0 commit comments

Comments
 (0)