fix: empty params are handled as if they were not provided at all

Author: panva

lib/helpers/params.js

@@ -10,7 +10,9 @@ module.exports = function getParams(whitelist) {
 
     const klass = class Params {
       constructor(params) {
-        whitelist.forEach((prop) => { this[prop] = params[prop]; });
+        whitelist.forEach((prop) => {
+          this[prop] = params[prop] || undefined;
+        });
       }
 
       toPlainObject() {

lib/helpers/validate_presence.js

@@ -3,7 +3,10 @@ const { InvalidRequest } = require('./errors');
 module.exports = function validatePresence(ctx, ...required) {
   const { params } = ctx.oidc;
   const missing = required.map((param) => {
-    if (params[param] === undefined) return param;
+    if (params[param] === undefined) {
+      return param;
+    }
+
     return undefined;
   }).filter(Boolean);
 

test/body_parser/body_parser.test.js

@@ -78,7 +78,7 @@ describe('body parser', () => {
         .expect(200);
 
       expect(
-        spy.calledWithMatch({ oidc: { params: { scope: '' } } }),
+        spy.calledWithMatch({ oidc: { params: { scope: undefined } } }),
       ).to.be.true;
     });
   });