refactor: separate sanitizePaths fn

2024-06-03 23:07:19 +05:30
parent 84445e0d1e
commit 1336b0c750
1 changed files with 12 additions and 8 deletions
@@ -1,3 +1,14 @@
+const sanitizePaths = paths => {
+  return paths.filter(path => {
+    if (!path) return false;
+    if (path === null && path === '') return false;
+    // if path is just a sequence of slashes
+    if (/^\/+$/.test(path)) return false;
+
+    return true;
+  });
+};
+
 /**
 * Join multiple paths together with a base URL
 * NOTE: This function is not designed to handle query strings or fragments
@@ -9,14 +20,7 @@
 export function joinUrl(baseUrl, ...paths) {
  // remove empty undefined and null path items
  // also handle if the path is just a slash or just multiple slashes only
-  const sanitizedPaths = paths.filter(path => {
-    if (!path) return false;
-    if (path === null && path === '') return false;
-    // if path is just a sequence of slashes
-    if (/^\/+$/.test(path)) return false;
-
-    return true;
-  });
+  const sanitizedPaths = sanitizePaths(paths);

  const fullUrl = sanitizedPaths.reduce(
    (acc, path) => {