Federico Ciardi comments

Results 198 comments of


                                            Federico Ciardi

Re-usable validation, with optional() and exists()

1. Yes, because `null` and `undefined` are passed to `validator.js` as empty strings if I am not wrong, but if it wasn't like that it would return an error as...

Re-usable validation, with optional() and exists()

It makes sense, I am quite full of things to do for the next month and I have no time to think about a detailed design and to implement the...

Re-usable validation, with optional() and exists()

> - @fedeci that means if I use a factory function to create a chain like explained here https://express-validator.github.io/docs/validation-chain-api.html, then I can just call that and add `.optional()` to the...

docs: mention `ContextBuilder` returned from `#check()`

Yes, for example in `check` it is exposed here. Maybe we should just document this so that the end user could create his own context with all his methods and...

docs: mention `ContextBuilder` returned from `#check()`

Oops nope, asked review for the wrong PR😄

After updating from 5.3.0 to 6.14.0, validation does not work expected

I @SJSusan, are you awaiting the promise returned from the `.run` method? ```js async function checker(req, res, next) => { await check(parameter,`${parameter} parameter invalid`) .isLength({ min: 1, max: 256 })...

[Feature Request] Ability to rename parameters

@supermacro are you still interested in raising a PR for this?

[Feature Request] Ability to rename parameters

Ok, I'll work on this!

Sanitizing arrays loses data except first item

Although `sanitizeBody()` is deprecated it also happens with new chain builders. Related to #931 and #883 @gustavohenke do you think that this behaviour should be changed?

Sanitizing arrays loses data except first item

@favph Sure I will, however to require a minimum array length you can use `isArray({ min: 1 })`