Resource mapper properly encodes special characters in the uri

Author: jaxoncreed

lib/resource-mapper.js

@@ -78,7 +78,10 @@ class ResourceMapper {
 
     // Determine the URL by chopping off everything after the dollar sign
     const pathname = this._removeDollarExtension(path)
-    const url = `${this.resolveUrl(hostname)}${encodeURI(pathname)}`
+    console.log(pathname)
+    const url = `${this.resolveUrl(hostname)}${
+      pathname.split('/').map((component) => encodeURIComponent(component)).join('/')
+    }`
     return { url, contentType: this._getContentTypeByExtension(path) }
   }
 

test/unit/resource-mapper-test.js

@@ -478,6 +478,13 @@ describe('ResourceMapper', () => {
         url: 'http://localhost/space/foo%20bar%20bar.html',
         contentType: 'text/html'
       })
+
+    itMapsFile(mapper, 'a file with even stranger disallowed IRI characters',
+      { path: `${rootPath}space/Blog discovery for the future? · Issue #96 · scripting:Scripting-News · GitHub.pdf` },
+      {
+        url: 'http://localhost/space/Blog%20discovery%20for%20the%20future%3F%20%C2%B7%20Issue%20%2396%20%C2%B7%20scripting%3AScripting-News%20%C2%B7%20GitHub.pdf',
+        contentType: 'application/pdf'
+      })
   })
 
   describe('A ResourceMapper instance for a multi-host setup', () => {