From d88502e3315f189af28ac6e86fccfa1406d75b52 Mon Sep 17 00:00:00 2001
From: BoHong Li <a60814billy@gmail.com>
Date: Wed, 12 Apr 2017 06:07:19 +0800
Subject: [PATCH] refactor(app.js): Move passport serialize and deserialize to
 auth module

---
 app.js                | 19 -------------------
 lib/web/auth/index.js | 22 ++++++++++++++++++++++
 2 files changed, 22 insertions(+), 19 deletions(-)

diff --git a/app.js b/app.js
index 3cfa2a18..1d0d9862 100644
--- a/app.js
+++ b/app.js
@@ -154,25 +154,6 @@ app.use(flash())
 app.use(passport.initialize())
 app.use(passport.session())
 
-// serialize and deserialize
-passport.serializeUser(function (user, done) {
-  logger.info('serializeUser: ' + user.id)
-  return done(null, user.id)
-})
-passport.deserializeUser(function (id, done) {
-  models.User.findOne({
-    where: {
-      id: id
-    }
-  }).then(function (user) {
-    logger.info('deserializeUser: ' + user.id)
-    return done(null, user)
-  }).catch(function (err) {
-    logger.error(err)
-    return done(err, null)
-  })
-})
-
 // check uri is valid before going further
 app.use(require('./lib/web/middleware/checkURiValid'))
 
diff --git a/lib/web/auth/index.js b/lib/web/auth/index.js
index 3a203800..18d70572 100644
--- a/lib/web/auth/index.js
+++ b/lib/web/auth/index.js
@@ -1,12 +1,34 @@
 'use strict'
 
 const Router = require('express').Router
+const passport = require('passport')
 
 const config = require('../../config')
 const logger = require('../../logger')
+const models = require('../../models')
 
 const authRouter = module.exports = Router()
 
+// serialize and deserialize
+passport.serializeUser(function (user, done) {
+  logger.info('serializeUser: ' + user.id)
+  return done(null, user.id)
+})
+
+passport.deserializeUser(function (id, done) {
+  models.User.findOne({
+    where: {
+      id: id
+    }
+  }).then(function (user) {
+    logger.info('deserializeUser: ' + user.id)
+    return done(null, user)
+  }).catch(function (err) {
+    logger.error(err)
+    return done(err, null)
+  })
+})
+
 if (config.facebook) authRouter.use('/', require('./facebook'))
 if (config.twitter) authRouter.use('/', require('./twitter'))
 if (config.github) authRouter.use('/', require('./github'))