Fix commit & notification escaping

Author: DigitallyRefined

README.md

@@ -39,7 +39,7 @@ The app can be started using the following `compose.yml`:
 services:
   containers-up:
     # https://github.com/DigitallyRefined/containers-up/releases
-    image: ghcr.io/digitallyrefined/containers-up:1.4.4
+    image: ghcr.io/digitallyrefined/containers-up:1.4.5
     restart: unless-stopped
     ports:
       - 3000:3000
@@ -63,7 +63,7 @@ Optional system wide configuration can be changed by copying `.env.default` to `
 services:
   containers-up:
     # https://github.com/DigitallyRefined/containers-up/releases
-    image: ghcr.io/digitallyrefined/containers-up:1.4.4
+    image: ghcr.io/digitallyrefined/containers-up:1.4.5
     restart: unless-stopped
     volumes:
       - ./containers-up/storage:/storage

compose.yml

@@ -1,7 +1,7 @@
 services:
   containers-up:
     # https://github.com/DigitallyRefined/containers-up/releases
-    image: ghcr.io/digitallyrefined/containers-up:1.4.4
+    image: ghcr.io/digitallyrefined/containers-up:1.4.5
     container_name: containers-up
     restart: unless-stopped
     volumes:

package.json

@@ -1,6 +1,6 @@
 {
   "name": "containers-up",
-  "version": "1.4.4",
+  "version": "1.4.5",
   "description": "Containers Up! a web based platform designed to manage and update containers",
   "author": "DigitallyRefined",
   "license": "ISC",

src/backend/utils/git.ts

@@ -10,7 +10,7 @@ const SQUASH_UPDATE_MESSAGE = process.env.SQUASH_UPDATE_MESSAGE || 'Update depen
 const SQUASH_DEPS_DAYS_AGO = Number.parseInt(process.env.SQUASH_DAYS_AGO || '5');
 const SQUASH_MAX_UPDATE_COMMITS = Number.parseInt(process.env.SQUASH_MAX_UPDATE_COMMITS || '5');
 const GIT_SQUASH_USER =
-  "GIT_AUTHOR_NAME='Containers Up!' GIT_AUTHOR_EMAIL='260200075+containers-up@users.noreply.github.com'";
+  'GIT_AUTHOR_NAME="Containers Up!" GIT_AUTHOR_EMAIL="260200075+containers-up@users.noreply.github.com"';
 
 const buildMessage = (subject: string, body?: string) => {
   const parts = [subject, body].filter(Boolean);
@@ -179,7 +179,7 @@ export const squashUpdates = async (
     const prevCommit = await getCommitMetadata(sshRun, 1);
 
     const isAutomatedAuthor = (authorName: string) =>
-      /dependabot|renovate|containers-up/i.test(authorName);
+      /dependabot|renovate|containers up/i.test(authorName);
     const X_DAYS_AGO = Date.now() - SQUASH_DEPS_DAYS_AGO * 24 * 60 * 60 * 1000;
 
     if (

src/backend/utils/notification.ts

@@ -5,9 +5,7 @@ const event = 'notification';
 const logger = mainLogger.child({ event });
 const exec = createExec(logger);
 
-const bashEscapeSingleQuoted = (content: string) => {
-  return `'${content.replace(/'/g, `'\\''`)}'`;
-};
+const bashEscapeSingleQuoted = (content: string) => `'${content.replace(/'/g, "'\\''")}'`;
 
 export const sendNotification = async ({
   hostName,