Merge pull request danielfsousa#16 from gkatsanos/patch-1

You need to use await when consuming the return value of an async function
nixpench · Oct 4, 2017 · ba7d423 · ba7d423
2 parents f5af49c + 7a93b38
commit ba7d423
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/api/models/user.model.js b/src/api/models/user.model.js
@@ -101,8 +101,7 @@ userSchema.method({
   },
 
   async passwordMatches(password) {
-    const matches = await bcrypt.compare(password, this.password);
-    return matches;
+    return bcrypt.compare(password, this.password);
   },
 });
 
@@ -155,7 +154,7 @@ userSchema.statics = {
       isPublic: true,
     };
     if (password) {
-      if (user && user.passwordMatches(password)) {
+      if (user && await user.passwordMatches(password)) {
         return { user, accessToken: user.token() };
       }
       err.message = 'Incorrect email or password';