Browse Source

Merge pull request #95 from jitsi/callstats_confID_case

Fixes issue with callstats confID
master
lyubomir 9 years ago
parent
commit
2a51c6edbc
2 changed files with 6 additions and 4 deletions
  1. 4
    2
      JitsiConference.js
  2. 2
    2
      modules/statistics/CallStats.js

+ 4
- 2
JitsiConference.js View File

@@ -45,7 +45,9 @@ function JitsiConference(options) {
45 45
     this.statistics = new Statistics(this.xmpp, {
46 46
         callStatsID: this.options.config.callStatsID,
47 47
         callStatsSecret: this.options.config.callStatsSecret,
48
-        disableThirdPartyRequests: this.options.config.disableThirdPartyRequests
48
+        disableThirdPartyRequests:
49
+            this.options.config.disableThirdPartyRequests,
50
+        roomName: this.options.name
49 51
     });
50 52
     setupListeners(this);
51 53
     var JitsiMeetJS = this.connection.JitsiMeetJS;
@@ -315,7 +317,7 @@ JitsiConference.prototype.addTrack = function (track) {
315 317
     {
316 318
         throw new Error(JitsiTrackErrors.TRACK_IS_DISPOSED);
317 319
     }
318
-    
320
+
319 321
     if (track.isVideoTrack() && this.rtc.getLocalVideoTrack()) {
320 322
         throw new Error("cannot add second video track to the conference");
321 323
     }

+ 2
- 2
modules/statistics/CallStats.js View File

@@ -106,9 +106,9 @@ var CallStats = _try_catch(function(jingleSession, Settings, options) {
106 106
 
107 107
         this.userID = Settings.getCallStatsUserName();
108 108
 
109
-    //FIXME:  change it to something else (maybe roomName)
110 109
         var location = window.location;
111
-        this.confID = location.hostname + location.pathname;
110
+        // The confID is case sensitive!!!
111
+        this.confID = location.hostname + "/" + options.roomName;
112 112
 
113 113
         //userID is generated or given by the origin server
114 114
         callStats.initialize(options.callStatsID,

Loading…
Cancel
Save