Merge pull request #2834 from RedisInsight/latest

Latest to main

Author: vlad-dargel

redisinsight/api/config/default.ts

@@ -60,7 +60,7 @@ export default {
     tlsKey: process.env.SERVER_TLS_KEY,
     staticContent: !!process.env.SERVER_STATIC_CONTENT || false,
     buildType: process.env.BUILD_TYPE || 'ELECTRON',
-    appVersion: process.env.APP_VERSION || '2.36.0',
+    appVersion: process.env.APP_VERSION || '2.38.0',
     requestTimeout: parseInt(process.env.REQUEST_TIMEOUT, 10) || 25000,
     excludeRoutes: [],
     excludeAuthRoutes: [],

redisinsight/api/config/swagger.ts

@@ -5,7 +5,7 @@ const SWAGGER_CONFIG: Omit<OpenAPIObject, 'paths'> = {
   info: {
     title: 'RedisInsight Backend API',
     description: 'RedisInsight Backend API',
-    version: '2.36.0',
+    version: '2.38.0',
   },
   tags: [],
 };

redisinsight/api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "redisinsight-api",
-  "version": "2.36.0",
+  "version": "2.38.0",
   "description": "RedisInsight API",
   "private": true,
   "author": {
@@ -42,7 +42,8 @@
     "word-wrap": "1.2.4",
     "mocha/minimatch": "^3.0.5",
     "@nestjs/platform-socket.io/socket.io": "^4.7.1",
-    "**/semver": "^7.5.2"
+    "**/semver": "^7.5.2",
+    "winston-daily-rotate-file/**/file-stream-rotator": "^1.0.0"
   },
   "dependencies": {
     "@nestjs/common": "^9.0.11",

redisinsight/api/src/modules/browser/services/keys-business/scanner/strategies/standalone.strategy.spec.ts

@@ -101,6 +101,40 @@ describe('Standalone Scanner Strategy', () => {
         null,
       );
     });
+    it('should scan 2000 items when count provided more then 2k', async () => {
+      const args = { ...getKeysDto, count: 10_000, match: '*' };
+      jest.spyOn(Utils, 'getTotal').mockResolvedValue(mockGetTotalResponse_1);
+
+      when(browserTool.execCommand)
+        .calledWith(
+          mockBrowserClientMetadata,
+          BrowserToolKeysCommands.Scan,
+          expect.anything(),
+          null,
+        )
+        .mockResolvedValue([0, [getKeyInfoResponse.name]]);
+
+      strategy.getKeysInfo = jest.fn().mockResolvedValue([getKeyInfoResponse]);
+
+      const result = await strategy.getKeys(mockBrowserClientMetadata, args);
+
+      expect(result).toEqual([
+        {
+          ...mockNodeEmptyResult,
+          total: 1,
+          scanned: 2000,
+          keys: [getKeyInfoResponse],
+        },
+      ]);
+      expect(strategy.getKeysInfo).toHaveBeenCalled();
+      expect(browserTool.execCommand).toHaveBeenNthCalledWith(
+        1,
+        mockBrowserClientMetadata,
+        BrowserToolKeysCommands.Scan,
+        ['0', 'MATCH', args.match, 'COUNT', 2000],
+        null,
+      );
+    });
     it('should return keys names and type only', async () => {
       const args = {
         ...getKeysDto, type: 'string', match: 'pattern*', keysInfo: false,

redisinsight/api/src/modules/browser/services/keys-business/scanner/strategies/standalone.strategy.ts

@@ -86,6 +86,8 @@ export class StandaloneStrategy extends AbstractStrategy {
     count: number,
     type?: RedisDataType,
   ): Promise<void> {
+    const COUNT = Math.min(2000, count);
+
     let fullScanned = false;
     // todo: remove settings from here. threshold should be part of query?
     const settings = await this.settingsService.getAppSettings('1');
@@ -97,7 +99,7 @@ export class StandaloneStrategy extends AbstractStrategy {
         node.scanned < settings.scanThreshold
       )
     ) {
-      let commandArgs = [`${node.cursor}`, 'MATCH', match, 'COUNT', count];
+      let commandArgs = [`${node.cursor}`, 'MATCH', match, 'COUNT', COUNT];
       if (type) {
         commandArgs = [...commandArgs, 'TYPE', type];
       }
@@ -112,7 +114,7 @@ export class StandaloneStrategy extends AbstractStrategy {
       // eslint-disable-next-line no-param-reassign
       node.cursor = parseInt(nextCursor, 10);
       // eslint-disable-next-line no-param-reassign
-      node.scanned += count;
+      node.scanned += COUNT;
       node.keys.push(...keys);
       fullScanned = node.cursor === 0;
     }

redisinsight/api/yarn.lock

@@ -3919,12 +3919,10 @@ file-entry-cache@^6.0.1:
   dependencies:
     flat-cache "^3.0.4"
 
-file-stream-rotator@^0.6.1:
-  version "0.6.1"
-  resolved "https://registry.yarnpkg.com/file-stream-rotator/-/file-stream-rotator-0.6.1.tgz#007019e735b262bb6c6f0197e58e5c87cb96cec3"
-  integrity sha512-u+dBid4PvZw17PmDeRcNOtCP9CCK/9lRN2w+r1xIS7yOL9JFrIBKTvrYsxT4P0pGtThYTn++QS5ChHaUov3+zQ==
-  dependencies:
-    moment "^2.29.1"
+file-stream-rotator@^0.6.1, file-stream-rotator@^1.0.0:
+  version "1.0.0"
+  resolved "https://registry.yarnpkg.com/file-stream-rotator/-/file-stream-rotator-1.0.0.tgz#de58379321a1ea6d2938ed5f5a2eff3b7f8b2780"
+  integrity sha512-qg5mQO7o+vhS7NPqkrkfJS8qqhz0d17Tnewmb5sUTUKwYe27LKaDtbTuRAtQWkBn6jROuFPVIDF5DtckzokFTQ==
 
 file-type@^16.5.4:
   version "16.5.4"
@@ -6201,11 +6199,6 @@ mocha@^8.4.0:
     yargs-parser "20.2.4"
     yargs-unparser "2.0.0"
 
-moment@^2.29.1:
-  version "2.29.4"
-  resolved "https://registry.yarnpkg.com/moment/-/moment-2.29.4.tgz#3dbe052889fe7c1b2ed966fcb3a77328964ef108"
-  integrity sha512-5LC9SOxjSc2HF6vO2CyuTDNivEdoz2IvyJJGj6X8DJ0eFyfszE0QiEd+iXmBvUP3WHxSjFH/vIsA0EN00cgr8w==
-
 ms@2.0.0:
   version "2.0.0"
   resolved "https://registry.yarnpkg.com/ms/-/ms-2.0.0.tgz#5608aeadfc00be6c2901df5f9861788de0d597c8"

redisinsight/desktop/src/lib/aboutPanel/aboutPanel.ts

@@ -8,7 +8,7 @@ const ICON_PATH = app.isPackaged
 
 export const AboutPanelOptions = {
   applicationName: 'RedisInsight',
-  applicationVersion: `${app.getVersion() || '2.36.0'}${
+  applicationVersion: `${app.getVersion() || '2.38.0'}${
     !config.isProduction ? `-dev-${process.getCreationTime()}` : ''
   }`,
   copyright: `Copyright © ${new Date().getFullYear()} Redis Ltd.`,

redisinsight/package.json

@@ -2,7 +2,7 @@
   "name": "redisinsight",
   "productName": "RedisInsight",
   "private": true,
-  "version": "2.36.0",
+  "version": "2.38.0",
   "description": "RedisInsight",
   "main": "./dist/main/main.js",
   "author": {

redisinsight/ui/src/pages/browser/modules/key-details/components/string-details/StringDetails.spec.tsx

@@ -1,7 +1,7 @@
 import React from 'react'
 import { instance, mock } from 'ts-mockito'
 import { render, screen } from 'uiSrc/utils/test-utils'
-import { stringDataSelector } from 'uiSrc/slices/browser/string'
+import { stringDataSelector, stringSelector } from 'uiSrc/slices/browser/string'
 import { Props, StringDetails } from './StringDetails'
 
 const mockedProps = mock<Props>()
@@ -15,6 +15,9 @@ jest.mock('uiSrc/slices/browser/string', () => ({
       data: [49, 50, 51, 52],
     }
   }),
+  stringSelector: jest.fn().mockReturnValue({
+    isCompressed: false
+  })
 }))
 
 jest.mock('uiSrc/slices/browser/keys', () => ({
@@ -64,6 +67,22 @@ describe('StringDetails', () => {
     expect(editValueBtn).toHaveProperty('disabled', true)
   })
 
+  it('should not be able to change value (compressed)', () => {
+    const stringSelectorMock = jest.fn().mockReturnValue({
+      isCompressed: true
+    })
+    stringSelector.mockImplementation(stringSelectorMock)
+
+    render(
+      <StringDetails
+        {...mockedProps}
+      />
+    )
+
+    const editValueBtn = screen.getByTestId(`${EDIT_VALUE_BTN_TEST_ID}`)
+    expect(editValueBtn).toHaveProperty('disabled', true)
+  })
+
   it('"edit-key-value-btn" should render', () => {
     const { queryByTestId } = render(<StringDetails {...instance(mockedProps)} />)
     expect(queryByTestId('edit-key-value-btn')).toBeInTheDocument()

redisinsight/ui/src/pages/browser/modules/key-details/components/string-details/StringDetails.tsx

@@ -56,7 +56,7 @@ const StringDetails = (props: Props) => {
     <EditItemAction
       title="Edit Value"
       tooltipContent={editToolTip}
-      isEditable={isStringEditable}
+      isEditable={isStringEditable && isEditable}
       onEditItem={() => setEditItem(!editItem)}
     />
   )