chore: fix authentication in social-network example (#378)

postlight · Sep 10, 2016 · e4966fd · e4966fd
1 parent bbbe317
commit e4966fd
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 33 deletions.
diff --git a/examples/social-network/app/controllers/users.js b/examples/social-network/app/controllers/users.js
@@ -9,7 +9,7 @@ class UsersController extends Controller {
     'password'
   ];
 
-  async login({
+  login({
     params: {
       data: {
         attributes: {
@@ -19,11 +19,7 @@ class UsersController extends Controller {
       }
     }
   }) {
-    const user = await User.findByEmail(email);
-
-    if (user) {
-      return await user.authenticate(password);
-    }
+    return User.authenticate(email, password);
   }
 }
 

diff --git a/examples/social-network/app/models/user.js b/examples/social-network/app/models/user.js
@@ -1,9 +1,5 @@
 import { Model } from 'lux-framework';
-
-import {
-  encryptPassword,
-  comparePassword
-} from 'app/utils/password';
+import bcrypt from 'bcrypt-as-promised';
 
 class User extends Model {
   static hasMany = {
@@ -38,13 +34,8 @@ class User extends Model {
 
   static hooks = {
     async beforeSave(user) {
-      const { id, password, dirtyAttributes } = user;
-
-      if ((typeof id !== 'number') && password || dirtyAttributes.has('password')) {
-
-        Object.assign(user, {
-          password: encryptPassword(password)
-        });
+      if (user.isNew || user.dirtyAttributes.has('password')) {
+        user.password = await bcrypt.hash(user.password, 10);
       }
     }
   };
@@ -63,8 +54,15 @@ class User extends Model {
     }
   };
 
-  authenticate(password) {
-    return comparePassword(password, this.password);
+  static async authenticate(email, password) {
+    const user = await this.findByEmail(email);
+
+    if (user) {
+      return await bcrypt
+        .compare(password, user.password)
+        .then(() => user)
+        .catch(bcrypt.MISMATCH_ERROR, () => false);
+    }
   }
 }
 

diff --git a/examples/social-network/app/utils/password.js b/examples/social-network/app/utils/password.js
diff --git a/examples/social-network/package.json b/examples/social-network/package.json
@@ -12,10 +12,10 @@
   "dependencies": {
     "babel-core": "6.14.0",
     "babel-preset-lux": "1.2.0",
+    "bcrypt-as-promised": "1.1.0",
     "knex": "0.11.10",
     "lux-framework": "1.0.0-rc.7",
-    "sqlite3": "3.1.4",
-    "bcrypt-as-promised": "1.1.0"
+    "sqlite3": "3.1.4"
   },
   "devDependencies": {
     "faker": "3.1.0"