Browse Source

Use rtcstats with keep-alive / add rtcstats enabled config

j8
Andrei Gavrilescu 4 years ago
parent
commit
d725c0ab8a
4 changed files with 11 additions and 5 deletions
  1. 6
    0
      config.js
  2. 2
    2
      package-lock.json
  3. 1
    1
      package.json
  4. 2
    2
      react/features/rtcstats/middleware.js

+ 6
- 0
config.js View File

459
         // amplitudeAPPKey: '<APP_KEY>'
459
         // amplitudeAPPKey: '<APP_KEY>'
460
 
460
 
461
         // Configuration for the rtcstats server:
461
         // Configuration for the rtcstats server:
462
++       // By enabling rtcstats server every time a conference is joined the rtcstats
463
+        // module connects to the provided rtcstatsEndpoint and sends statistics regarding
464
+        // PeerConnection states along with getStats metrics polled at the specified
465
+        // interval.
466
+        // rtcstatsEnabled: true,
467
+
462
         // In order to enable rtcstats one needs to provide a endpoint url.
468
         // In order to enable rtcstats one needs to provide a endpoint url.
463
         // rtcstatsEndpoint: wss://rtcstats-server-pilot.jitsi.net/,
469
         // rtcstatsEndpoint: wss://rtcstats-server-pilot.jitsi.net/,
464
 
470
 

+ 2
- 2
package-lock.json View File

22300
       }
22300
       }
22301
     },
22301
     },
22302
     "rtcstats": {
22302
     "rtcstats": {
22303
-      "version": "github:jitsi/rtcstats#02a1a089d9a97d1414d216ff7d9c432253e50190",
22304
-      "from": "github:jitsi/rtcstats#v6.1.3",
22303
+      "version": "github:jitsi/rtcstats#0597c6a928f321d3cdec947d877012b7e8c2f1dc",
22304
+      "from": "github:jitsi/rtcstats#v6.2.0",
22305
       "requires": {
22305
       "requires": {
22306
         "@jitsi/js-utils": "1.0.0"
22306
         "@jitsi/js-utils": "1.0.0"
22307
       },
22307
       },

+ 1
- 1
package.json View File

90
     "redux": "4.0.4",
90
     "redux": "4.0.4",
91
     "redux-thunk": "2.2.0",
91
     "redux-thunk": "2.2.0",
92
     "rnnoise-wasm": "github:jitsi/rnnoise-wasm.git#566a16885897704d6e6d67a1d5ac5d39781db2af",
92
     "rnnoise-wasm": "github:jitsi/rnnoise-wasm.git#566a16885897704d6e6d67a1d5ac5d39781db2af",
93
-    "rtcstats": "github:jitsi/rtcstats#v6.1.3",
93
+    "rtcstats": "github:jitsi/rtcstats#v6.2.0",
94
     "styled-components": "3.4.9",
94
     "styled-components": "3.4.9",
95
     "util": "0.12.1",
95
     "util": "0.12.1",
96
     "uuid": "3.1.0",
96
     "uuid": "3.1.0",

+ 2
- 2
react/features/rtcstats/middleware.js View File

25
 
25
 
26
     switch (action.type) {
26
     switch (action.type) {
27
     case LIB_WILL_INIT: {
27
     case LIB_WILL_INIT: {
28
-        if (analytics.rtcstatsEndpoint) {
28
+        if (analytics.rtcstatsEnabled) {
29
             // RTCStats "proxies" WebRTC functions such as GUM and RTCPeerConnection by rewriting the global
29
             // RTCStats "proxies" WebRTC functions such as GUM and RTCPeerConnection by rewriting the global
30
             // window functions. Because lib-jitsi-meet uses references to those functions that are taken on
30
             // window functions. Because lib-jitsi-meet uses references to those functions that are taken on
31
             // init, we need to add these proxies before it initializes, otherwise lib-jitsi-meet will use the
31
             // init, we need to add these proxies before it initializes, otherwise lib-jitsi-meet will use the
47
         break;
47
         break;
48
     }
48
     }
49
     case CONFERENCE_JOINED: {
49
     case CONFERENCE_JOINED: {
50
-        if (analytics.rtcstatsEndpoint && RTCStats.isInitialized()) {
50
+        if (analytics.rtcstatsEnabled && RTCStats.isInitialized()) {
51
             // Once the conference started connect to the rtcstats server and send data.
51
             // Once the conference started connect to the rtcstats server and send data.
52
             try {
52
             try {
53
                 RTCStats.connect();
53
                 RTCStats.connect();

Loading…
Cancel
Save