#RI-5748 fix hosting provider + add more rules

Author: ArtemHoruzhenko

redisinsight/api/src/modules/database/entities/database.entity.ts

@@ -13,7 +13,8 @@ export enum HostingProvider {
   RE_CLUSTER = 'RE_CLUSTER',
   RE_CLOUD = 'RE_CLOUD',
   REDIS_MANAGED = 'REDIS_MANAGED',
-  AZURE = 'AZURE',
+  AZURE_CACHE = 'AZURE_CACHE',
+  AZURE_CACHE_REDIS_ENTERPRISE = 'AZURE_CACHE_REDIS_ENTERPRISE',
   COMMUNITY_EDITION = 'COMMUNITY_EDITION',
   AWS_ELASTICACHE = 'AWS_ELASTICACHE',
   AWS_MEMORYDB = 'AWS_MEMORYDB',

redisinsight/api/src/utils/hosting-provider-helper.spec.ts

@@ -17,11 +17,20 @@ const getHostingProviderTests = [
     input: 'redis-16781.c273.us-east-1-2.ec2.cloud.redislabs.com',
     output: HostingProvider.RE_CLOUD,
   },
+  {
+    input: 'redis-16781.c273.us-east-1-2.ec2.cloud.redis-cloud.com',
+    output: HostingProvider.RE_CLOUD,
+  },
+  {
+    input: 'redis-16781.c273.us-east-1-2.ec2.cloud.rlrcp.com',
+    output: HostingProvider.RE_CLOUD,
+  },
   {
     input: 'askubuntu.mki5tz.0001.use1.cache.amazonaws.com',
     output: HostingProvider.AWS_ELASTICACHE,
   },
-  { input: 'contoso5.redis.cache.windows.net', output: HostingProvider.AZURE },
+  { input: 'contoso5.redis.cache.windows.net', output: HostingProvider.AZURE_CACHE },
+  { input: 'contoso5.redisenterprise.cache.azure.net', output: HostingProvider.AZURE_CACHE_REDIS_ENTERPRISE },
   { input: 'demo-redis-provider.unknown.com', output: HostingProvider.UNKNOWN },
   {
     input: 'localhost',

redisinsight/api/src/utils/hosting-provider-helper.ts

@@ -2,25 +2,27 @@ import { IP_ADDRESS_REGEX, PRIVATE_IP_ADDRESS_REGEX } from 'src/constants';
 import { HostingProvider } from 'src/modules/database/entities/database.entity';
 import { RedisClient } from 'src/modules/redis/client';
 
-// Ignore LGTM [js/incomplete-url-substring-sanitization] alert.
 // Because we do not bind potentially dangerous logic to this.
 // We define a hosting provider for telemetry only.
 export const getHostingProvider = async (client: RedisClient, databaseHost: string): Promise<HostingProvider> => {
   try {
     const host = databaseHost.toLowerCase();
 
     // Tries to detect the hosting provider from the hostname.
-    if (host.endsWith('rlrcp.com') || host.endsWith('redislabs.com')) { // lgtm[js/incomplete-url-substring-sanitization]
+    if (host.endsWith('rlrcp.com') || host.endsWith('redislabs.com') || host.endsWith('redis-cloud.com')) {
       return HostingProvider.RE_CLOUD;
     }
-    if (host.endsWith('cache.amazonaws.com')) { // lgtm[js/incomplete-url-substring-sanitization]
+    if (host.endsWith('cache.amazonaws.com')) {
       return HostingProvider.AWS_ELASTICACHE;
     }
     if (host.includes('memorydb')) {
       return HostingProvider.AWS_MEMORYDB;
     }
-    if (host.endsWith('cache.windows.net')) { // lgtm[js/incomplete-url-substring-sanitization]
-      return HostingProvider.AZURE;
+    if (host.endsWith('cache.windows.net')) {
+      return HostingProvider.AZURE_CACHE;
+    }
+    if (host.endsWith('redisenterprise.cache.azure.net')) {
+      return HostingProvider.AZURE_CACHE_REDIS_ENTERPRISE;
     }
 
     try {