RI-6928 fix sessionId calculation to match schema requirements (#4417)

Author: ArtemHoruzhenko

redisinsight/api/src/modules/analytics/analytics.service.spec.ts

@@ -14,6 +14,7 @@ import { AppType } from 'src/modules/server/models/server';
 import { SettingsService } from 'src/modules/settings/settings.service';
 import { ConstantsProvider } from 'src/modules/constants/providers/constants.provider';
 import { LocalConstantsProvider } from 'src/modules/constants/providers/local.constants.provider';
+import { convertAnyStringToPositiveInteger } from 'src/utils';
 import {
   AnalyticsService,
   Telemetry,
@@ -178,7 +179,7 @@ describe('AnalyticsService', () => {
 
       expect(service.getSessionId()).toEqual(-1);
       expect(service.getSessionId(mockSessionMetadata))
-        .toEqual(Number(BigInt(`0x${Buffer.from(mockSessionMetadata.sessionId).toString('hex')}`)));
+        .toEqual(convertAnyStringToPositiveInteger(mockSessionMetadata.sessionId));
     });
   });
 

redisinsight/api/src/modules/analytics/analytics.service.ts

@@ -9,6 +9,7 @@ import { SettingsService } from 'src/modules/settings/settings.service';
 import { ConstantsProvider } from 'src/modules/constants/providers/constants.provider';
 import { ServerService } from 'src/modules/server/server.service';
 import { SessionMetadata } from 'src/common/models';
+import { convertAnyStringToPositiveInteger } from 'src/utils';
 
 export const NON_TRACKING_ANONYMOUS_ID = '00000000-0000-0000-0000-000000000001';
 
@@ -87,8 +88,7 @@ export class AnalyticsService {
     }
 
     if (sessionMetadata?.sessionId) {
-      // convert string to number
-      return Number(BigInt(`0x${Buffer.from(sessionMetadata?.sessionId).toString('hex')}`));
+      return convertAnyStringToPositiveInteger(sessionMetadata?.sessionId);
     }
 
     return -1;

redisinsight/api/src/utils/converter.spec.ts

@@ -1,6 +1,7 @@
 import {
   convertIntToSemanticVersion,
   convertStringToNumber,
+  convertAnyStringToPositiveInteger,
 } from './converter';
 
 const convertIntToSemanticVersionTests: Record<string, any>[] = [
@@ -49,3 +50,24 @@ describe('convertStringToNumber', () => {
     });
   });
 });
+
+const convertAnyStringToPositiveIntegerTests = [
+  [undefined, -1],
+  [null, -1],
+  [123123, -1],
+  [[], -1],
+  [{}, -1],
+  [{ length: 12 }, -1],
+  ['', -1],
+  ['1', 49],
+  ['4f5daa5e-6139-4e95-8e7c-3283287f4218', 1347108680],
+  [
+    (new Array(1000).fill('4f5daa5e-6139-4e95-8e7c-3283287f4218')).join(),
+    229890988,
+  ],
+] as [string, number][];
+describe('convertAnyStringToPositiveInteger', () => {
+  it.each(convertAnyStringToPositiveIntegerTests)('for input: %s, should return: %s', (input, result) => {
+    expect(convertAnyStringToPositiveInteger(input)).toEqual(result);
+  });
+});

redisinsight/api/src/utils/converter.ts

@@ -36,3 +36,19 @@ export const convertStringToNumber = (value: any, defaultValue?: number): number
 
   return num;
 };
+
+export const convertAnyStringToPositiveInteger = (str: string) => {
+  if (!isString(str) || !str.length) {
+    return -1;
+  }
+
+  let hash = 0;
+  for (let i = 0; i < str.length; i += 1) {
+    const char = str.charCodeAt(i);
+    // eslint-disable-next-line no-bitwise
+    hash = (hash << 5) - hash + char;
+    // eslint-disable-next-line no-bitwise
+    hash |= 0;
+  }
+  return Math.abs(hash);
+};