Skip to content
Snippets Groups Projects
Commit b6c922f1 authored by Eugen Rochko's avatar Eugen Rochko
Browse files

Fix potential resource leaks in streaming server

parent 1ee4a17f
No related branches found
No related tags found
No related merge requests found
......@@ -45,7 +45,7 @@ const authenticationMiddleware = (req, res, next) => {
return next(err)
}
client.query('SELECT oauth_access_tokens.resource_owner_id, users.account_id FROM oauth_access_tokens INNER JOIN users ON oauth_access_tokens.resource_owner_id = users.id WHERE token = $1 LIMIT 1', [token], (err, result) => {
client.query('SELECT oauth_access_tokens.resource_owner_id, users.account_id FROM oauth_access_tokens INNER JOIN users ON oauth_access_tokens.resource_owner_id = users.id WHERE oauth_access_tokens.token = $1 LIMIT 1', [token], (err, result) => {
done()
if (err) {
......@@ -115,8 +115,13 @@ const streamFrom = (id, req, res, needsFiltering = false) => {
}
})
// Heartbeat to keep connection alive
setInterval(() => res.write(':thump\n'), 15000)
const heartbeat = setInterval(() => res.write(':thump\n'), 15000)
req.on('close', () => {
log.verbose(`Ending stream from ${id} for ${req.accountId}`)
clearInterval(heartbeat)
redisClient.quit()
})
redisClient.subscribe(id)
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment