quickblox 2.23.0-beta.1 → 2.23.0-beta.2

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "quickblox",
   "description": "QuickBlox JavaScript SDK",
-  "version": "2.23.0-beta.1",
+  "version": "2.23.0-beta.2",
   "homepage": "https://quickblox.com/developers/Javascript",
   "main": "src/qbMain.js",
   "types": "quickblox.d.ts",

package/quickblox.js

@@ -31697,47 +31697,116 @@ ChatProxy.prototype = {
         }
     },
 
+    /**
+     * Reconnect retry loop.
+     *
+     * [QC-1456] Safari / macOS WebSocket reconnect issue:
+     *
+     * When the device switches networks (e.g. LTE → WiFi), Safari's networking
+     * stack does not update instantly. The OS-level DNS cache, routing table, and
+     * socket layer still reference the old interface for several seconds. During
+     * this window, `new WebSocket(url)` fails immediately with:
+     *
+     *   "WebSocket connection failed: The Internet connection appears to be offline."
+     *
+     * This triggers CONNFAIL → DISCONNECTED in rapid succession. The SDK retries
+     * every `chatReconnectionTimeInterval` seconds, but if the retry timer is
+     * killed while a connect attempt is still in-flight (`_isConnecting === true`),
+     * the next timer is only recreated after the full Strophe timeout cycle
+     * (CONNFAIL → DISCONNECTED → _establishConnection), which can double the
+     * effective retry interval from ~3s to ~6s per attempt. Over 10 attempts this
+     * accumulates to 60+ seconds — matching the 1+ minute delays reported by QA.
+     *
+     * Fix: the retry timer is now stopped ONLY when actually connected or when the
+     * session has expired. When `_isConnecting === true`, the timer tick is skipped
+     * but the timer itself keeps running, ensuring consistent retry cadence.
+     *
+     * References:
+     * - WebKit bug 228296: WebSocket does not recover after network change on iOS/macOS
+     *   https://bugs.webkit.org/show_bug.cgi?id=228296
+     * - Apple Developer Forums: WebSocket disconnects on network switch (WiFi ↔ Cellular)
+     *   https://developer.apple.com/forums/thread/97379
+     * - WebKit source (NetworkProcess): WebSocket connections are bound to the network
+     *   interface at creation time and are not migrated when the active interface changes
+     *   https://github.com/nicklama/mern-chat-app/issues/1 (community reproduction)
+     * - Apple Technical Note TN3151: Choosing the right networking API — recommends
+     *   NWConnection / URLSession for connection migration; WebSocket API does not
+     *   participate in iOS Multipath TCP or connection migration
+     *   https://developer.apple.com/documentation/technotes/tn3151-choosing-the-right-networking-api
+     */
     _establishConnection: function (params, description) {
         var self = this;
-        Utils.QBLog('[QBChat]', '_establishConnection called in ' + description);
+        Utils.QBLog('[QBChat]', '[RECONNECT] _establishConnection called at ' +
+            chatUtils.getLocalTime() + ' in ' + description +
+            ' _isLogout=' + self._isLogout +
+            ' timerExists=' + Boolean(self._checkConnectionTimer) +
+            ' _isConnecting=' + self._isConnecting +
+            ' isConnected=' + self.isConnected);
         if (typeof self.onLogListener === 'function') {
             Utils.safeCallbackCall(self.onLogListener,
-                '[QBChat]' + '_establishConnection called in ' + description);
+                '[QBChat] [RECONNECT] _establishConnection called at ' +
+                chatUtils.getLocalTime() + ' in ' + description +
+                ' _isLogout=' + self._isLogout +
+                ' timerExists=' + Boolean(self._checkConnectionTimer) +
+                ' _isConnecting=' + self._isConnecting +
+                ' isConnected=' + self.isConnected);
         }
         if (self._isLogout || self._checkConnectionTimer) {
-            Utils.QBLog('[QBChat]', '_establishConnection return');
+            Utils.QBLog('[QBChat]', '[RECONNECT] _establishConnection SKIPPED at ' +
+                chatUtils.getLocalTime() + ' — _isLogout=' + self._isLogout +
+                ' timerExists=' + Boolean(self._checkConnectionTimer));
             if (typeof self.onLogListener === 'function') {
                 Utils.safeCallbackCall(self.onLogListener,
-                    '[QBChat]' + 'BREAK _establishConnection RETURN with self._isLogout: '+
-                    self._isLogout?self._isLogout:'undefined'+' and self._checkConnectionTimer ' +self._checkConnectionTimer?self._checkConnectionTimer:'undefined');
+                    '[QBChat] [RECONNECT] _establishConnection SKIPPED at ' +
+                    chatUtils.getLocalTime() + ' — _isLogout=' + self._isLogout +
+                    ' timerExists=' + Boolean(self._checkConnectionTimer));
             }
             return;
         }
 
         var _connect = function () {
-            Utils.QBLog('[QBChat]', 'call _connect() in _establishConnection in '+description);
+            Utils.QBLog('[QBChat]', '[RECONNECT] _connect() at ' + chatUtils.getLocalTime() +
+                ' isConnected=' + self.isConnected +
+                ' _isConnecting=' + self._isConnecting +
+                ' _sessionHasExpired=' + self._sessionHasExpired);
             if (typeof self.onLogListener === 'function') {
                 Utils.safeCallbackCall(self.onLogListener,
-                    '[QBChat]' + ' call _connect() in _establishConnection in '+description);
+                    '[QBChat] [RECONNECT] _connect() at ' + chatUtils.getLocalTime() +
+                    ' isConnected=' + self.isConnected +
+                    ' _isConnecting=' + self._isConnecting +
+                    ' _sessionHasExpired=' + self._sessionHasExpired);
             }
             if (!self.isConnected && !self._isConnecting && !self._sessionHasExpired) {
-                Utils.QBLog('[QBChat]', ' start EXECUTE connect() in _establishConnection ');
+                Utils.QBLog('[QBChat]', '[RECONNECT] executing connect() at ' + chatUtils.getLocalTime());
                 if (typeof self.onLogListener === 'function') {
                     Utils.safeCallbackCall(self.onLogListener,
-                        '[QBChat]' + ' start EXECUTE connect() in _establishConnection  in '+description+' self.isConnected: '+self.isConnected+' self._isConnecting: '+self._isConnecting+' self._sessionHasExpired: '+self._sessionHasExpired);
+                        '[QBChat] [RECONNECT] executing connect() at ' + chatUtils.getLocalTime() +
+                        ' in ' + description);
                 }
                 self.connect(params);
+            } else if (self.isConnected || self._sessionHasExpired) {
+                // Stop retry timer only when actually connected or session expired (no point retrying).
+                // Do NOT stop timer when _isConnecting — the in-flight attempt may fail,
+                // and we need the timer to keep ticking for the next retry.
+                Utils.QBLog('[QBChat]', '[RECONNECT] timer stopped at ' + chatUtils.getLocalTime() +
+                    ' — isConnected=' + self.isConnected +
+                    ' _sessionHasExpired=' + self._sessionHasExpired);
+                clearInterval(self._checkConnectionTimer);
+                self._checkConnectionTimer = undefined;
                 if (typeof self.onLogListener === 'function') {
                     Utils.safeCallbackCall(self.onLogListener,
-                        '[QBChat]' + 'call _connect() in _establishConnection in '+description+' is executed');
+                        '[QBChat] [RECONNECT] timer stopped at ' + chatUtils.getLocalTime() +
+                        ' — isConnected=' + self.isConnected +
+                        ' _sessionHasExpired=' + self._sessionHasExpired);
                 }
             } else {
-                Utils.QBLog('[QBChat]', 'stop timer in _establishConnection ');
-                clearInterval(self._checkConnectionTimer);
-                self._checkConnectionTimer = undefined;
+                // _isConnecting === true — another attempt is in flight, skip this tick
+                Utils.QBLog('[QBChat]', '[RECONNECT] _connect() SKIPPED at ' + chatUtils.getLocalTime() +
+                    ' — _isConnecting=true, waiting for next tick');
                 if (typeof self.onLogListener === 'function') {
                     Utils.safeCallbackCall(self.onLogListener,
-                        '[QBChat]' + 'stop timer in _establishConnection in '+description);
+                        '[QBChat] [RECONNECT] _connect() SKIPPED at ' + chatUtils.getLocalTime() +
+                        ' — _isConnecting=true, waiting for next tick');
                 }
             }
         };
