Node-sqlite3 asynchronous callback

Question

I am writing a simple script to retrieve a password from the table and validate in node.js Here is the script

module.exports = {
  login: function (email, pass) {
    var sqlite3 = require('sqlite3').verbose();
    var db = new sqlite3.Database('test.db');

    db.get("SELECT password FROM users WHERE user_email = ?", email, function(err, row) {
      if (err !== null) {
        console.log("An error has occured");
        return "error";
      } else if(row.password === pass) {
        console.log("success");
        return "success";
      } else {
        console.log("Incorrect password");
        return "failure";
      }
    });
  }
};

The console log statements are correct when the if else cases are evaluated. However the return value is undefined. I do not understand why the return value is undefined if the logging is done correctly.

possible duplicate of Why is my variable unaltered after I modify it inside of a function? - Asynchronous code reference — Aaron Dufour
– Aaron Dufour, Commented Oct 12, 2014 at 15:32

mscdex · Accepted Answer · 2014-10-12 14:30:45Z

1

You can't return values from a callback because doing so is meaningless. You have to pass in a callback to your login() function and call that with (err, result) inside your db.get() callback:

module.exports = {
  login: function (email, pass, cb) {
    var sqlite3 = require('sqlite3').verbose();
    var db = new sqlite3.Database('test.db');

    db.get("SELECT password FROM users WHERE user_email = ?",
           email,
           function(err, row) {
      if (err)
        return cb(err);

      cb(null, row.password === pass);
    });
  } 
};

answered Oct 12, 2014 at 14:30

mscdex

107k15 gold badges201 silver badges159 bronze badges

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

Collectives™ on Stack Overflow

Node-sqlite3 asynchronous callback

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related