checkNoUnknown()

[CarlosRodar]

So I would like to add this to the v4+ APIs, not to the legacy APIs as they are difficult to implement, modularise and type.

My API suggestion for now is:

app.get('/some/path', [
  check('something').not().isEmpty(),
  checkNoUnknown()
], (req, res, next) => {});

Any better names? 🤔

Originally posted by @gustavohenke in #266 (comment)

Did you added this feature for the library?

[geritol]

Would love to see something like this if there is a possibility for it!

[gustavohenke]

Duplicate of #266 and #578

[alexandernanberg]

I'm using a schema but this seems to work quite well

const schema: Schema = {
  avatar: {
    in: ['body'],
    isURL: true,
    optional: { options: { nullable: true } },
  },
};

const schemaKeys = Object.keys(schema);

await checkSchema({
  ...schema,
  '*': {
    in: ['body'],
    custom: {
      options: (field, { path }) => {
        if (!schemaKeys.includes(path)) {
          throw new Error(`Unknown field: ${path}`);
        }
        return true;
      },
    },
  },
}).run(req);

[gustavohenke]

This is coming to v7 🙂

[gustavohenke]

Hi hi, https://github.com/express-validator/express-validator/releases/tag/v7.0.0 is out with a fix for this 🙂