github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-21 20:47:08 -05:00
Code Issues Projects Releases Packages Wiki Activity

[misc] apply review feedback

Browse source
This commit is contained in:
Jakob Ackermann 2020-05-11 13:08:13 +02:00 committed by Henry Oswald
parent 2a3c2dd3d5
commit 36e81cbe15
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
services/clsi/app.js
View file

@ -192,7 +192,7 @@ app.get('/oops', function(req, res, next) {
app.get('/status', (req, res, next) => res.send('CLSI is alive\n'))
let PROCESS_IS_TOO_OLD = false
Settings.processTooOld = false
if (Settings.processLifespanLimitMs) {
Settings.processLifespanLimitMs +=
Settings.processLifespanLimitMs * (Math.random() / 10)
@ -203,16 +203,16 @@ if (Settings.processLifespanLimitMs) {
setTimeout(() => {
logger.log('shutting down, process is too old')
PROCESS_IS_TOO_OLD = true
Settings.processTooOld = true
}, Settings.processLifespanLimitMs)
}
if (Settings.smokeTest) {
function runSmokeTest() {
if (PROCESS_IS_TOO_OLD) return
if (Settings.processTooOld) return
logger.log('running smoke tests')
smokeTest.triggerRun(err => {
logger.error({ err }, 'smoke tests failed')
if (err) logger.error({ err }, 'smoke tests failed')
setTimeout(runSmokeTest, 30 * 1000)
})
}
@ -220,7 +220,9 @@ if (Settings.smokeTest) {
}
app.get('/health_check', function(req, res) {
if (PROCESS_IS_TOO_OLD) return res.status(500).json({ processToOld: true })
if (Settings.processTooOld) {
return res.status(500).json({ processTooOld: true })
}
smokeTest.sendLastResult(res)
})