rework fix to not affect the rest of the connections

Author: ArtemHoruzhenko

redisinsight/api/src/modules/redis-sentinel/redis-sentinel.service.ts

@@ -124,7 +124,7 @@ export class RedisSentinelService {
         sessionMetadata: this.constantsProvider.getSystemSessionMetadata(),
         databaseId: database.id || uuidv4(),
         context: ClientContext.Common,
-      }, database, { useRetry: false });
+      }, database, { useRetry: false, enableReadyCheck: false });
       result = await discoverSentinelMasterGroups(client);
       this.redisSentinelAnalytics.sendGetSentinelMastersSucceedEvent(result);
 

redisinsight/api/src/modules/redis/connection/ioredis.redis.connection.strategy.ts

@@ -60,7 +60,7 @@ export class IoredisRedisConnectionStrategy extends RedisConnectionStrategy {
       maxRetriesPerRequest: REDIS_CLIENTS_CONFIG.maxRetriesPerRequest,
       retryStrategy: options?.useRetry ? this.retryStrategy.bind(this) : this.dummyFn.bind(this),
       autoResendUnfulfilledCommands: false,
-      enableReadyCheck: false,
+      enableReadyCheck: options.enableReadyCheck ?? true,
     };
 
     if (tls) {

redisinsight/api/src/modules/redis/redis.client.factory.ts

@@ -22,6 +22,7 @@ export interface IRedisConnectionOptions {
   useRetry?: boolean,
   connectionName?: string,
   clientLib?: RedisClientLib,
+  enableReadyCheck?: boolean,
 }
 
 @Injectable()