Skip to content
This repository was archived by the owner on Jul 6, 2022. It is now read-only.

Commit c18ae04

Browse files
committed
Making only logins require https.
1 parent 9730830 commit c18ae04

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

‎server/index.js‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,8 +40,6 @@ app.use(compression())
4040
app.use(bodyParser.json());
4141
app.use(bodyParser.urlencoded({ extended: true }));
4242

43-
app.use(checks.ssl_headers)
44-
app.use(checks.redirect_to_https)
4543
app.use(checks.static_rewrite);
4644

4745
app.set('trust proxy', 1);
@@ -59,7 +57,6 @@ app.use(session({
5957
secret: settings.session_secret,
6058
resave: true,
6159
saveUninitialized: true,
62-
cookie: { secure: (process.env.NODE_ENV === 'development') ? false : true },
6360
store: require('mongoose-session')(mongoose)
6461
}))
6562

‎server/routes/index.js‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,13 @@ var express = require('express')
66
, application = require(__dirname + '/../controllers/application')
77

88
router.route('/login')
9+
.get(checks.ssl_headers)
10+
.get(checks.redirect_to_https)
911
.get(application.login);
1012

1113
router.route('/login')
14+
.post(checks.ssl_headers)
15+
.post(checks.redirect_to_https)
1216
.post(passport.authenticate('atlassian-crowd',
1317
{ failureRedirect:'/login', failureFlash:"Invalid username or password." }))
1418
.post(application.authenticate);

0 commit comments

Comments
 (0)