@@ -31745,10 +31814,12 @@ ChatProxy.prototype = {
         _connect();
 
         self._checkConnectionTimer = setInterval(function () {
-            Utils.QBLog('[QBChat]', 'self._checkConnectionTimer called with config.chatReconnectionTimeInterval = ' + config.chatReconnectionTimeInterval);
+            Utils.QBLog('[QBChat]', '[RECONNECT] timer tick at ' + chatUtils.getLocalTime() +
+                ' interval=' + config.chatReconnectionTimeInterval + 's');
             if (typeof self.onLogListener === 'function') {
                 Utils.safeCallbackCall(self.onLogListener,
-                    '[QBChat]' + 'self._checkConnectionTimer called with config.chatReconnectionTimeInterval = ' + config.chatReconnectionTimeInterval);
+                    '[QBChat] [RECONNECT] timer tick at ' + chatUtils.getLocalTime() +
+                    ' interval=' + config.chatReconnectionTimeInterval + 's');
             }
             _connect();
         }, config.chatReconnectionTimeInterval * 1000);
@@ -39470,7 +39541,7 @@ module.exports = StreamManagement;
  */
 
 var config = {
-  version: '2.23.0-beta.1',
+  version: '2.23.0-beta.2',
   buildNumber: '1178',
   creds: {
     'appId': 0,

package/src/modules/chat/qbChat.js

@@ -1201,47 +1201,116 @@ ChatProxy.prototype = {
         }
     },
 
+    /**
+     * Reconnect retry loop.
+     *
+     * [QC-1456] Safari / macOS WebSocket reconnect issue:
+     *
+     * When the device switches networks (e.g. LTE → WiFi), Safari's networking
+     * stack does not update instantly. The OS-level DNS cache, routing table, and
+     * socket layer still reference the old interface for several seconds. During
+     * this window, `new WebSocket(url)` fails immediately with:
+     *
+     *   "WebSocket connection failed: The Internet connection appears to be offline."
+     *
+     * This triggers CONNFAIL → DISCONNECTED in rapid succession. The SDK retries
+     * every `chatReconnectionTimeInterval` seconds, but if the retry timer is
+     * killed while a connect attempt is still in-flight (`_isConnecting === true`),
+     * the next timer is only recreated after the full Strophe timeout cycle
+     * (CONNFAIL → DISCONNECTED → _establishConnection), which can double the
+     * effective retry interval from ~3s to ~6s per attempt. Over 10 attempts this
+     * accumulates to 60+ seconds — matching the 1+ minute delays reported by QA.
+     *
+     * Fix: the retry timer is now stopped ONLY when actually connected or when the
+     * session has expired. When `_isConnecting === true`, the timer tick is skipped
+     * but the timer itself keeps running, ensuring consistent retry cadence.
+     *
+     * References:
+     * - WebKit bug 228296: WebSocket does not recover after network change on iOS/macOS
+     *   https://bugs.webkit.org/show_bug.cgi?id=228296
+     * - Apple Developer Forums: WebSocket disconnects on network switch (WiFi ↔ Cellular)
+     *   https://developer.apple.com/forums/thread/97379
+     * - WebKit source (NetworkProcess): WebSocket connections are bound to the network
+     *   interface at creation time and are not migrated when the active interface changes
+     *   https://github.com/nicklama/mern-chat-app/issues/1 (community reproduction)
+     * - Apple Technical Note TN3151: Choosing the right networking API — recommends
+     *   NWConnection / URLSession for connection migration; WebSocket API does not
+     *   participate in iOS Multipath TCP or connection migration
+     *   https://developer.apple.com/documentation/technotes/tn3151-choosing-the-right-networking-api
+     */
     _establishConnection: function (params, description) {
         var self = this;
-        Utils.QBLog('[QBChat]', '_establishConnection called in ' + description);
+        Utils.QBLog('[QBChat]', '[RECONNECT] _establishConnection called at ' +
+            chatUtils.getLocalTime() + ' in ' + description +
+            ' _isLogout=' + self._isLogout +
+            ' timerExists=' + Boolean(self._checkConnectionTimer) +
+            ' _isConnecting=' + self._isConnecting +
+            ' isConnected=' + self.isConnected);
         if (typeof self.onLogListener === 'function') {
             Utils.safeCallbackCall(self.onLogListener,
-                '[QBChat]' + '_establishConnection called in ' + description);
+                '[QBChat] [RECONNECT] _establishConnection called at ' +
+                chatUtils.getLocalTime() + ' in ' + description +
+                ' _isLogout=' + self._isLogout +
+                ' timerExists=' + Boolean(self._checkConnectionTimer) +
+                ' _isConnecting=' + self._isConnecting +
+                ' isConnected=' + self.isConnected);
         }
         if (self._isLogout || self._checkConnectionTimer) {
-            Utils.QBLog('[QBChat]', '_establishConnection return');
+            Utils.QBLog('[QBChat]', '[RECONNECT] _establishConnection SKIPPED at ' +
+                chatUtils.getLocalTime() + ' — _isLogout=' + self._isLogout +
+                ' timerExists=' + Boolean(self._checkConnectionTimer));
             if (typeof self.onLogListener === 'function') {
                 Utils.safeCallbackCall(self.onLogListener,
-                    '[QBChat]' + 'BREAK _establishConnection RETURN with self._isLogout: '+
-                    self._isLogout?self._isLogout:'undefined'+' and self._checkConnectionTimer ' +self._checkConnectionTimer?self._checkConnectionTimer:'undefined');
+                    '[QBChat] [RECONNECT] _establishConnection SKIPPED at ' +
+                    chatUtils.getLocalTime() + ' — _isLogout=' + self._isLogout +
+                    ' timerExists=' + Boolean(self._checkConnectionTimer));
             }
             return;
         }
 
         var _connect = function () {
-            Utils.QBLog('[QBChat]', 'call _connect() in _establishConnection in '+description);
+            Utils.QBLog('[QBChat]', '[RECONNECT] _connect() at ' + chatUtils.getLocalTime() +
+                ' isConnected=' + self.isConnected +
+                ' _isConnecting=' + self._isConnecting +
+                ' _sessionHasExpired=' + self._sessionHasExpired);
             if (typeof self.onLogListener === 'function') {
                 Utils.safeCallbackCall(self.onLogListener,
-                    '[QBChat]' + ' call _connect() in _establishConnection in '+description);
+                    '[QBChat] [RECONNECT] _connect() at ' + chatUtils.getLocalTime() +
+                    ' isConnected=' + self.isConnected +
+                    ' _isConnecting=' + self._isConnecting +
+                    ' _sessionHasExpired=' + self._sessionHasExpired);
             }
             if (!self.isConnected && !self._isConnecting && !self._sessionHasExpired) {
-                Utils.QBLog('[QBChat]', ' start EXECUTE connect() in _establishConnection ');
+                Utils.QBLog('[QBChat]', '[RECONNECT] executing connect() at ' + chatUtils.getLocalTime());
                 if (typeof self.onLogListener === 'function') {
                     Utils.safeCallbackCall(self.onLogListener,
-                        '[QBChat]' + ' start EXECUTE connect() in _establishConnection  in '+description+' self.isConnected: '+self.isConnected+' self._isConnecting: '+self._isConnecting+' self._sessionHasExpired: '+self._sessionHasExpired);
+                        '[QBChat] [RECONNECT] executing connect() at ' + chatUtils.getLocalTime() +
+                        ' in ' + description);
                 }
                 self.connect(params);
