Refactor to maintain url and content type

Author: jaxoncreed

lib/handlers/post.js

@@ -86,7 +86,7 @@ async function handler (req, res, next) {
     const extension = mimeType in extensions ? `.${extensions[mimeType][0]}` : ''
 
     ldp.post(req.hostname, containerPath, req,
-      { slug, extension, container: links.isBasicContainer }).then(
+      { slug, extension, container: links.isBasicContainer, contentType }).then(
       resourcePath => {
         debug('File stored in ' + resourcePath)
         header.addLinks(res, links)

lib/ldp.js

@@ -129,7 +129,7 @@ class LDP {
     }
   }
 
-  async post (host, containerPath, stream, { container, slug, extension }) {
+  async post (hostname, containerPath, stream, { container, slug, extension, contentType }) {
     const ldp = this
     debug.handlers('POST -- On parent: ' + containerPath)
     // prepare slug
@@ -144,26 +144,26 @@ class LDP {
       extension = ''
     }
     // TODO: possibly package this in ldp.post
-    let resourcePath = await ldp.getAvailablePath(host, containerPath, { slug, extension })
-    debug.handlers('POST -- Will create at: ' + resourcePath)
-    let originalPath = resourcePath
+    let resourceUrl = await ldp.getAvailableUrl(hostname, containerPath, { slug, extension })
+    debug.handlers('POST -- Will create at: ' + resourceUrl)
+    let originalUrl = resourceUrl
     if (container) {
       // Create directory by an LDP PUT to the container's .meta resource
-      resourcePath = join(originalPath, ldp.suffixMeta)
-      if (originalPath && !originalPath.endsWith('/')) {
-        originalPath += '/'
+      resourceUrl = `${resourceUrl}${resourceUrl.endsWith('/') ? '' : '/'}${ldp.suffixMeta}`
+      if (originalUrl && !originalUrl.endsWith('/')) {
+        originalUrl += '/'
       }
     }
-    const { url: putUrl, contentType } = await this.resourceMapper.mapFileToUrl({ path: resourcePath, hostname: host })
+    // const { url: putUrl } = await this.resourceMapper.mapFileToUrl({ path: resourceUrl, hostname })
 
     // HACK: the middleware in webid-oidc.js uses body-parser, thus ending the stream of data
     // for JSON bodies. So, the stream needs to be reset
     if (contentType.includes('application/json')) {
       stream = intoStream(JSON.stringify(stream.body))
     }
 
-    await ldp.put(putUrl, stream, contentType)
-    return originalPath
+    await ldp.put(resourceUrl, stream, contentType)
+    return URL.parse(originalUrl).path
   }
 
   /**
@@ -432,11 +432,11 @@ class LDP {
     }
   }
 
-  async getAvailablePath (host, containerURI, { slug = uuid.v1(), extension }) {
+  async getAvailableUrl (hostname, containerURI, { slug = uuid.v1(), extension }) {
     const fileName = slug + extension
     function ensureNotExists (self, newPath) {
       // Verify whether the new path already exists
-      return self.exists(host, newPath).then(
+      return self.exists(hostname, newPath).then(
         // If it does, generate another one
         () => ensureNotExists(self, URI.joinPaths(containerURI,
                 `${uuid.v1().split('-')[0]}-${fileName}`).toString()),
@@ -445,7 +445,7 @@ class LDP {
       )
     }
     const relativePath = await ensureNotExists(this, URI.joinPaths(containerURI, fileName).toString())
-    return this.resourceMapper.resolveRelativeFilePath(host, relativePath)
+    return this.resourceMapper.resolveUrl(hostname, relativePath)
   }
 
   getTrustedOrigins (req) {