@reactoo/watchtogether-sdk-js 2.7.34 → 2.7.35

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reactoo/watchtogether-sdk-js",
-  "version": "2.7.34",
+  "version": "2.7.35",
   "description": "Javascript SDK for Reactoo",
   "main": "src/index.js",
   "unpkg": "dist/watchtogether-sdk.min.js",

package/src/models/iot.js

@@ -31,11 +31,9 @@ let iot = function () {
 
         onConnect: () => {
             this.__privates.iot.log('MQTT client connected');
-            this.iot.setupVisibilityChangeListener();
         },
         onClosed: () => {
             this.__privates.iot.log('MQTT client closed');
-            this.iot.disableVisibilityChangeListener();
         },
         onError: () => {
             this.__privates.iot.log('MQTT client error');
@@ -80,6 +78,7 @@ let iot = function () {
             let __currentTopicsCopy = new Set(__currentTopics); // Keep this line
             __currentTopics.clear();
 
+            this.iot.setupVisibilityChangeListener();
             this.iot.$on('connect', this.iot.onConnect, this);
             this.iot.$on('closed', this.iot.onClosed, this);
             this.iot.$on('error', this.iot.onError, this);
@@ -116,6 +115,7 @@ let iot = function () {
             return this.__privates.iot.disconnect()
                 .then(() => {
 
+                    this.iot.disableVisibilityChangeListener();
                     this.iot.$off('connect', this.iot.onConnect, this);
                     this.iot.$off('closed', this.iot.onClosed, this);
                     this.iot.$off('error', this.iot.onError, this);

package/src/modules/wt-iot.js

@@ -6,7 +6,6 @@ class Iot {
     constructor(enableDebugFlag) {
         Object.assign(this, emitter());
         this.decoder = new TextDecoder('utf-8');
-        this.connectionActive = false;
         this.log = Iot.noop;
         // Remove: this.debugFlag = enableDebugFlag;
         this.credentialsExpirationCheckIntervalId = null;
@@ -50,32 +49,26 @@ class Iot {
                 this.connection = client.new_connection(config);
 
                 this.connection.on('connect', () => {
-                    this.connectionActive = true;
                     this.emit('connect');
                 });
 
                 this.connection.on('closed', (error) => {
-                    this.connectionActive = false;
                     this.emit('closed', error);
                 });
 
                 this.connection.on('disconnect', () => {
-                    this.connectionActive = false;
                     this.emit('disconnect');
                 });
 
                 this.connection.on('error', (error) => {
-                    this.connectionActive = false;
                     this.emit('error', error);
                 });
 
                 this.connection.on('interrupt', (error) => {
-                    this.connectionActive = false;
                     this.emit('interrupt', error);
                 });
 
                 this.connection.on('resume', (error) => {
-                    this.connectionActive = true;
                     this.emit('resume', error);
                 });
 
@@ -84,12 +77,10 @@ class Iot {
                 });
 
                 this.connection.on('connection_success', (error) => {
-                    this.connectionActive = true;
                     this.emit('connection_success', error);
                 });
 
                 this.connection.on('connection_failure', (error) => {
-                    this.connectionActive = false;
                     this.emit('connection_failure', error);
                 });
 
@@ -106,13 +97,12 @@ class Iot {
         this.log('iot disconnect');
         this.stopCredentialsExpirationCheck();
         if (this.connection) {
-            this.connectionActive = false;
             return this.connection.disconnect();
         } else return Promise.resolve();
     }
 
     isConnected() {
-        return this.connectionActive;
+        return this.connection && this.connection.currentState === 0 && this.connection.desiredState === 0;
     }
 
     subscribe(topic) {
@@ -245,7 +235,8 @@ class Iot {
         this.currentCredentialsExpirationStamp = new Date(expiration).getTime();
         this.credentialsExpirationCheckIntervalId = setInterval(() => {
             const currentTimeStamp = new Date().getTime();
-            if(this.currentCredentialsExpirationStamp - currentTimeStamp <= 300000) {
+            // update 15 minutes before expiration
+            if(this.currentCredentialsExpirationStamp - currentTimeStamp <= 900000) {
                 this.emit('updateCredentials');
             }
         }, 5000);
@@ -279,7 +270,7 @@ class Iot {
 
     checkConnection() {
         return new Promise((resolve, reject) => {
-            if (!this.connection || !this.connectionActive) {
+            if (!this.connection || !(this.connection.currentState === 0 && this.connection.desiredState === 0)) {
                 reject(new Error('Not connected'));
                 return;
             }