fix(sessions): define maxAge in the cookie property (#210)

In #209, I re-introduced cookie expiration using express-session's maxAge property, but had defined it at the wrong path of the settings object. Now defined in cookie.maxAge instead of just maxAge.
openwhyd · Jul 14, 2019 · 6d3ffbb · 6d3ffbb
1 parent 0742955
commit 6d3ffbb
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/app.js b/app.js
@@ -152,8 +152,10 @@ function start() {
     store: new MongoStore({
       url: makeMongoUrl(params)
     }),
+    cookie: {
+      maxAge: 365 * 24 * 60 * 60 * 1000 // cookies expire in 1 year (provided in milliseconds)
+    },
     name: 'whydSid',
-    maxAge: 365 * 24 * 60 * 60 * 1000, // cookies expire in 1 year (provided in milliseconds)
     resave: false, // required, cf https://www.npmjs.com/package/express-session#resave
     saveUninitialized: false // required, cf https://www.npmjs.com/package/express-session#saveuninitialized
   });