Uploaded image for project: 'Openmeetings'
  1. Openmeetings
  2. OPENMEETINGS-2496

a/v connection regression

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 5.0.1
    • 5.1.0
    • Audio/Video
    • None

    Description

      There is regression in a/v connection handshake js code which manifests itself as some participants randomly not starting to receive feed for some other participants. But other participants receive them fine and not receiving some others. It can even be that user A doesn't receive feed from user B, but user B receiving user A just fine. Reconnecting code doesn't trigger. Half of the time a/v pod refresh button temporarily fix issue with particular user. This issue become a more evident a more number of participants there are in the room.

      Introduced in this commit: https://github.com/apache/openmeetings/commit/446b93a1d74826c9e54626ceaf3c75ee013c3301

      Reverting this commit entirely or making change below seems to fix the issue at first glance. I'll test this change in production tomorrow.

      --- a/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/raw-video.js
      +++ b/openmeetings-web/src/main/java/org/apache/openmeetings/web/room/raw-video.js
      @@ -586,7 +586,7 @@ var Video = (function() {
              }
              function _processIceCandidate(candidate) {
                      const state = states.length > 0 ? states[0] : null;
      -               if (!state || state.disposed || !state.data.rtcPeer || state.data.rtcPeer.cleaned || state.data.rtcPeer.peerConnection.signaling
      State === 'stable') {
      +               if (!state || state.disposed || !state.data.rtcPeer || state.data.rtcPeer.cleaned) {
                              return;
                      }
                      state.data.rtcPeer.addIceCandidate(candidate, function (error) {
      
      
      

      Attachments

        Activity

          People

            solomax Maxim Solodovnik
            NoSFeRaTU Konstantin Kuzov
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: