Skip to content

Commit

Permalink
webaccess: do not resave session
Browse files Browse the repository at this point in the history
Before this change, the database was spammed with session values.
Modern express-session has this baked in.
See https://www.npmjs.com/package/express-session#resave for docs.
  • Loading branch information
JohnMcLear authored and muxator committed Apr 3, 2020
1 parent 1f0058d commit c2ea2b3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/node/hooks/express/webaccess.js
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ exports.expressConfigure = function (hook_name, args, cb) {
args.app.use(sessionModule({
secret: exports.secret,
store: args.app.sessionStore,
resave: true,
resave: false,
saveUninitialized: true,
name: 'express_sid',
proxy: true,
Expand Down

0 comments on commit c2ea2b3

Please sign in to comment.