From e437c3fecf8c22d592b6b29fd14a9db15bf1f2ce Mon Sep 17 00:00:00 2001
From: Remco Ros <remcoros@live.nl>
Date: Fri, 4 Oct 2024 12:14:48 +0000
Subject: [PATCH] add 'reconnect' setting and patch noVNC to fix issue after
 browser tab has been idle.

---
 Dockerfile                       | 38 +++++++++++++++++++++-
 Dockerfile.aarch64               | 43 ++++++++++++++++++++++++-
 Makefile                         |  5 +--
 docker_entrypoint.sh             |  7 +++++
 manifest.yaml                    |  5 +--
 patches/novnc.patch              | 54 ++++++++++++++++++++++++++++++++
 scripts/procedures/getConfig.ts  |  6 ++++
 scripts/procedures/migrations.ts | 21 ++++++++++++-
 8 files changed, 172 insertions(+), 7 deletions(-)
 create mode 100644 patches/novnc.patch

diff --git a/Dockerfile b/Dockerfile
index 59d806a..9df0050 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,4 +1,39 @@
-FROM ghcr.io/linuxserver/baseimage-kasmvnc:debianbookworm-12c6f55b-ls87 AS buildstage
+# taken from https://github.com/linuxserver/docker-baseimage-kasmvnc/blob/debianbookworm/Dockerfile
+# modified to apply 'novnc.patch' (fixing a disconnect/reconnect issue)
+FROM node:12-buster AS wwwstage
+
+ARG KASMWEB_RELEASE="5ba4695e6526a27b8e38ec8d55dc33b39143e68a"
+
+RUN \
+  echo "**** build clientside ****" && \
+  mkdir /src && \
+  cd /src && \
+  wget https://github.com/kasmtech/noVNC/tarball/${KASMWEB_RELEASE} -O - \
+    | tar  --strip-components=1 -xz
+
+COPY ./patches/novnc.patch /src/
+RUN \
+  export QT_QPA_PLATFORM=offscreen && \
+  export QT_QPA_FONTDIR=/usr/share/fonts && \
+  echo "apply novnc.patch" && \
+  cd /src && \
+  patch -p1 -i novnc.patch && \
+  npm install && \
+  npm run-script build
+
+RUN \
+  echo "**** organize output ****" && \
+  mkdir /build-out && \
+  cd /src && \
+  rm -rf node_modules/ && \
+  cp -R ./* /build-out/ && \
+  cd /build-out && \
+  rm *.md && \
+  rm AUTHORS && \
+  cp index.html vnc.html && \
+  mkdir Downloads
+
+FROM ghcr.io/linuxserver/baseimage-kasmvnc:debianbookworm-f7a8978f-ls89 AS buildstage
 
 # these are specified in Makefile
 ARG ARCH
@@ -107,6 +142,7 @@ RUN \
 FROM scratch
 
 COPY --from=buildstage / .
+COPY --from=wwwstage /build-out /usr/local/share/kasmvnc/www
 
 # since we start from scratch, we need these env variables from the base images
 ENV \
diff --git a/Dockerfile.aarch64 b/Dockerfile.aarch64
index 745bd9b..d8b8799 100644
--- a/Dockerfile.aarch64
+++ b/Dockerfile.aarch64
@@ -1,4 +1,44 @@
-FROM ghcr.io/linuxserver/baseimage-kasmvnc:arm64v8-debianbookworm-12c6f55b-ls87 AS buildstage
+# taken from https://github.com/linuxserver/docker-baseimage-kasmvnc/blob/debianbookworm/Dockerfile.aarch64
+# modified to apply 'novnc.patch' (fixing a disconnect/reconnect issue)
+FROM node:12-buster AS wwwstage
+
+ARG KASMWEB_RELEASE="5ba4695e6526a27b8e38ec8d55dc33b39143e68a"
+
+RUN \
+  echo "**** install build deps ****" && \
+  apt-get update && \
+  apt-get install -y phantomjs
+
+RUN \
+  echo "**** build clientside ****" && \
+  mkdir /src && \
+  cd /src && \
+  wget https://github.com/kasmtech/noVNC/tarball/${KASMWEB_RELEASE} -O - \
+    | tar  --strip-components=1 -xz
+
+COPY ./patches/novnc.patch /src/
+RUN \
+  export QT_QPA_PLATFORM=offscreen && \
+  export QT_QPA_FONTDIR=/usr/share/fonts && \
+  echo "apply novnc.patch" && \
+  cd /src && \
+  patch -p1 -i novnc.patch && \
+  npm install && \
+  npm run-script build
+
+RUN \
+  echo "**** organize output ****" && \
+  mkdir /build-out && \
+  cd /src && \
+  rm -rf node_modules/ && \
+  cp -R ./* /build-out/ && \
+  cd /build-out && \
+  rm *.md && \
+  rm AUTHORS && \
+  cp index.html vnc.html && \
+  mkdir Downloads
+
+FROM ghcr.io/linuxserver/baseimage-kasmvnc:arm64v8-debianbookworm-f7a8978f-ls89 AS buildstage
 
 # these are specified in Makefile
 ARG ARCH
@@ -107,6 +147,7 @@ RUN \
 FROM scratch
 
 COPY --from=buildstage / .
+COPY --from=wwwstage /build-out /usr/local/share/kasmvnc/www
 
 # since we start from scratch, we need these env variables from the base images
 ENV \
diff --git a/Makefile b/Makefile
index cafaca1..607ff55 100644
--- a/Makefile
+++ b/Makefile
@@ -11,6 +11,7 @@ PKG_VERSION := $(shell yq e ".version" manifest.yaml)
 TS_FILES := $(shell find ./ -name \*.ts)
 ROOT_FILES := $(shell find ./root)
 ASSET_FILES := $(shell find ./assets/compat)
+PATCH_FILES := $(shell find ./patches)
 
 .DELETE_ON_ERROR:
 
@@ -43,7 +44,7 @@ clean:
 scripts/embassy.js: $(TS_FILES)
 	deno bundle scripts/embassy.ts scripts/embassy.js
 
-docker-images/aarch64.tar: manifest.yaml Dockerfile.aarch64 docker_entrypoint.sh $(ROOT_FILES)
+docker-images/aarch64.tar: manifest.yaml Dockerfile.aarch64 docker_entrypoint.sh $(ROOT_FILES) $(PATCH_FILES)
 ifeq ($(ARCH),x86_64)
 else
 	mkdir -p docker-images
@@ -58,7 +59,7 @@ else
 		--platform=linux/arm64 -o type=docker,dest=docker-images/aarch64.tar -f Dockerfile.aarch64 .
 endif
 
-docker-images/x86_64.tar: manifest.yaml Dockerfile docker_entrypoint.sh $(ROOT_FILES)
+docker-images/x86_64.tar: manifest.yaml Dockerfile docker_entrypoint.sh $(ROOT_FILES) $(PATCH_FILES)
 ifeq ($(ARCH),aarch64)
 else
 	mkdir -p docker-images
diff --git a/docker_entrypoint.sh b/docker_entrypoint.sh
index 472b72b..abd39e5 100755
--- a/docker_entrypoint.sh
+++ b/docker_entrypoint.sh
@@ -89,6 +89,13 @@ if [ $(yq e '.sparrow.managesettings' /root/data/start9/config.yaml) = "true" ];
   esac
 fi
 
+# remove reference to non-existing file, see: https://github.com/linuxserver/kclient/issues/8
+sed -i '/<script src="public\/js\/pcm-player\.js"><\/script>/d' /kclient/public/index.html
+
+# add '&reconnect=' setting to kclient html
+RECONNECT=$(yq e '.reconnect' /root/data/start9/config.yaml)
+sed -i "s/\(index\.html?autoconnect=1\)/&\&reconnect=$RECONNECT/" /kclient/public/index.html
+
 # setup a proxy on localhost, Sparrow will not use Tor for local addresses
 # this means we can connect straight to bitcoind/electrs and use Tor for everything else (whirlpool)
 /usr/bin/socat tcp-l:8332,fork,reuseaddr,su=nobody,bind=127.0.0.1 tcp:bitcoind.embassy:8332 &
diff --git a/manifest.yaml b/manifest.yaml
index 3bb8033..9b5a106 100644
--- a/manifest.yaml
+++ b/manifest.yaml
@@ -1,8 +1,9 @@
 id: sparrow-webtop
 title: "Sparrow"
-version: 2.0.0
+version: 2.0.0.1
 release-notes: |
-  * Update to Sparrow 2.0.0 - See [full changelog](https://github.com/sparrowwallet/sparrow/releases/tag/2.0.0)
+  * Add 'Automatically reconnect' option. Enable this to reconnect to the desktop when the connection is lost or the browser tab has been idle for too long.
+  * fix a minor issue when browser tab becomes idle (Cannot read properties of undefined (reading 'lastActiveAt'))
 license: GPLv3
 wrapper-repo: "https://github.com/remcoros/sparrow-webtop-startos"
 upstream-repo: "https://github.com/sparrowwallet/sparrow"
diff --git a/patches/novnc.patch b/patches/novnc.patch
new file mode 100644
index 0000000..1950f72
--- /dev/null
+++ b/patches/novnc.patch
@@ -0,0 +1,54 @@
+diff --git a/app/ui.js b/app/ui.js
+index d973ffd..8d9515e 100644
+--- a/app/ui.js
++++ b/app/ui.js
+@@ -1628,15 +1628,17 @@ const UI = {
+         // UI.disconnect() won't be used in those cases.
+         UI.connected = false;
+ 
++        clearInterval(UI._sessionTimeoutInterval);
++
+         UI.rfb = undefined;
+ 
+         if (!e.detail.clean) {
+             UI.updateVisualState('disconnected');
+             if (wasConnected) {
+                 UI.showStatus(_("Something went wrong, connection is closed"),
+-                              'error');
++                              'error', 0, true);
+             } else {
+-                UI.showStatus(_("Failed to connect to server"), 'error');
++                UI.showStatus(_("Failed to connect to server"), 'error', 0, true);
+             }
+         } else if (UI.getSetting('reconnect', false) === true && !UI.inhibitReconnect) {
+             UI.updateVisualState('reconnecting');
+@@ -1646,7 +1648,7 @@ const UI = {
+             return;
+         } else {
+             UI.updateVisualState('disconnected');
+-            UI.showStatus(_("Disconnected"), 'normal');
++            UI.showStatus(_("Disconnected"), 'error', 0, true);
+         }
+ 
+         document.title = PAGE_TITLE;
+diff --git a/vnc.html b/vnc.html
+index e5b5177..565f273 100644
+--- a/vnc.html
++++ b/vnc.html
+@@ -478,13 +478,14 @@
+                                                 </div>
+                                             </li>
+                                             <li><hr></li>
+-                                            <li>
++                                            <li class="noVNC_hidden">
+                                                 <label class="switch">
+                                                     <input id="noVNC_setting_reconnect" type="checkbox"> 
++                                                    <span class="slider round"></span>
+                                                     <span class="slider-label">Automatic Reconnect</span>
+                                                 </label>
+                                             </li>
+-                                            <li>
++                                            <li class="noVNC_hidden">
+                                                 <label for="noVNC_setting_reconnect_delay">Reconnect Delay (ms):</label>
+                                                 <input id="noVNC_setting_reconnect_delay" type="number">
+                                             </li>
diff --git a/scripts/procedures/getConfig.ts b/scripts/procedures/getConfig.ts
index ebdc033..d517590 100644
--- a/scripts/procedures/getConfig.ts
+++ b/scripts/procedures/getConfig.ts
@@ -38,6 +38,12 @@ export const getConfig: T.ExpectedExports.getConfig = compat.getConfig({
     "pattern-description": "Must not contain newline or quote characters.",
     copyable: true,
   },
+  reconnect: {
+    type: "boolean",
+    name: "Automatically reconnect",
+    description: "Automatically reconnect when the connection to the desktop is lost or the browser tab has been idle for too long.",
+    default: false,
+  },
   sparrow: {
     type: "object",
     name: "Sparrow settings",
diff --git a/scripts/procedures/migrations.ts b/scripts/procedures/migrations.ts
index 90159ea..180b417 100644
--- a/scripts/procedures/migrations.ts
+++ b/scripts/procedures/migrations.ts
@@ -1,4 +1,23 @@
 import { compat, types as T } from "../deps.ts";
 
 export const migration: T.ExpectedExports.migration = compat.migrations
-    .fromMapping({}, "2.0.0" );
+  .fromMapping({
+    "2.0.0.1": {
+      up: compat.migrations.updateConfig(
+        (config: any) => {
+          config["reconnect"] = false;
+          return config;
+        },
+        true,
+        { version: "2.0.0.1", type: "up" },
+      ),
+      down: compat.migrations.updateConfig(
+        (config: any) => {
+          delete config["reconnect"];
+          return config;
+        },
+        true,
+        { version: "2.0.0.1", type: "down" },
+      ),
+    },
+  }, "2.0.0.1");