Merge pull request #14 from Emyrk/stevenmasley/kasm

bpmct · web-flow · commit 99ed112f7c2d · 2024-08-19T09:54:01.000-05:00
chore: kasmweb client to honor path based apps
diff --git a/better-vnc/for-dogfood/build/Dockerfile b/better-vnc/for-dogfood/build/Dockerfile
@@ -10,4 +10,15 @@ RUN echo "kasm-user ALL=(ALL) NOPASSWD:ALL" >>/etc/sudoers.d/nopasswd
 RUN DEBIAN_FRONTEND=noninteractive make-ssl-cert generate-default-snakeoil --force-overwrite
 RUN adduser kasm-user ssl-cert
 
+# By default, the kasm web client html page connects to `/websockify` for the websocket connection.
+# This fails for path based apps, as we need to prepend the application path.
+# Replace the line that sets the websocket path to include the application path.
+# Kasm web should only be hosted on the path "/", but just incase it is moved, this regex match
+# will only replace the websocket path if the path is in the format of "/@<username>/<appname>/apps/<appname>/"
+#
+# note: this effect is compiled to some location, so a restart of the container is not enough. The container needs to be
+# rebuilt from scratch (empty volume) to apply.
+RUN sed "s#'websockify'#(window.location.pathname.match(/@[^\\\\/]+\\\\/[^\\\\/]+\\\\/apps\\\\/[^\\\\/]+\\\\//) ?? [''])[0]+'websockify'#g" < $KASM_VNC_PATH/www/dist/main.bundle.js > $KASM_VNC_PATH/www/dist/main.bundle.js.tmp
+RUN mv $KASM_VNC_PATH/www/dist/main.bundle.js.tmp $KASM_VNC_PATH/www/dist/main.bundle.js
+
 USER 1000