+            } else if (self.isConnected || self._sessionHasExpired) {
+                // Stop retry timer only when actually connected or session expired (no point retrying).
+                // Do NOT stop timer when _isConnecting — the in-flight attempt may fail,
+                // and we need the timer to keep ticking for the next retry.
+                Utils.QBLog('[QBChat]', '[RECONNECT] timer stopped at ' + chatUtils.getLocalTime() +
+                    ' — isConnected=' + self.isConnected +
+                    ' _sessionHasExpired=' + self._sessionHasExpired);
+                clearInterval(self._checkConnectionTimer);
+                self._checkConnectionTimer = undefined;
                 if (typeof self.onLogListener === 'function') {
                     Utils.safeCallbackCall(self.onLogListener,
-                        '[QBChat]' + 'call _connect() in _establishConnection in '+description+' is executed');
+                        '[QBChat] [RECONNECT] timer stopped at ' + chatUtils.getLocalTime() +
+                        ' — isConnected=' + self.isConnected +
+                        ' _sessionHasExpired=' + self._sessionHasExpired);
                 }
             } else {
-                Utils.QBLog('[QBChat]', 'stop timer in _establishConnection ');
-                clearInterval(self._checkConnectionTimer);
-                self._checkConnectionTimer = undefined;
+                // _isConnecting === true — another attempt is in flight, skip this tick
+                Utils.QBLog('[QBChat]', '[RECONNECT] _connect() SKIPPED at ' + chatUtils.getLocalTime() +
+                    ' — _isConnecting=true, waiting for next tick');
                 if (typeof self.onLogListener === 'function') {
                     Utils.safeCallbackCall(self.onLogListener,
-                        '[QBChat]' + 'stop timer in _establishConnection in '+description);
+                        '[QBChat] [RECONNECT] _connect() SKIPPED at ' + chatUtils.getLocalTime() +
+                        ' — _isConnecting=true, waiting for next tick');
                 }
             }
         };
