fix borked safeJoin

Author: sw-yx

package-lock.json

@@ -16,9 +16,9 @@
     "inquirer": "^6.2.2",
     "netlify": "ssh+git://[email protected]:netlify/js-client-private",
     "netlify-rules-proxy": "git+ssh://[email protected]/netlify/netlify-rules-proxy.git",
-    "safe-join": "^0.1.1",
     "node-fetch": "^2.3.0",
     "opn": "^5.5.0",
+    "safe-join": "^0.1.2",
     "static-dev-server": "^1.0.0",
     "wait-port": "^0.2.2"
   },

package.json

@@ -1,9 +1,9 @@
-const url = require("url");
-const fetch = require("node-fetch");
-const safeJoin = require("safe-join");
+const url = require('url')
+const fetch = require('node-fetch')
+const { safeJoin } = require('safe-join')
 
 // supported repo host types
-const GITHUB = Symbol("GITHUB");
+const GITHUB = Symbol('GITHUB')
 // const BITBUCKET = Symbol('BITBUCKET')
 // const GITLAB = Symbol('GITLAB')
 
@@ -12,52 +12,40 @@ const GITHUB = Symbol("GITHUB");
  * and returns https://api.github.com/repos/netlify-labs/all-the-functions/contents/functions/9-using-middleware
  */
 async function readRepoURL(_url) {
-  const URL = url.parse(_url);
-  const repoHost = validateRepoURL(URL);
-  const [owner_and_repo, contents_path] = parseRepoURL(repoHost, URL);
-  const folderContents = await getRepoURLContents(
-    repoHost,
-    owner_and_repo,
-    contents_path
-  );
-  return folderContents;
+  const URL = url.parse(_url)
+  const repoHost = validateRepoURL(URL)
+  const [owner_and_repo, contents_path] = parseRepoURL(repoHost, URL)
+  const folderContents = await getRepoURLContents(repoHost, owner_and_repo, contents_path)
+  return folderContents
 }
 
 async function getRepoURLContents(repoHost, owner_and_repo, contents_path) {
   // naive joining strategy for now
   if (repoHost === GITHUB) {
     // https://developer.github.com/v3/repos/contents/#get-contents
-    const APIURL = safeJoin(
-      "https://api.github.com/repos",
-      owner_and_repo,
-      "contents",
-      contents_path
-    );
+    const APIURL = safeJoin('https://api.github.com/repos', owner_and_repo, 'contents', contents_path)
     return fetch(APIURL)
       .then(x => x.json())
-      .catch(err =>
-        console.error("Error occurred while fetching ", APIURL, err)
-      );
+      .catch(err => console.error('Error occurred while fetching ', APIURL, err))
   } else {
-    throw new Error("unsupported host ", repoHost);
+    throw new Error('unsupported host ', repoHost)
   }
 }
 
 function validateRepoURL(URL) {
-  if (URL.host !== "github.com")
-    throw new Error("only github repos are supported for now");
+  if (URL.host !== 'github.com') throw new Error('only github repos are supported for now')
   // other validation logic here
-  return GITHUB;
+  return GITHUB
 }
 function parseRepoURL(repoHost, URL) {
   // naive splitting strategy for now
   if (repoHost === GITHUB) {
     // https://developer.github.com/v3/repos/contents/#get-contents
-    const [owner_and_repo, contents_path] = URL.path.split("/tree/master"); // what if it's not master? note that our contents retrieval may assume it is master
-    return [owner_and_repo, contents_path];
+    const [owner_and_repo, contents_path] = URL.path.split('/tree/master') // what if it's not master? note that our contents retrieval may assume it is master
+    return [owner_and_repo, contents_path]
   } else {
-    throw new Error("unsupported host ", repoHost);
+    throw new Error('unsupported host ', repoHost)
   }
 }
 
-module.exports = readRepoURL;
+module.exports = readRepoURL