Skip to content

Commit fbb5e6d

Browse files
authored
fix: Security upgrade passport from 0.5.3 to 0.6.0 (#3000)
1 parent d3389f8 commit fbb5e6d

File tree

3 files changed

+146
-116
lines changed

3 files changed

+146
-116
lines changed

Parse-Dashboard/Authentication.js

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -55,13 +55,20 @@ function initialize(app, options) {
5555

5656
const cookieSessionSecret = options.cookieSessionSecret || require('crypto').randomBytes(64).toString('hex');
5757
const cookieSessionMaxAge = options.cookieSessionMaxAge;
58-
app.use(require('connect-flash')());
58+
5959
app.use(require('body-parser').urlencoded({ extended: true }));
60-
app.use(require('cookie-session')({
61-
key : 'parse_dash',
62-
secret : cookieSessionSecret,
63-
maxAge : cookieSessionMaxAge
60+
app.use(require('express-session')({
61+
name: 'parse_dash',
62+
secret: cookieSessionSecret,
63+
resave: false,
64+
saveUninitialized: false,
65+
cookie: {
66+
maxAge: cookieSessionMaxAge,
67+
httpOnly: true,
68+
sameSite: 'lax',
69+
}
6470
}));
71+
app.use(require('connect-flash')());
6572
app.use(passport.initialize());
6673
app.use(passport.session());
6774

@@ -80,9 +87,11 @@ function initialize(app, options) {
8087
},
8188
);
8289

83-
app.get('/logout', function(req, res){
84-
req.logout();
85-
res.redirect(`${self.mountPath}login`);
90+
app.get('/logout', function (req, res, next) {
91+
req.logout(function (err) {
92+
if (err) { return next(err); }
93+
res.redirect(`${self.mountPath}login`);
94+
});
8695
});
8796
}
8897

package-lock.json

Lines changed: 127 additions & 106 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)