fix pr comments

Author: ArtemHoruzhenko

redisinsight/api/src/modules/database/providers/database.factory.ts

@@ -1,4 +1,5 @@
 import { Injectable, Logger, NotFoundException } from '@nestjs/common';
+import { v4 as uuidv4 } from 'uuid';
 import { ConnectionType, HostingProvider } from 'src/modules/database/entities/database.entity';
 import { catchRedisConnectionError, getHostingProvider } from 'src/utils';
 import { Database } from 'src/modules/database/models/database';
@@ -35,7 +36,7 @@ export class DatabaseFactory {
     const client = await this.redisClientFactory.getConnectionStrategy().createStandaloneClient(
       {
         sessionMetadata,
-        databaseId: database.id || 'new', // we assume that if no database id defined we are in creation process
+        databaseId: database.id || uuidv4(), // we assume that if no database id defined we are in creation process
         context: ClientContext.Common,
       },
       database,
@@ -110,7 +111,7 @@ export class DatabaseFactory {
       const clusterClient = await this.redisClientFactory.getConnectionStrategy().createClusterClient(
         {
           sessionMetadata,
-          databaseId: model.id || 'new', // we assume that if no database id defined we are in creation process
+          databaseId: model.id || uuidv4(), // we assume that if no database id defined we are in creation process
           context: ClientContext.Common,
         },
         model,
@@ -157,7 +158,7 @@ export class DatabaseFactory {
       const sentinelClient = await this.redisClientFactory.getConnectionStrategy().createSentinelClient(
         {
           sessionMetadata,
-          databaseId: model.id || 'new', // we assume that if no database id defined we are in creation process
+          databaseId: model.id || uuidv4(), // we assume that if no database id defined we are in creation process
           context: ClientContext.Common,
         },
         model,

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

@@ -1,6 +1,7 @@
 import {
   HttpException, Injectable, Logger,
 } from '@nestjs/common';
+import { v4 as uuidv4 } from 'uuid';
 import { CreateSentinelDatabaseResponse } from 'src/modules/redis-sentinel/dto/create.sentinel.database.response';
 import { CreateSentinelDatabasesDto } from 'src/modules/redis-sentinel/dto/create.sentinel.databases.dto';
 import { Database } from 'src/modules/database/models/database';
@@ -121,7 +122,7 @@ export class RedisSentinelService {
       const database = await this.databaseFactory.createStandaloneDatabaseModel(dto);
       const client = await this.redisClientFactory.getConnectionStrategy().createStandaloneClient({
         sessionMetadata: this.constantsProvider.getSystemSessionMetadata(),
-        databaseId: database.id || 'new',
+        databaseId: database.id || uuidv4(),
         context: ClientContext.Common,
       }, database, { useRetry: false });
       result = await discoverSentinelMasterGroups(client);