Mongoose returning string instead of object

Question

I am experiencing a slight issue while trying to query mongodb using mongoose. Here is a sample of my code:

...
resolve: (root, params) => {
  var user = User.findOne({email: params.data.email}).exec()
  return user;
}
...

This returns the following:

{
  "data": {
"login": "{ _id: 596f4cc4f51fa12bf0f5a001,\n  updatedAt: 2017-07-19T12:12:52.736Z,\n  createdAt: 2017-07-19T12:12:52.736Z,\n  email: '[email protected]',\n  password: '$2a$12$bhPG4TPGR6by/UBTeAnzq.lyxhfMAJnBymDbkFDIHWl5.XF2JG62O',\n  __v: 0 }"

} }

I have no idea why this is happening. Any help would be appreciated.

EDIT
Here is the full code :

var bcrypt = require('bcryptjs');
var _ = require('lodash');
var { GraphQLNonNull, GraphQLString } = require('graphql');
var jwt = require('jsonwebtoken');

var { UserInputType } = require('./UserSchema');
var User = require('./UserModel');

var login = {
  type: new GraphQLNonNull(GraphQLString),
  args: {
    data: {
        name: 'Data',
        type: new GraphQLNonNull(UserInputType)
    }
  },
  resolve: (root, params, { SECRET }) => {
    var user = User.findOne({email: params.data.email}).exec();
    var authorized = bcrypt.compareSync(params.data.password, user.password);
    if (!authorized) throw new Error('Invalid password');
    var token = jwt.sign({
        user: _.pick(user, ['_id', 'name'])
    }, SECRET, {expiresIn: '1y'});
    return token;
  }
};

Mustehssun Iqbal · Accepted Answer · 2017-07-19 19:28:36Z

1

This might be in the function which calls resolve and calls .then of this user promise. Can't tell for sure without more code...

answered Jul 19, 2017 at 19:28

Mustehssun Iqbal

5563 silver badges19 bronze badges

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

2 Comments

Mustehssun Iqbal Over a year ago

This function is just returning a promise. There are clearly some modifications made to the data in that promise's resolve function...

wmik Over a year ago

Apparently it only occurs in this specific part of the code. I have somewhat similar code elsewhere which returns an object as expected.

wmik · Accepted Answer · 2017-07-20 21:07:31Z

0

resolve: (root, params, { SECRET }) => {
    return new Promise(resolve => {
        User.findOne({email: params.data.email}).exec().then(user => {
            bcrypt.compare(params.data.password, user.password, (err, authorized) => {
                if (!authorized) throw new Error('Invalid password');
                var token = jwt.sign({
                    user: _.pick(user, ['_id', 'name'])
                }, SECRET, {expiresIn: '1y'});
                resolve(token);
            });
        });
    }).then(token => token);
}

Decided to do this instead and voila! Solved my issue. Thanks for the help.

edited Jul 20, 2017 at 21:07

answered Jul 20, 2017 at 17:18

wmik

7841 gold badge10 silver badges25 bronze badges

Collectives™ on Stack Overflow

Mongoose returning string instead of object

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related