fix: auth command should be not allowed in auto pipeline. (#1242)

The auth command should be sent before other commands. So it should be exclude from auto pipeline command list as well.
redis · Dec 13, 2020 · bafdd4b · bafdd4b
1 parent cd336bf
commit bafdd4b
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/lib/autoPipelining.ts b/lib/autoPipelining.ts
@@ -5,6 +5,7 @@ import asCallback from "standard-as-callback";
 export const kExec = Symbol("exec");
 export const kCallbacks = Symbol("callbacks");
 export const notAllowedAutoPipelineCommands = [
+  "auth",
   "info",
   "script",
   "quit",
@@ -17,7 +18,11 @@ export const notAllowedAutoPipelineCommands = [
   "unpsubscribe",
 ];
 
-function findAutoPipeline(client, _commandName, ...args: Array<string>): string {
+function findAutoPipeline(
+  client,
+  _commandName,
+  ...args: Array<string>
+): string {
   if (!client.isCluster) {
     return "main";
   }