Merge pull request #581 from jitsi/rn-callstats

[RN] Fixes problem with CallStats on mobile

modules/statistics/CallStats.js

@@ -288,7 +288,12 @@ export default class CallStats {
                 // to not log the whole log batch
                 // FIXME check the current logging level (currently not exposed
                 // by the logger implementation)
-                console && console.debug('reportError', pc, cs, type);
+                // NOTE it is not safe to log whole objects on react-native as
+                // those contain too many circular references and may crash
+                // the app.
+                if (!RTCBrowserType.isReactNative()) {
+                    console && console.debug('reportError', pc, cs, type);
+                }
             } else {
                 logger.debug('reportError', pc, cs, type, ...args);
             }
@@ -506,7 +511,7 @@ export default class CallStats {
      * @return {boolean} true if the call was successful or false otherwise.
      */
     _addNewFabric() {
-        logger.info('addNewFabric', this.remoteUserID, this);
+        logger.info('addNewFabric', this.remoteUserID);
         try {
             const ret
                 = CallStats.backend.addNewFabric(

package.json

@@ -19,7 +19,7 @@
     "async": "0.9.0",
     "current-executing-script": "0.1.3",
     "jitsi-meet-logger": "jitsi/jitsi-meet-logger",
-    "react-native-callstats": "3.19.8",
+    "react-native-callstats": "3.23.0",
     "retry": "0.6.1",
     "sdp-interop": "0.1.12",
     "sdp-simulcast": "0.2.1",