updates

Author: bourgeoa

lib/ldp.js

@@ -243,8 +243,8 @@ class LDP {
         'PUT request requires a content-type via the Content-Type header')
     }
 
-    // reject resource with $ extension
-    const dollarExtensionRegex = /\$\.[^$]*$/
+    // reject resource with % encoded $ extension
+    const dollarExtensionRegex = /%(?:24)\.[^%(?:24)]*$/ // /\$\.[^$]*$/
     if ((url.url || url).match(dollarExtensionRegex)) {
       throw error(400, 'Resource with a $.ext is not allowed by the server')
     }

test/integration/http-test.js

@@ -535,15 +535,15 @@ describe('HTTP APIs', function () {
         .expect(hasHeader('acl', 'baz.ttl' + suffixAcl))
         .expect(201, done)
     })
-    it('should not create a resource with $.ext', function (done) { // alain
-      server.put('/foo/bar/baz$.ttl')
+    it('should not create a resource with % encoded $.ext', function (done) {
+      server.put('/foo/bar/baz%24.ttl')
         .send(putRequestBody)
         .set('content-type', 'text/turtle')
         // .expect(hasHeader('describedBy', 'baz.ttl' + suffixMeta))
         // .expect(hasHeader('acl', 'baz.ttl' + suffixAcl))
         .expect(400, done) // 404
     })
-    it('should not create a resource with $.ext', function (done) { // alain
+    it('should create a resource without extension', function (done) {
       server.put('/foo/bar/baz')
         .send(putRequestBody)
         .set('content-type', 'text/turtle')