diff --git a/src/addons.js b/src/addons.js index 64c980f00..9ce911eb2 100644 --- a/src/addons.js +++ b/src/addons.js @@ -326,7 +326,7 @@ function containerStatus(containerName, tokenEnvName, callback) { if (error && (error.reason === BoxError.NOT_FOUND || error.reason === BoxError.INACTIVE)) return callback(null, { status: exports.SERVICE_STATUS_STOPPED }); if (error) return callback(error); - request.get(`https://${addonDetails.ip}:3000/healthcheck?access_token=${addonDetails.token}`, { json: true, rejectUnauthorized: false }, function (error, response) { + request.get(`https://${addonDetails.ip}:3000/healthcheck?access_token=${addonDetails.token}`, { json: true, rejectUnauthorized: false, timeout: 3000 }, function (error, response) { if (error) return callback(null, { status: exports.SERVICE_STATUS_STARTING, error: `Error waiting for ${containerName}: ${error.message}` }); if (response.statusCode !== 200 || !response.body.status) return callback(null, { status: exports.SERVICE_STATUS_STARTING, error: `Error waiting for ${containerName}. Status code: ${response.statusCode} message: ${response.body.message}` }); @@ -623,7 +623,7 @@ function waitForContainer(containerName, tokenEnvName, callback) { if (error) return callback(error); async.retry({ times: 10, interval: 15000 }, function (retryCallback) { - request.get(`https://${result.ip}:3000/healthcheck?access_token=${result.token}`, { json: true, rejectUnauthorized: false }, function (error, response) { + request.get(`https://${result.ip}:3000/healthcheck?access_token=${result.token}`, { json: true, rejectUnauthorized: false, timeout: 3000 }, function (error, response) { if (error) return retryCallback(new BoxError(BoxError.ADDONS_ERROR, `Network error waiting for ${containerName}: ${error.message}`)); if (response.statusCode !== 200 || !response.body.status) return retryCallback(new BoxError(BoxError.ADDONS_ERROR, `Error waiting for ${containerName}. Status code: ${response.statusCode} message: ${response.body.message}`)); diff --git a/src/routes/mailserver.js b/src/routes/mailserver.js index 29fb5e444..5e13703ce 100644 --- a/src/routes/mailserver.js +++ b/src/routes/mailserver.js @@ -43,6 +43,7 @@ function proxy(req, res, next) { proxyOptions.rejectUnauthorized = false; const mailserverProxy = middleware.proxy(proxyOptions); + req.clearTimeout(); // TODO: add timeout to mail server proxy logic instead of this mailserverProxy(req, res, function (error) { if (!error) return next();