From 8c223f8543db479310ab534f5d07c4083c92263e Mon Sep 17 00:00:00 2001 From: Damien Arrachequesne Date: Thu, 28 Dec 2017 23:50:38 +0100 Subject: [PATCH] [chore] Update default values for pingTimeout `pingTimeout` now defaults to 5 seconds instead of 60 seconds. --- README.md | 2 +- lib/server.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index d7fd075ea..989e348d0 100644 --- a/README.md +++ b/README.md @@ -196,7 +196,7 @@ to a single process. - `Object`: optional, options object - **Options** - `pingTimeout` (`Number`): how many ms without a pong packet to - consider the connection closed (`60000`) + consider the connection closed (`5000`) - `pingInterval` (`Number`): how many ms before sending a new ping packet (`25000`) - `upgradeTimeout` (`Number`): how many ms before an uncompleted transport upgrade is cancelled (`10000`) diff --git a/lib/server.js b/lib/server.js index 4d6ec0272..8cd0e994d 100644 --- a/lib/server.js +++ b/lib/server.js @@ -37,7 +37,7 @@ function Server (opts) { opts = opts || {}; this.wsEngine = opts.wsEngine || process.env.EIO_WS_ENGINE || 'ws'; - this.pingTimeout = opts.pingTimeout || 60000; + this.pingTimeout = opts.pingTimeout || 5000; this.pingInterval = opts.pingInterval || 25000; this.upgradeTimeout = opts.upgradeTimeout || 10000; this.maxHttpBufferSize = opts.maxHttpBufferSize || 10E7;