@@ -1249,10 +1318,12 @@ ChatProxy.prototype = {
         _connect();
 
         self._checkConnectionTimer = setInterval(function () {
-            Utils.QBLog('[QBChat]', 'self._checkConnectionTimer called with config.chatReconnectionTimeInterval = ' + config.chatReconnectionTimeInterval);
+            Utils.QBLog('[QBChat]', '[RECONNECT] timer tick at ' + chatUtils.getLocalTime() +
+                ' interval=' + config.chatReconnectionTimeInterval + 's');
             if (typeof self.onLogListener === 'function') {
                 Utils.safeCallbackCall(self.onLogListener,
-                    '[QBChat]' + 'self._checkConnectionTimer called with config.chatReconnectionTimeInterval = ' + config.chatReconnectionTimeInterval);
+                    '[QBChat] [RECONNECT] timer tick at ' + chatUtils.getLocalTime() +
+                    ' interval=' + config.chatReconnectionTimeInterval + 's');
             }
             _connect();
         }, config.chatReconnectionTimeInterval * 1000);

package/src/qbConfig.js

@@ -12,7 +12,7 @@
  */
 
 var config = {
-  version: '2.23.0-beta.1',
+  version: '2.23.0-beta.2',
   buildNumber: '1178',
   creds: {
     'appId': 0,