break validation chain on first error

[kpadu]

I apologize in advanced if I missed this somewhere in the docs, but say I have the following code:

check('foo')
    .exists().withMessage('some message')
    .custom(...)

Is there a way that I can prevent custom from running if the exists check fails?

[gustavohenke]

It's not possible yet. #500 implements a basic version of it.

[BigFax]

#500 isn't implemented yet. @gustavohenke Do you know when it would be available ? :)

[pkyeck]

I have the same problem with sanitizers that I want to run - but only when all validators were successful. Same issue? And what is the status of #751?

[gustavohenke]

Hey all! This is out with v6.2.0! 🚢

[ItsGosho]

Hey ,the bail() is working perfectly ,but only on one chain:

Router.all('/test', [
        Check('chushka').custom(UserRequestValidators.containsA).bail()
            .custom(UserRequestValidators.containsB).bail()
            .custom(UserRequestValidators.containsC).bail()
    ],
    (request, response, next) => {
        const errors = ValidationResult(request);
        if (!errors.isEmpty()) {
            return response.status(400).json({errors: errors.array()});
        }
        next();
    });

But it doesn't want to when they are no different chain ,how can I do that?

Router.all('/test', [
        Check('chushka').custom(UserRequestValidators.containsA).bail(),
        Check('chushka').custom(UserRequestValidators.containsB).bail(),
        Check('chushka').custom(UserRequestValidators.containsC).bail()
    ],
    (request, response, next) => {
        const errors = ValidationResult(request);
        if (!errors.isEmpty()) {
            return response.status(400).json({errors: errors.array()});
        }
        next();
    });

[gustavohenke]

Hi @ItsGosho, that's because different chains don't know about the errors generated by other chains, they are isolated.
Also because the way that you wrote these, they work in parallel, so it would even be more difficult to detect errors.

[ItsGosho]

@gustavohenke Hey,
I saw the two type of chains yesterday and made the idea that I wanted with a body ,so all of the validations can be at one chain ,like the example in your documentation,but this is like hacking ...
Can you make something like a one chain ,with multiple validations ,like check(..).custom(...).check(...).custom(...),because i achieved the following with a body and then in custom I get the value ,which is not so bad ,but it can be better :)
BWT the middleware is amazing ,keep it up 👍

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.