Fixes conductor-oss#7, another dormant bug (conductor-oss#9)

* Fixes conductor-oss#7, another dormant bug * Review comment - Combine Regexes
preqin · May 9, 2024 · d5a269b · d5a269b
1 parent 5a8c8d1
commit d5a269b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/ui/src/components/NavLink.jsx b/ui/src/components/NavLink.jsx
@@ -26,7 +26,8 @@ export default React.forwardRef((props, ref) => {
       </Link>
     );
   } else {
-    const href = absolutePath ? url.toString() : cleanDuplicateSlash(getBasename() + url.toString());
+    // Note: + '/' + is required here
+    const href = absolutePath ? url.toString() : cleanDuplicateSlash(getBasename() + '/' + url.toString());
     return (
       <Link ref={ref} target="_blank" href={href}>
         {rest.children}

diff --git a/ui/src/plugins/fetch.js b/ui/src/plugins/fetch.js
@@ -41,6 +41,11 @@ export function fetchWithContext(
     });
 }
 
+/**
+ * @param {string} path 
+ * @returns path with '/' not duplicated, except at ://
+ * 
+ */
 export function cleanDuplicateSlash(path) {
-  return path.replace(/([^:]\/)\/+/g, "$1");
+  return path.replace(/(:\/\/)\/*|(\/)+/g, "$1$2");
 }