Callback Not Working in NodeJs

Question

I am trying to handle asynchronous calls in NodeJs. Here is my code:

function getUserFromToken(token) {
  decodeJWT(token, function (err, data) {
    console.log("Call back Worked");
  })
}

function getUserProfile(user_id, username, hashPass, callback) {
  let sql = "SELECT first_name, last_name, username FROM User where user_id=? AND username=? AND password=?";

  db.query(sql, [user_id, username, hashPass], function (err, result) {
    if (err) callback(err, null);
    else callback(null, result)
  });
}

function decodeJWT(token, cb) {
  jwt.verify(token, config.secret, function (err, decoded) {

    if (err) res.status(401).send({auth: false, message: miscConstants.INVALID_TOKEN});

    const {user_id} = decoded;
    const {username} = decoded;
    const {hashPass} = decoded;

    getUserProfile(user_id, username, hashPass, res, function (err, profile) {

      cb(null,'yo');
    });

  });
}

The callback used in the getUserProfile function works but, it never reaches the console.log("Call back Worked");. Any ideas?

handle error properly in decodeJWT you are calling getUserProfile , check if its throwing err. Its better to use promise you dont need to handle all errors — Mohammad Raheem
– Mohammad Raheem, Commented Apr 20, 2018 at 17:30
How did you find - The callback used in the getUserProfile function works ? — Vasan
– Vasan, Commented Apr 20, 2018 at 19:14

TMarafon · Accepted Answer · 2018-04-21 02:34:10Z

1

The callback is not being called because your call to getUserProfile is passing 5 parameters user_id, username, hashPass, res, function, while the function is expecting only 4:

getUserProfile(user_id, username, hashPass, res, function (err, profile) {...}

And the function:

function getUserProfile(user_id, username, hashPass, callback)

To make it work, just delete the parameter res in the getUserProfile call.

answered Apr 21, 2018 at 2:34

TMarafon

3372 silver badges6 bronze badges

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

Collectives™ on Stack Overflow

Callback Not Working in NodeJs

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related