How to wrap multiple middleware functions into one?

Question

I have a number of middleware functions similar to the following:

function validate(req, res, next) {
  req.validationError = new Error('invalid');
}

function checkValid(req, res, next) {
  if (req.validationError) {
    next(req.validationError);
  } else {
    next();
  }
}

function respond() {
  res.json({result: 'success'});
}

Is there a way to wrap them into one function? So I'd do something like:

function respondIfValid(req, res, next) {
  // Evoke the following middleware: 

  // validate
  // checkValid
  // respond
}

app.use('/', respondIfValid);

Instead of:

app.use('/', validate, checkValid, respond);

Is this goal for this middleware to be an atomic unit? Or another way of asking, what's wrong with just putting all the logic in one function? — Brennan
– Brennan, Commented Apr 8, 2015 at 1:57
I'm generally curious whether middleware can be used in the above way. — sabof
– sabof, Commented Apr 8, 2015 at 11:31

Dineshaws · Accepted Answer · 2015-04-07 13:09:23Z

5

try with following code

app.use('/', [validate, checkValid,respond]);

OR

var middleware = [validate, checkValid,respond];

app.use('/', middleware );

Need to placed all function in that series as your requirement of execution.

Thanks

edited Apr 7, 2015 at 13:09

answered Apr 7, 2015 at 12:52

Dineshaws

2,08317 silver badges26 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

sabof Over a year ago

Ideally middleware would be a function.

Dineshaws Over a year ago

You can wrap up all function without taking in an array. so we can pass directly.

Collectives™ on Stack Overflow

How to wrap multiple middleware functions into one?

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related