Adjust sanitization and logging

kj800x · Nov 19, 2021 · 6d121b9 · 6d121b9
1 parent 3989776
commit 6d121b9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/localproxy-client/src/client.ts b/localproxy-client/src/client.ts
@@ -6,7 +6,7 @@ import path from "path";
 tmp.setGracefulCleanup();
 
 const LOCALPROXY_CONFIG_DIR = "/etc/localproxy/sites";
-const sanitize = (s: string) => s.replace(/[^a-z0-9]/gi, "_");
+const sanitize = (s: string) => s.replace(/[^a-z0-9-]/gi, "-");
 let tmpFileCleanups: { [key: string]: () => void } = {};
 
 export type RouteType = "ui" | "api" | "data";

diff --git a/localproxy-server/store.js b/localproxy-server/store.js
@@ -9,7 +9,7 @@ const WATCH_DIR = "/etc/localproxy/sites";
 let apps = {};
 let onSyncHandler = () => {};
 
-const sanitize = (s) => s.replace(/[^a-z0-9]/gi, "_");
+const sanitize = (s) => s.replace(/[^a-z0-9-]/gi, "-");
 
 const forceSync = () => {
   const files = fs.readdirSync(WATCH_DIR);
@@ -52,6 +52,8 @@ function deRegister(id) {
   const file = `${sanitize(id)}.json`;
   if (fs.existsSync(path.join(WATCH_DIR, file))) {
     fs.unlinkSync(path.join(WATCH_DIR, file));
+  } else {
+    console.warn(`Unable to deregister ${id}, does a ${file} file exist?`);
   }
   forceSync();
 }