PassportJS and user creation with postgres

Question

I've been trying to use pg-promise with passportjs, but I can't get to work local-signup. My login works well. Please see below:

    passport.use(
      "local-signup",
      new LocalStrategy(
        {
          usernameField: "email",
          passwordField: "pass",
          passReqToCallback: true
        },
        (req, email, pass, done) => {
          process.nextTick(function() {
            q.db
              .one("select email from users where email = $1", email)
              .then(data => {
                if (data) {
                  return done(
                    null,
                    false,
                    req.flash("signupMessage", "User already exists")
                  );
                } else {
                  console.log("here!?");
                  q.db
                    .none(
                      "insert into users (email, pass)" +
                        `values (${email}, ${pass})`
                    )
                    .then(data => {
                      console.log("created?");
                    });
                }
              })
              .catch(err => {
                console.log(err);
              });
          });
        }
      )
    );

The problem here is that, it actually detects if an user is already in the database, but if the user doesn't exists, it doesn't create the username, but instead, skips the whole process.

Any ideas how to fix this issue?

Thank you

Where does it fails? Is "here!?" message logged? Try to use second argument on none method for putting values to query — iofjuupasli
– iofjuupasli, Commented Jun 28, 2018 at 15:41
No, "here" does not logs. It goes directly to the catch method: QueryResultError { code: queryResultErrorCode.noData message: "No data returned from the query." received: 0 query: "select email from users where email = '78'" } — Blinhawk
– Blinhawk, Commented Jun 28, 2018 at 15:43
From docs on one method: "When 0 or more than 1 rows are returned, the method rejects" — iofjuupasli
– iofjuupasli, Commented Jun 28, 2018 at 15:45

vitaly-t · Accepted Answer · 2018-06-28 18:53:08Z

2

Your code inside nextTick should be something like this:

q.db.task(async t => {
    const user = await t.oneOrNone('SELECT * FROM users WHERE email = $1', email);
    if (user) {
        return false;
    }
    await t.none('INSERT INTO users(email, pass) VALUES($1, $2)', [email, pass]);
    return true;
})
    .then(created => {
        done(
            null,
            false,
            req.flash('signupMessage', 'Created: ' + created)
        );
    })
    .catch(error => {
        console.log(error);
    });

answered Jun 28, 2018 at 18:53

vitaly-t

26.1k18 gold badges130 silver badges151 bronze badges

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

Collectives™ on Stack Overflow

PassportJS and user creation with postgres

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