Skip to content

Commit e71610e

Browse files
authored
Merge pull request #1201 from mcneilco/revert-1200-revert-1197-ACAS-828
Revert "Revert "ACAS-828 Use postgres for session management""
2 parents 16aa697 + 54d13f8 commit e71610e

File tree

3 files changed

+11
-5
lines changed

3 files changed

+11
-5
lines changed

app_template.coffee

+8-4
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ startApp = ->
1515
logger = require('morgan')
1616
methodOverride = require('method-override')
1717
session = require('express-session')
18-
MemoryStore = require('memorystore')(session)
18+
PostgresqlStore = require('connect-pg-simple')(session)
1919
bodyParser = require('body-parser')
2020
errorHandler = require('errorhandler')
2121
cookieParser = require('cookie-parser')
@@ -78,7 +78,9 @@ startApp = ->
7878
console.error("NOT USING SSO configs! config.all.server.security.saml.use is set true but CustomerSpecificServerFunction 'ssoLoginStrategy' is not defined.")
7979

8080
loginRoutes = require './routes/loginRoutes'
81-
sessionStore = new MemoryStore();
81+
sessionStore = new PostgresqlStore(
82+
conString: "postgres://#{config.all.server.database.username}:#{config.all.server.database.password}@#{config.all.server.database.host}:#{config.all.server.database.port}/#{config.all.server.database.name}"
83+
)
8284
global.app = express()
8385
app.set 'port', config.all.client.port
8486
app.set 'listenHost', config.all.client.listenHost
@@ -117,12 +119,14 @@ startApp = ->
117119

118120
# added for login support
119121
app.use cookieParser()
122+
console.log "Session timeout set to #{config.all.server.sessionTimeOutMinutes} minutes"
123+
sessionTimeOutMilliseconds = config.all.server.sessionTimeOutMinutes * 60 * 1000
120124
app.use session
121125
secret: 'acas needs login'
122-
cookie: maxAge: 365 * 24 * 60 * 60 * 1000
126+
cookie: maxAge: sessionTimeOutMilliseconds
123127
resave: true
124128
saveUninitialized: true,
125-
store: sessionStore # MemoryStore is used automatically if no "store" field is set, but we need a handle on the sessionStore object for Socket.IO, so we'll manually create the store so we have a handle on the object
129+
store: sessionStore
126130

127131
app.use flash()
128132
app.use passport.initialize()

conf/config.properties.example

+2
Original file line numberDiff line numberDiff line change
@@ -768,6 +768,8 @@ client.cmpdreg.serverSettings.liveDesign.url=http://localhost:8010/ld-chem
768768

769769
client.cmpdreg.serverSettings.maxStandardizationDisplay=20000
770770

771+
# Sets cookie maxAge to 1440 minutes = 24 hours
772+
server.sessionTimeOutMinutes=1440
771773

772774
# About configs
773775
client.about.acas.version=0.0.0

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
"less": "^3.12.2",
4646
"lodash": "^4.17.20",
4747
"marked": "4.0.10",
48-
"memorystore": "1.6.4",
48+
"connect-pg-simple": "10.0.0",
4949
"method-override": "^3.0.0",
5050
"mocha": "8.2.1",
5151
"mochawesome": "^6.2.1",

0 commit comments

Comments
 (0)