From 55ff16c4a23281db860f78d83441a114076080ce Mon Sep 17 00:00:00 2001 From: Siyuan Miao Date: Mon, 24 Nov 2025 13:03:34 +0100 Subject: [PATCH] backport: versioning cloud app (0.3.8) --- ui/package-lock.json | 9 +++++---- ui/package.json | 2 +- ui/src/components/Button.tsx | 6 +++--- ui/src/components/KvmCard.tsx | 27 ++++++++++++++++++++++++++- ui/src/main.tsx | 3 ++- ui/src/routes/devices.tsx | 8 +++++++- 6 files changed, 44 insertions(+), 11 deletions(-) diff --git a/ui/package-lock.json b/ui/package-lock.json index 14ed59b5..ccb839d4 100644 --- a/ui/package-lock.json +++ b/ui/package-lock.json @@ -32,7 +32,7 @@ "react-simple-keyboard": "^3.7.112", "react-xtermjs": "^1.0.9", "recharts": "^2.15.1", - "semver": "^7.7.1", + "semver": "^7.7.3", "tailwind-merge": "^2.5.5", "usehooks-ts": "^3.1.1", "validator": "^13.12.0", @@ -5336,9 +5336,10 @@ } }, "node_modules/semver": { - "version": "7.7.1", - "resolved": "https://registry.npmjs.org/semver/-/semver-7.7.1.tgz", - "integrity": "sha512-hlq8tAfn0m/61p4BVRcPzIGr6LKiMwo4VM6dGi6pt4qcRkmNzTcWq6eCEjEh+qXjkMDvPlOFFSGwQjoEa6gyMA==", + "version": "7.7.3", + "resolved": "https://registry.npmjs.org/semver/-/semver-7.7.3.tgz", + "integrity": "sha512-SdsKMrI9TdgjdweUSR9MweHA4EJ8YxHn8DFaDisvhVlUOe4BF1tLD7GAj0lIqWVl+dPb/rExr0Btby5loQm20Q==", + "license": "ISC", "bin": { "semver": "bin/semver.js" }, diff --git a/ui/package.json b/ui/package.json index fbeebc10..5a37ede2 100644 --- a/ui/package.json +++ b/ui/package.json @@ -41,7 +41,7 @@ "react-simple-keyboard": "^3.7.112", "react-xtermjs": "^1.0.9", "recharts": "^2.15.1", - "semver": "^7.7.1", + "semver": "^7.7.3", "tailwind-merge": "^2.5.5", "usehooks-ts": "^3.1.1", "validator": "^13.12.0", diff --git a/ui/src/components/Button.tsx b/ui/src/components/Button.tsx index 2e9a1fec..57392b79 100644 --- a/ui/src/components/Button.tsx +++ b/ui/src/components/Button.tsx @@ -210,7 +210,7 @@ export const Button = React.forwardRef( Button.displayName = "Button"; -type LinkPropsType = Pick & +type LinkPropsType = Pick & React.ComponentProps & { disabled?: boolean }; export const LinkButton = ({ to, ...props }: LinkPropsType) => { const classes = cx( @@ -223,13 +223,13 @@ export const LinkButton = ({ to, ...props }: LinkPropsType) => { if (to.toString().startsWith("http")) { return ( - + ); } else { return ( - + ); diff --git a/ui/src/components/KvmCard.tsx b/ui/src/components/KvmCard.tsx index a0dc1e2e..aee9ec13 100644 --- a/ui/src/components/KvmCard.tsx +++ b/ui/src/components/KvmCard.tsx @@ -4,6 +4,8 @@ import { MdConnectWithoutContact } from "react-icons/md"; import { Menu, MenuButton, MenuItem, MenuItems } from "@headlessui/react"; import { Link } from "react-router-dom"; import { LuEllipsisVertical } from "react-icons/lu"; +import semver from "semver"; +import { useMemo } from "react"; function getRelativeTimeString(date: Date | number, lang = navigator.language): string { // Allow dates or times to be passed @@ -43,12 +45,33 @@ export default function KvmCard({ id, online, lastSeen, + appVersion, }: { title: string; id: string; online: boolean; lastSeen: Date | null; + appVersion: string; }) { + /** + * Constructs the URL for connecting to this KVM device's interface. + * + * Version 0.4.91 is the last backwards-compatible UI that works with older devices. + * Devices on v0.4.91 or below are served that version, while newer devices get + * their actual version. Unparseable versions fall back to 0.4.91 for safety. + */ + const kvmUrl = useMemo(() => { + const BACKWARDS_COMPATIBLE_VERSION = "0.4.91"; + + // Use device version if valid and >= 0.4.91, otherwise fall back to backwards-compatible version + const shouldUseDeviceVersion = + semver.valid(appVersion) && semver.gte(appVersion, BACKWARDS_COMPATIBLE_VERSION); + const version = shouldUseDeviceVersion ? appVersion : BACKWARDS_COMPATIBLE_VERSION; + + return new URL(`/v/${version}/devices/${id}`, window.location.origin).toString(); + }, [appVersion, id]); + + return (
@@ -87,7 +110,9 @@ export default function KvmCard({ text="Connect to KVM" LeadingIcon={MdConnectWithoutContact} textAlign="center" - to={`/devices/${id}`} + reloadDocument + target="_self" + to={kvmUrl} /> ) : (