fix(redis): Do not set connection name to prevent calling CLIENT.SETN…

…AME command
postalsys · Oct 25, 2024 · 166a947 · 166a947
1 parent 5231327
commit 166a947
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/db.js b/lib/db.js
@@ -57,8 +57,9 @@ const REDIS_CONF = Object.assign(
             logger.fatal({ msg: 'Redis connection error', err });
             // always try to reconnect
             return true;
-        },
-        connectionName: `${packageData.name}@${packageData.version}[${process.pid}${threadId ? `:${threadId}` : ''}]`
+        }
+        // Setting connection name triggers CLIENT.SETNAME command which is not supported by many managed hosts
+        //connectionName: `${packageData.name}@${packageData.version}[${process.pid}${threadId ? `:${threadId}` : ''}]`
     },
     typeof redisConf === 'string' ? redisUrl(redisConf) : redisConf || {}
 );