#RI-2998-add useSNI to add db telemetry event (#765)

* #RI-2998-add useSNI to add db telemetry event

Author: AmirAllayarovSofteq

redisinsight/api/src/__mocks__/redis-databases.ts

@@ -27,6 +27,7 @@ export const mockStandaloneDatabaseEntity: DatabaseInstanceEntity = {
   provider: HostingProvider.LOCALHOST,
   modules: '[]',
   encryption: null,
+  tlsServername: 'server-name',
 };
 
 export const mockOSSClusterDatabaseEntity: DatabaseInstanceEntity = {

redisinsight/api/src/modules/core/services/redis/redis.service.spec.ts

@@ -25,6 +25,7 @@ jest.mock('ioredis');
 
 const mockTlsConfigResult: ConnectionOptions = {
   rejectUnauthorized: true,
+  servername: mockStandaloneDatabaseEntity.tlsServername,
   checkServerIdentity: () => undefined,
   ca: [mockCaCertDto.cert],
   key: mockClientCertDto.key,

redisinsight/api/src/modules/shared/services/instances-business/instances-analytics.service.spec.ts

@@ -26,6 +26,7 @@ const mockDatabaseInstanceDto: DatabaseInstanceResponse = {
     verifyServerCert: true,
     caCertId: mockCaCertEntity.id,
     clientCertPairId: mockClientCertEntity.id,
+    servername: mockStandaloneDatabaseEntity.tlsServername,
   },
   modules: [],
 };
@@ -100,7 +101,7 @@ describe('InstancesAnalytics', () => {
   });
 
   describe('sendInstanceAddedEvent', () => {
-    it('should emit event with enabled tls', () => {
+    it('should emit event with enabled tls and sni', () => {
       const instance = mockDatabaseInstanceDto;
       service.sendInstanceAddedEvent(instance, mockRedisGeneralInfo);
 
@@ -113,6 +114,7 @@ describe('InstancesAnalytics', () => {
           useTLS: 'enabled',
           verifyTLSCertificate: 'enabled',
           useTLSAuthClients: 'enabled',
+          useSNI: 'enabled',
           version: mockRedisGeneralInfo.version,
           numberOfKeys: mockRedisGeneralInfo.totalKeys,
           numberOfKeysRange: '0 - 500 000',
@@ -123,7 +125,7 @@ describe('InstancesAnalytics', () => {
         },
       );
     });
-    it('should emit event with disabled tls', () => {
+    it('should emit event with disabled tls and sni', () => {
       const instance = {
         ...mockDatabaseInstanceDto,
         tls: undefined,
@@ -139,6 +141,7 @@ describe('InstancesAnalytics', () => {
           useTLS: 'disabled',
           verifyTLSCertificate: 'disabled',
           useTLSAuthClients: 'disabled',
+          useSNI: 'disabled',
           version: mockRedisGeneralInfo.version,
           numberOfKeys: mockRedisGeneralInfo.totalKeys,
           numberOfKeysRange: '0 - 500 000',
@@ -167,6 +170,7 @@ describe('InstancesAnalytics', () => {
           useTLS: 'enabled',
           verifyTLSCertificate: 'enabled',
           useTLSAuthClients: 'enabled',
+          useSNI: 'enabled',
           version: mockRedisGeneralInfo.version,
           numberOfKeys: undefined,
           numberOfKeysRange: undefined,

redisinsight/api/src/modules/shared/services/instances-business/instances-analytics.service.ts

@@ -49,6 +49,9 @@ export class InstancesAnalyticsService extends TelemetryBaseService {
           useTLSAuthClients: instance?.tls?.clientCertPairId
             ? 'enabled'
             : 'disabled',
+          useSNI: instance?.tls?.servername
+            ? 'enabled'
+            : 'disabled',
           version: additionalInfo.version,
           numberOfKeys: additionalInfo.totalKeys,
           numberOfKeysRange: getRangeForNumber(additionalInfo.totalKeys, TOTAL_KEYS_BREAKPOINTS),

redisinsight/api/src/modules/shared/services/instances-business/instances-business.service.spec.ts

@@ -71,6 +71,7 @@ const addDatabaseDto: AddDatabaseInstanceDto = {
     verifyServerCert: true,
     caCertId: mockCaCertEntity.id,
     clientCertPairId: mockClientCertEntity.id,
+    servername: mockStandaloneDatabaseEntity.tlsServername,
   },
 };