passing variable to next middleware in req in express.js

Question

I am trying to pass variable to the next middlewares through the req object. getting some data from database and passing that data to request for next middlewares to use.

User.findone({ _id: someId })
    .then(user => { req.user = user })
    .catch(err => {  })

After that then and catch block i am using next(). Therefore for the next middlewares i am getting req.user undefined. but if i pass the next() function in the then block after req.user = user like .then(user=> {req.user = user; next()}) than i am getting req.user a valid user object to use for the next middlewares. what is the reason for this behaviour??

Does this answer your question? Express middleware, next and Promises — Aditya
– Aditya, Commented Jul 30, 2020 at 16:26

Duc Nguyen · Accepted Answer · 2020-07-30 16:30:56Z

1

That's because the User.findOne function is asynchronous. The result of that function is only known in the then block.

const middleware = (req, res, next) => {
    User.findOne({ _id: someId })
        .then(user => {
            req.user = user;
        })
        .catch(err => { });
    next(); // If you put next() here, the request will go straight to the next middleware without waiting for User.findOne() to complete.
};

const middleware = (req, res, next) => {
    User.findOne({ _id: someId })
        .then(user => {
            req.user = user;
            next(); // Putting next() here is correct
        })
        .catch(err => {
            next(err); // Remember to handle error to avoid hanging the request
        });
};

answered Jul 30, 2020 at 16:30

Duc Nguyen

8536 silver badges12 bronze badges

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

Comments

userhooke · Accepted Answer · 2020-07-30 16:32:34Z

0

then... is called after the User.findone promise resolves. Thus, if you put next() outside of then, it will be called before then.

You could read more details at promise-basics

Alternatively try to use async-await as it looks more straightforward.

answered Jul 30, 2020 at 16:32

userhooke

867 bronze badges

Collectives™ on Stack Overflow

passing variable to next middleware in req in express.js

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related