From aac904b65e043935a5e845c37c1b300c385dc177 Mon Sep 17 00:00:00 2001 From: Fabian Jakobs Date: Thu, 28 May 2015 12:35:12 +0000 Subject: [PATCH 1/2] add timeout handler to rest client --- node_modules/c9/rest_client.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/node_modules/c9/rest_client.js b/node_modules/c9/rest_client.js index d8910d20..5919e33b 100644 --- a/node_modules/c9/rest_client.js +++ b/node_modules/c9/rest_client.js @@ -111,6 +111,10 @@ function RestClient(host, port, config) { done(e); }); + res.on("timeout", function() { + done(new Error("Request timed out")); + }); + var called = false; function done(err, json) { if (called) From 60360f8598ea09dddaba9d9a7068e3d6016aac89 Mon Sep 17 00:00:00 2001 From: Fabian Jakobs Date: Fri, 29 May 2015 08:50:49 +0000 Subject: [PATCH 2/2] document req timeout --- node_modules/c9/request_timeout.js | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/node_modules/c9/request_timeout.js b/node_modules/c9/request_timeout.js index 76f24360..990f8ae4 100644 --- a/node_modules/c9/request_timeout.js +++ b/node_modules/c9/request_timeout.js @@ -1,3 +1,8 @@ +/** + * the default response timeout in node.js is 2min. If a request takes longer + * to process then it needs to be increased + */ + module.exports = function(timeout) { return function(req, res, next) { req.setTimeout(timeout);