Thanks. Please rebase and remove the AUTH addition to MIGRATE as it has been added in 3ce1c28.
Also make sure you have only one issue per PR and fix the title accordingly ("bugfix" isn't descriptive enough).

@yossigo does this 6910113 fix seem correct?

@yoav-steinberg I think it's correct. @ushachar what do you think?

@yossigo There's no need for this, there's a nodeIsSlave check at the beginning of the handling for all SETSLOT subcommands...

There are quite a few more issues here as well. For 1, this is merging against Redis 3.0, which is definitely no longer supported. Some of this code is redundant. We probably just want to replace this with a new PR with the couple of fixes that are still relevant.

@yossigo There's no need for this, there's a nodeIsSlave check at the beginning of the handling for all SETSLOT subcommands...

The call to nodeIsSalve() at the top checks myself, but there are variants of this command (IMPORTING, MIGRATING, NODE) that receive a node id as and argument. The fix checks that argument. That's why it seemed relevant to me.

@yoav-steinberg I agree I think those are useful, I'll open another PR with just those fixes.