Formats callstats application logs as JSON.

JitsiConference.js

@@ -1212,7 +1212,11 @@ JitsiConference.prototype._reportAudioProblem = function (ssrc) {
 };
 
 /**
- * Logs an "application log" message
+ * Logs an "application log" message.
+ * @param message {string} The message to log. Note that while this can be a
+ * generic string, the convention used by lib-jitsi-meet and jitsi-meet is to
+ * log valid JSON strings, with an "id" field used for distinguishing between
+ * message types. E.g.: {id: "recorder_status", status: "off"}
  */
 JitsiConference.prototype.sendApplicationLog = function(message) {
     Statistics.sendLog(message);

JitsiConferenceEventManager.js

@@ -215,8 +215,14 @@ JitsiConferenceEventManager.prototype.setupChatRoomListeners = function () {
         if (conference.statistics && conference.isModerator()) {
             conference.on(JitsiConferenceEvents.RECORDER_STATE_CHANGED,
                 function (status, error) {
-                    Statistics.sendLog("[Recorder] status: " + status
-                        + (error? " error: " + error : ""));
+                    var logObject = {
+                        id: "recorder_status",
+                        status: status
+                    };
+                    if (error) {
+                        logObject.error = error;
+                    }
+                    Statistics.sendLog(JSON.stringify(logObject));
                 });
         }
     });
@@ -254,11 +260,13 @@ JitsiConferenceEventManager.prototype.setupChatRoomListeners = function () {
 
     conference.room.addListener(XMPPEvents.LOCAL_UFRAG_CHANGED,
         function (ufrag) {
-            Statistics.sendLog("Local ufrag: " + ufrag);
+            Statistics.sendLog(
+                JSON.stringify({id: "local_ufrag", value: ufrag}));
         });
     conference.room.addListener(XMPPEvents.REMOTE_UFRAG_CHANGED,
         function (ufrag) {
-            Statistics.sendLog("Remote ufrag: " + ufrag);
+            Statistics.sendLog(
+                JSON.stringify({id: "remote_ufrag", value: ufrag}));
         });
 
     chatRoom.addPresenceListener("startmuted", function (data, from) {

JitsiMeetJS.js

@@ -67,6 +67,7 @@ var LibJitsiMeet = {
     logLevels: Logger.levels,
     mediaDevices: JitsiMediaDevices,
     init: function (options) {
+        var logObject, attr;
         Statistics.audioLevelsEnabled = !options.disableAudioLevels;
 
         if(typeof options.audioLevelsInterval === 'number') {
@@ -78,16 +79,28 @@ var LibJitsiMeet = {
                 this.getGlobalOnErrorHandler.bind(this));
         }
 
-        // Lets send some general stats useful for debugging problems
+        // Log deployment-specific information, if available.
         if (window.jitsiRegionInfo
             && Object.keys(window.jitsiRegionInfo).length > 0) {
-            // remove quotes to make it prettier
-            Statistics.sendLog(
-                JSON.stringify(window.jitsiRegionInfo).replace(/\"/g, ""));
+            logObject = {};
+            for (attr in window.jitsiRegionInfo) {
+                if (window.jitsiRegionInfo.hasOwnProperty(attr)) {
+                    logObject[attr] = window.jitsiRegionInfo[attr];
+                }
+            }
+
+            logObject.id = "deployment_info";
+            Statistics.sendLog(JSON.stringify(logObject));
         }
 
-        if(this.version)
-            Statistics.sendLog("LibJitsiMeet:" + this.version);
+        if(this.version) {
+            logObject = {
+                id: "component_version",
+                component: "lib-jitsi-meet",
+                version: this.version
+            }
+            Statistics.sendLog(JSON.stringify(logObject));
+        }
 
         return RTC.init(options || {});
     },
@@ -180,7 +193,11 @@ var LibJitsiMeet = {
                     // User cancelled action is not really an error, so only
                     // log it as an event to avoid having conference classified
                     // as partially failed
-                    Statistics.sendLog(error.message);
+                    var logObject = {
+                        id: "chrome_extension_user_canceled",
+                        message: error.message
+                    };
+                    Statistics.sendLog(JSON.stringify(logObject));
                 } else {
                     // Report gUM failed to the stats
                     Statistics.sendGetUserMediaFailed(error);

modules/version/ComponentsVersions.js

@@ -48,7 +48,7 @@ ComponentsVersions.prototype.processPresence =
         return;
     }
 
-    var log = "";
+    var log = [];
     node.children.forEach(function(item){
 
         var componentName = item.attributes.name;
@@ -66,14 +66,16 @@ ComponentsVersions.prototype.processPresence =
             this.versions[componentName] = version;
             logger.info("Got " + componentName + " version: " + version);
 
-            log += (log.length > 0? ", " : "")
-                + componentName + ": " + version;
+            log.push({
+                id: "component_version",
+                component: componentName,
+                version: version});
         }
     }.bind(this));
 
     // logs versions to stats
     if (log.length > 0)
-        Statistics.sendLog(log);
+        Statistics.sendLog(JSON.stringify(log));
 };
 
 /**