102

PROBLEM

I've been looking for request/response timeouts for Express.js but everything seems to be related to the connection rather than the request/response itself.

If a request is taking a long time, it should be timed out. Obviously this shouldn't happen but even a simple mistake as having a route handler without a call to the callback or without res.send(), the browser will keep waiting for a reply forever.

An empty route handler is a perfect example of this.

app.get('/sessions/', function(req, res, callback){});

FIX

I added the following before app.use(app,router); and it seemed to add the timeout functionality. Does anyone have any experience/opinion on this?

app.use(function(req, res, next){
    res.setTimeout(120000, function(){
        console.log('Request has timed out.');
            res.send(408);
        });

    next();
});

Note that I've set the timeout to 2 minutes.

Xerri
  • 4,916
  • 6
  • 45
  • 54
  • 2
    I would use this for development purposes only – I can't think of a single use case where you would want to ship production code with empty routes. – srquinn Feb 11 '14 at 17:11
  • If course, my point with that is that its possible to have an issue where the request would just keep waiting. Bugs happen. I'm just trying to set a response timeout just in case. – Xerri Feb 11 '14 at 17:14
  • Gotcha – see answer below – srquinn Feb 11 '14 at 17:29
  • 9
    empty routes as a trap for web spiders? – prototype Oct 23 '15 at 17:06
  • This 'feature' actually came quite useful when I was trying to use Express.js (well, actually json-server) to build a test service with the purpose of simulating / inducing all manners of error conditions on the server side. Most such errors corresponded to HTTP status codes (e.g. Bad Request), yes, but I also wanted to cause a SocketTimeoutException on the caller end. – Ferenc Dósa-Rácz Mar 08 '18 at 13:17
  • what will happen if the value of response timeout is large then the http's timeout? – 欧阳维杰 Apr 07 '18 at 10:55

8 Answers8

86

There is already a Connect Middleware for Timeout support:

var timeout = express.timeout // express v3 and below
var timeout = require('connect-timeout'); //express v4

app.use(timeout(120000));
app.use(haltOnTimedout);

function haltOnTimedout(req, res, next){
  if (!req.timedout) next();
}

If you plan on using the Timeout middleware as a top-level middleware like above, the haltOnTimedOut middleware needs to be the last middleware defined in the stack and is used for catching the timeout event. Thanks @Aichholzer for the update.

Side Note:

Keep in mind that if you roll your own timeout middleware, 4xx status codes are for client errors and 5xx are for server errors. 408s are reserved for when:

The client did not produce a request within the time that the server was prepared to wait. The client MAY repeat the request without modifications at any later time.

Community
  • 1
  • 1
srquinn
  • 10,134
  • 2
  • 48
  • 54
  • Perfect. Even simpler. Thanks for the tip on the error message as well. – Xerri Feb 11 '14 at 17:34
  • 3
    According to the official documentation, using it as top-level middleware is not recommended, at least not just like that. https://github.com/expressjs/timeout – Stefan Sep 03 '14 at 09:12
  • This answer seems to be fairly useless without increasing the socket timeout. http://stackoverflow.com/questions/12651466/how-to-set-the-http-keep-alive-timeout-in-a-nodejs-server – Jeff Fischer Jan 20 '15 at 01:25
  • @JeffFischer: calling `socket.setTimeout()` only increases the time before the timeout event is called but the connection is not severed by default http://nodejs.org/api/net.html#net_socket_settimeout_timeout_callback. In node v0.9.12, a setTimeout API was added to the `http.server` object to handle the functionality of this middleware in the node standard http lib. However, before that API update this middleware would handle destroying the socket. You can read more on the innards of the middleware here https://github.com/expressjs/timeout/blob/master/index.js – srquinn Jan 20 '15 at 02:22
  • I have the following **app.get('/timeout', function (req, res) { for (var i = 0; i < 1111211111; i++) {} res.send('d') })** but I always get d back, why?, Note, I'm using **app.use(timeout('1s'));** – SuperUberDuper Jun 10 '16 at 13:51
  • 1
    @SuperUberDuper: Please post you query as a separate question so the community can help you with your specific issue. – srquinn Jun 10 '16 at 19:07
  • sure mate: https://stackoverflow.com/questions/37787255/express-timeout-issue-always-getting-back-response-when-forcing-blocking-operat – SuperUberDuper Jun 13 '16 at 10:23
  • `Error: Cannot find module 'connect-timeout'` – 5ervant - techintel.github.io May 22 '19 at 16:17
  • @5ervant use $ npm install connect-timeout here is a link to documentation http://expressjs.com/en/resources/middleware/timeout.html – Farasi78 Nov 30 '20 at 23:07
81

You don't need other npm modules to do this

var server = app.listen();
server.setTimeout(500000);

inspired by https://github.com/expressjs/express/issues/3330

or

app.use(function(req, res, next){
    req.setTimeout(500000, function(){
        // call back function is called when request timed out.
    });
    next();
});
bereket gebredingle
  • 12,064
  • 3
  • 36
  • 47
36

An update if one is using Express 4.2 then the timeout middleware has been removed so need to manually add it with

npm install connect-timeout

and in the code it has to be (Edited as per comment, how to include it in the code)

 var timeout = require('connect-timeout');
 app.use(timeout('100s'));
V31
  • 7,626
  • 3
  • 26
  • 44
  • 1
    Didn't DV, but some people probably need to use var timeout = require('connect-timeout') and had to search google for it =) – Mohamed Salad Mar 11 '18 at 04:00
5

In case you would like to use timeout middleware and exclude a specific route:

var timeout = require('connect-timeout');
app.use(timeout('5s')); //set 5s timeout for all requests

app.use('/my_route', function(req, res, next) {
    req.clearTimeout(); // clear request timeout
    req.setTimeout(20000); //set a 20s timeout for this request
    next();
}).get('/my_route', function(req, res) {
    //do something that takes a long time
});
cider
  • 397
  • 5
  • 11
1

If you need to test your api, this solotion can you help. I used this in middleware to test my frontend. For exmaple: if you need to test loader in frontend.

const router = require('express').Router();
const { data } = require('./data');

router.get('/api/data', (req, res, next) => {
  setTimeout(() => {
      res.set('Content-Type', 'application/json')
      res.status(200).send(data)

      next()
  }, 2000)
})

module.exports = router;
TiiGRUS
  • 85
  • 1
  • 4
-1

request.setTimeout(< time in milliseconds >) does the job

https://nodejs.org/api/http.html#http_request_settimeout_timeout_callback

rahul shukla
  • 233
  • 3
  • 11
-3

You can try:

return await new Promise((resolve) =>
  setTimeout(() => {
    resolve(resp);
  }, 3000),
);

In above code, 3000 = 3 sec. Change it according to your requirement.

I have not tried for very long scenarios though. Let me know the results in comments.

cyperpunk
  • 664
  • 7
  • 13
-20

Before you set your routes, add the code:

app.all('*', function(req, res, next) {
    setTimeout(function() {
        next();
    }, 120000); // 120 seconds
});
destino
  • 59
  • 3