Skip to content

Commit 36a935f

Browse files
Change session settings to store less in the session store. Closes #604.
1 parent 4d1dcc1 commit 36a935f

File tree

1 file changed

+5
-38
lines changed

1 file changed

+5
-38
lines changed

app.js

+5-38
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ var methodOverride = require('method-override');
1313
var morgan = require('morgan');
1414
var bodyParser = require('body-parser');
1515
var compression = require('compression');
16-
var cookieParser = require('cookie-parser');
1716
var favicon = require('serve-favicon');
1817

1918
var minify = null;
@@ -39,8 +38,6 @@ var sessionSecret = process.env.SESSION_SECRET || settings.secret;
3938
var db = mongoose.connection;
4039
var dbOptions = { server: { socketOptions: { keepAlive: 1 } } };
4140

42-
var scriptStorage = require('./controllers/scriptStorage');
43-
4441
app.set('port', process.env.PORT || 8080);
4542

4643
// Connect to the database
@@ -83,46 +80,16 @@ app.use(bodyParser.json({
8380
app.use(compression());
8481
app.use(methodOverride('X-HTTP-Method-Override'));
8582

86-
// Intercept script/library/metadata requests to prevent
87-
// the creation of useless session data
88-
app.use(function (aReq, aRes, aNext) {
89-
var matches = null;
90-
91-
if (aReq.method === 'GET' &&
92-
(matches =
93-
/^\/(install|meta|src)(?:\/(scripts|libs))?\/([^\/]+)\/([^\/]+)/
94-
.exec(aReq.url))) {
95-
96-
// Set route parameters to mimick express route middleware
97-
aReq.params = {};
98-
if (matches[1] === 'src' && matches[2]) {
99-
aReq.params.type = matches[2];
100-
}
101-
aReq.params.username = matches[3];
102-
aReq.params.scriptname = matches[4];
103-
104-
switch (matches[1]) {
105-
case 'meta':
106-
scriptStorage.sendMeta(aReq, aRes, aNext);
107-
break;
108-
default:
109-
scriptStorage.sendScript(aReq, aRes, aNext);
110-
break;
111-
}
112-
} else {
113-
aNext();
114-
}
115-
});
116-
11783
// Order is very important here (i.e mess with at your own risk)
118-
app.use(cookieParser());
84+
app.use(passport.initialize());
11985
app.use(session({
120-
resave: true,
121-
saveUninitialized: true,
86+
resave: false,
87+
saveUninitialized: false,
88+
unset: 'destroy',
89+
cookie: { maxAge: null },
12290
secret: sessionSecret,
12391
store: sessionStore
12492
}));
125-
app.use(passport.initialize());
12693
app.use(modifySessions.init(sessionStore));
12794
app.use(favicon(__dirname + '/public/images/favicon.ico'));
12895

0 commit comments

Comments
 (0)