Honor a numeric timeout of 0 instead of using the 5000ms default

HISTORY.md

@@ -1,3 +1,9 @@
+unreleased
+==========
+
+  * Fix: a numeric timeout of `0` was treated as the 5000ms default
+    because `time || 5000` is falsy for zero
+
 1.9.1 / 2025-07-17
 ==========
 

index.js

@@ -39,7 +39,7 @@ function timeout (time, options) {
 
   var delay = typeof time === 'string'
     ? ms(time)
-    : Number(time || 5000)
+    : Number(time == null ? 5000 : time)
 
   var respond = opts.respond === undefined || opts.respond === true
 

test/test.js

@@ -21,6 +21,13 @@ describe('timeout()', function () {
     .expect(503, /123ms/, done)
   })
 
+  it('should accept a zero millisecond timeout', function (done) {
+    var server = createServer(0)
+    request(server)
+    .get('/')
+    .expect(503, /after 0ms/, done)
+  })
+
   it('should accept string timeout', function (done) {
     var server = createServer('45ms')
     request(server)