expressjs / timeout

Request timeout middleware for Connect/Express

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Inconsistent test case documentation.

dsummersl opened this issue · comments

The test case 'should respond with 408 Request timeout' suggests that a 408 should be returned on timeout, while the library returns a 503. Personally I'm not quite clear as to the difference between 408/503 - I expect the correct response code depends on the context of the timeout.

Thanks, it is just a typo in the test description; 5xx is server errors (like what this module causes) and 4xx are client errors.