Merge pull request #1 from antonselukh/sockjs#403

sockjs#403 Provide SockJS config for minimum transport fallback timeout
antonselukh · Nov 7, 2017 · 08f4a09 · 08f4a09
2 parents 4d18fd5 + 4c11b76
commit 08f4a09
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/main.js b/lib/main.js
@@ -49,6 +49,7 @@ function SockJS(url, protocols, options) {
   }
   this._transportsWhitelist = options.transports;
   this._transportOptions = options.transportOptions || {};
+  this._transportMinTimeout = options.transportMinTimeout || null;
 
   var sessionId = options.sessionId || 8;
   if (typeof sessionId === 'function') {
@@ -208,8 +209,10 @@ SockJS.prototype._connect = function() {
       }
     }
 
-    // calculate timeout based on RTO and round trips. Default to 5s
-    var timeoutMs = (this._rto * Transport.roundTrips) || 5000;
+    // calculate timeout based on RTO and round trips. Default to 5s. Min timeout is null, can be overridden
+    var timeoutMs = (
+        (this._rto > this._transportMinTimeout ? this._rto : this._transportMinTimeout) * Transport.roundTrips
+    ) || 5000;
     this._transportTimeoutId = setTimeout(this._transportTimeout.bind(this), timeoutMs);
     debug('using timeout', timeoutMs);