Inconsistent body validation behavior

[payu]

Inconsistent body validation behavior

[kobik]

Hi @setaman , this kind f error message isn't something that should be returned in any circumstances, so it's a bug indeed.

I'm not sure about how this package is handling multipart requests and would require investing some time on it.

[setaman]

Hi @kobik, It is getting even worse. I just found out that the validation of the multipart/form-data body fails, if i set image to be required. The root of all evil is the _validateBody method.

My first issue, where TypeError: validator.validate is not a function is thrown, is caused by:

...
// is undefined, no body --> no Content-Type header
const contentType = this._getContentType(requestOptions.headers);
...
// the validator is there, but we cannot reach it because of the undefined 'contentType'
// the validator is in methodSchema.body.multipart/form-data 
const validator = methodSchema.body[contentType] || methodSchema.body;

I put the validation right after multer in the chain. Multer assigns the image to req.file, so the validation of the body do not pass:

const validator = methodSchema.body[contentType] || methodSchema.body;
// body has no image field
// methodSchema.body.multipart/form-data has no image to
if (!validator.validate(body)) {
  return validator.errors || [];
 }

I really want this library to work correctly and would like to help. But there are inconsistencies in the validation strategies in general. We see that in the described cases the headers are not checked, which in turn leads to the fact that Body cannot be validated. The question is whether you, the creators and maintainers, want to see the validation of the presence of the headers and the body and the correctness of the headers according to the specification as part of the functionality of this library.

[kobik]

Hi @setaman, indeed we'd like this package to be as consistent and validate correctly according to the specification.

You're more than welcome to submit a PR.

[setaman]

@kobik I will try to find some time between my work and studies to look at the issues

[kobik]

Thanks @setaman, much appreciated.