Skip to content

Commit 3fa55b4

Browse files
authored
Merge pull request #2265 from RedisInsight/release/2.28.0
Release/2.28.0 to main
2 parents 64b6ce2 + 86ce285 commit 3fa55b4

File tree

28 files changed

+201
-124
lines changed

28 files changed

+201
-124
lines changed

.circleci/config.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -345,6 +345,8 @@ jobs:
345345
parallelism: << parameters.parallelism >>
346346
steps:
347347
- checkout
348+
- node/install:
349+
node-version: '18.15.0'
348350
- attach_workspace:
349351
at: .
350352
- run: sudo apt-get install net-tools
@@ -1179,6 +1181,12 @@ workflows:
11791181
- windows:
11801182
name: Build app - Windows (stage)
11811183
requires: *stageElectronBuildRequires
1184+
# e2e desktop tests on AppImage build
1185+
- e2e-app-image:
1186+
name: E2ETest (AppImage)
1187+
parallelism: 2
1188+
requires:
1189+
- Build app - Linux (stage)
11821190
# release to AWS (stage)
11831191
- release-aws-test:
11841192
name: Release AWS stage

.circleci/e2e/test.app-image.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ docker-compose -f tests/e2e/rte.docker-compose.yml up --force-recreate -d -V
1515
./tests/e2e/wait-for-redis.sh localhost 12000 && \
1616

1717
# run tests
18-
COMMON_URL=$(tail -n 1 apppath)/resources/app.asar/index.html \
18+
COMMON_URL=$(tail -n 1 apppath)/resources/app.asar/dist/renderer/index.html \
1919
ELECTRON_PATH=$(tail -n 1 apppath)/redisinsight \
2020
SOCKETS_CORS=true \
2121
yarn --cwd tests/e2e dotenv -e .desktop.env yarn --cwd tests/e2e test:desktop:ci

.circleci/e2e/test.exe.cmd

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
@echo off
22

3-
set COMMON_URL=%USERPROFILE%/AppData/Local/Programs/redisinsight/resources/app.asar/index.html
4-
set ELECTRON_PATH=%USERPROFILE%/AppData/Local/Programs/redisinsight/RedisInsight-preview.exe
3+
set COMMON_URL=%USERPROFILE%/AppData/Local/Programs/redisinsight/resources/app.asar/dist/renderer/index.html
4+
set ELECTRON_PATH=%USERPROFILE%/AppData/Local/Programs/redisinsight/RedisInsight-v2.exe
55
set OSS_STANDALONE_HOST=%E2E_CLOUD_DATABASE_HOST%
66
set OSS_STANDALONE_PORT=%E2E_CLOUD_DATABASE_PORT%
77
set OSS_STANDALONE_USERNAME=%E2E_CLOUD_DATABASE_USERNAME%

electron-builder.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
"type": "distribution",
3030
"hardenedRuntime": true,
3131
"darkModeSupport": true,
32-
"bundleVersion": "30",
32+
"bundleVersion": "50",
3333
"icon": "resources/icon.icns",
3434
"artifactName": "${productName}-${os}-${arch}.${ext}",
3535
"entitlements": "resources/entitlements.mac.plist",

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,14 +230,14 @@
230230
"connection-string": "^4.3.2",
231231
"d3": "^7.6.1",
232232
"date-fns": "^2.16.1",
233-
"detect-port": "^1.3.0",
234233
"electron-context-menu": "^3.1.0",
235234
"electron-log": "^4.2.4",
236235
"electron-store": "^8.0.0",
237236
"electron-updater": "^5.0.5",
238237
"file-saver": "^2.0.5",
239238
"formik": "^2.2.9",
240239
"fzstd": "^0.1.0",
240+
"get-port": "^7.0.0",
241241
"gzip-js": "^0.3.2",
242242
"html-entities": "^2.3.2",
243243
"html-react-parser": "^1.2.4",

redisinsight/api/config/default.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ export default {
5858
tlsKey: process.env.SERVER_TLS_KEY,
5959
staticContent: !!process.env.SERVER_STATIC_CONTENT || false,
6060
buildType: process.env.BUILD_TYPE || 'ELECTRON',
61-
appVersion: process.env.APP_VERSION || '2.28.0',
61+
appVersion: process.env.APP_VERSION || '2.28.1',
6262
requestTimeout: parseInt(process.env.REQUEST_TIMEOUT, 10) || 25000,
6363
excludeRoutes: [],
6464
excludeAuthRoutes: [],

redisinsight/api/config/swagger.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const SWAGGER_CONFIG: Omit<OpenAPIObject, 'paths'> = {
55
info: {
66
title: 'RedisInsight Backend API',
77
description: 'RedisInsight Backend API',
8-
version: '2.28.0',
8+
version: '2.28.1',
99
},
1010
tags: [],
1111
};

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ const ICON_PATH = app.isPackaged
88

99
export const AboutPanelOptions = {
1010
applicationName: 'RedisInsight-v2',
11-
applicationVersion: `${app.getVersion() || '2.28.0'}${
11+
applicationVersion: `${app.getVersion() || '2.28.1'}${
1212
!config.isProduction ? `-dev-${process.getCreationTime()}` : ''
1313
}`,
1414
copyright: `Copyright © ${new Date().getFullYear()} Redis Ltd.`,

redisinsight/desktop/src/lib/menu/menu.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -257,6 +257,8 @@ export class MenuBuilder {
257257
accelerator: 'F11',
258258
click: () => {
259259
this.mainWindow.setFullScreen(!this.mainWindow.isFullScreen())
260+
// on Linux menubar is hidden on full screen mode
261+
this.mainWindow.setMenuBarVisibility(true)
260262
}
261263
},
262264
{

redisinsight/desktop/src/lib/server/server.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import detectPort from 'detect-port'
21
import log from 'electron-log'
2+
import getPort, { portNumbers } from 'get-port'
33

44
import { wrapErrorMessageSensitiveData } from 'desktopSrc/utils'
55
import { configMain as config } from 'desktopSrc/config'
@@ -22,7 +22,7 @@ export class ElectronWindowAuthStrategy extends AbstractWindowAuthStrategy {
2222
let backendGracefulShutdown: Function
2323
export const launchApiServer = async () => {
2424
try {
25-
const detectPortConst = await detectPort(port)
25+
const detectPortConst = await getPort({ port: portNumbers(port, port + 1_000) })
2626
process.env.API_PORT = detectPortConst?.toString()
2727
log.info('Available port:', detectPortConst)
2828
const { gracefulShutdown: beGracefulShutdown, app: beApp } = await server()

0 commit comments

Comments
 (0)