From e253fe6cee8795252ea27c44140f201dacfa4591 Mon Sep 17 00:00:00 2001
From: Niklas <33813894+Niklas04@users.noreply.github.com>
Date: Thu, 26 Dec 2024 21:34:33 +0100
Subject: [PATCH] ChangeDetection Update: Update also Browsers (#1027)

* Changedetection: Update also browsers

* Changedetection: Adding more msg for updating, Make code more readable
---
 ct/changedetection.sh | 21 +++++++++++++++++++--
 1 file changed, 19 insertions(+), 2 deletions(-)

diff --git a/ct/changedetection.sh b/ct/changedetection.sh
index 6046b399..86901e7d 100644
--- a/ct/changedetection.sh
+++ b/ct/changedetection.sh
@@ -28,28 +28,45 @@ function update_script() {
   header_info
   check_container_storage
   check_container_resources
+
   if [[ ! -f /etc/systemd/system/changedetection.service ]]; then
     msg_error "No ${APP} Installation Found!"
     exit
   fi
-  msg_info "Updating ${APP} LXC"
+
   if ! dpkg -s libjpeg-dev >/dev/null 2>&1; then
+    msg_info "Installing Dependencies"
     apt-get update
     apt-get install -y libjpeg-dev
+    msg_ok "Updated Dependencies"
   fi
+
+  msg_info "Updating ${APP}"
   pip3 install changedetection.io --upgrade &>/dev/null
+  msg_ok "Updated ${APP}"
+
+  msg_info "Updating Playwright"
   pip3 install playwright --upgrade &>/dev/null
+  msg_ok "Updated Playwright"
+
   if [[ -f /etc/systemd/system/browserless.service ]]; then
+    msg_info "Updating Browserless (Patience)"
     git -C /opt/browserless/ fetch --all &>/dev/null
     git -C /opt/browserless/ reset --hard origin/main &>/dev/null
     npm update --prefix /opt/browserless &>/dev/null
+    /opt/browserless/node_modules/playwright-core/cli.js install --with-deps &>/dev/null
+    # Update Chrome separately, as it has to be done with the force option. Otherwise the installation of other browsers will not be done if Chrome is already installed.
+    /opt/browserless/node_modules/playwright-core/cli.js install --force chrome &>/dev/null
+    /opt/browserless/node_modules/playwright-core/cli.js install chromium firefox webkit &>/dev/null
     npm run build --prefix /opt/browserless &>/dev/null
     npm run build:function --prefix /opt/browserless &>/dev/null
     npm prune production --prefix /opt/browserless &>/dev/null
     systemctl restart browserless
+    msg_ok "Updated Browserless"
   else
     msg_error "No Browserless Installation Found!"
   fi
+
   systemctl restart changedetection
   msg_ok "Updated Successfully"
   exit
@@ -62,4 +79,4 @@ description
 msg_ok "Completed Successfully!\n"
 echo -e "${CREATING}${GN}${APP} setup has been successfully initialized!${CL}"
 echo -e "${INFO}${YW} Access it using the following URL:${CL}"
-echo -e "${TAB}${GATEWAY}${BGN}http://${IP}:5000${CL}"
\ No newline at end of file
+echo -e "${TAB}${GATEWAY}${BGN}http://${IP}:5000${CL}"