From f062e2b37fcc0c633037030addf9ec4c7c7746c8 Mon Sep 17 00:00:00 2001 From: Joey Baker Date: Sun, 4 Sep 2016 16:49:07 -0700 Subject: [PATCH] Fix (validation) params is a plain object If we use a Joi object here, we can't use `defaultsDeep` to extend b/c the joi prototype won't extend cleanly. We'd need to use joi's `contact` method, but that gets really complicated and error prone. So, just use a plain object which is more correct anyway. http://hapijs.com/tutorials/validation --- src/crud.js | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/crud.js b/src/crud.js index 5050dd0..0006b3b 100644 --- a/src/crud.js +++ b/src/crud.js @@ -257,9 +257,9 @@ export const get = ({ server, model, prefix = '/', config }) => { }, config: _.defaultsDeep(config, { validate: { - params: joi.object().keys({ + params: { id: joi.any(), - }), + }, }, }), }); @@ -285,9 +285,9 @@ export const scope = ({ server, model, prefix = '/', config }) => { }, config: _.defaultsDeep(config, { validate: { - params: joi.object().keys({ + params: { scope: joi.string().valid(...scopes), - }), + }, }, }), }); @@ -372,9 +372,9 @@ export const destroyScope = ({ server, model, prefix = '/', config }) => { }, config: _.defaultsDeep(config, { validate: { - params: joi.object().keys({ + params: { scope: joi.string().valid(...scopes), - }), + }, }, }), }); @@ -400,9 +400,9 @@ export const update = ({ server, model, prefix = '/', config }) => { config: _.defaultsDeep(config, { validate: { payload: joi.object().required(), - params: joi.object().keys({ + params: { id: joi.any(), - }), + }, }, }), });