adapt-authoring-server 1.4.0 → 1.4.2

package/lib/Router.js

@@ -79,6 +79,9 @@ class Router {
     if (_.isString(this.parentRouter.path)) {
       p += this.parentRouter.path
     }
+    if (p[p.length - 1] === '/' && this.root[0] === '/') {
+      return p + this.root.slice(1)
+    }
     if (p[p.length - 1] !== '/' && this.root[0] !== '/') {
       p += '/'
     }

package/lib/ServerUtils.js

@@ -183,7 +183,7 @@ class ServerUtils {
     }
     ['body', 'params', 'query'].forEach(attr => {
       if (!req[attr]) {
-        return storeVal(attr, true)
+        return storeVal(attr, false)
       }
       const entries = Object.entries(req[attr])
       let deleted = 0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "adapt-authoring-server",
-  "version": "1.4.0",
+  "version": "1.4.2",
   "description": "Provides an Express application routing and more",
   "homepage": "https://github.com/adapt-security/adapt-authoring-server",
   "license": "GPL-3.0",

package/tests/Router.spec.js

@@ -98,7 +98,7 @@ describe('Router', () => {
       const childRouter = new Router('/users', parentRouter)
 
       // Documents existing behavior: double slashes are allowed
-      assert.equal(childRouter.path, '/api//users')
+      assert.equal(childRouter.path, '/api/users')
     })
 
     it('should handle root path without parent path property', () => {

package/tests/ServerUtils.spec.js

@@ -76,9 +76,9 @@ describe('ServerUtils', () => {
 
       ServerUtils.addExistenceProps(req, res, next)
 
-      assert.equal(req.hasBody, true)
-      assert.equal(req.hasParams, true)
-      assert.equal(req.hasQuery, true)
+      assert.equal(req.hasBody, false)
+      assert.equal(req.hasParams, false)
+      assert.equal(req.hasQuery, false)
     })
 
     it('should mark has* false when all entries are null or undefined', () => {