moving out configuration items from express.js code to a proper confi…

…guration directive
meanjs · Jan 5, 2014 · ee63f17 · ee63f17
1 parent e2ea8da
commit ee63f17
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/config/env/all.js b/config/env/all.js
@@ -6,5 +6,11 @@ var rootPath = path.normalize(__dirname + '/../..');
 module.exports = {
 	root: rootPath,
 	port: process.env.PORT || 3000,
-	db: process.env.MONGOHQ_URL
+	db: process.env.MONGOHQ_URL,
+
+	// The secret should be set to a non-guessable string that
+	// is used to compute a session hash
+	sessionSecret: 'MEAN',
+	// The name of the MongoDB collection to store sessions in
+	sessionCollection: 'sessions'
 }
diff --git a/config/express.js b/config/express.js
@@ -49,10 +49,10 @@ module.exports = function(app, passport, db) {
 
         // Express/Mongo session storage
         app.use(express.session({
-            secret: 'MEAN',
+            secret: config.sessionSecret,
             store: new mongoStore({
                 db: db.connection.db,
-                collection: 'sessions'
+                collection: config.sessionCollection
             })
         }));