Merge pull request #466 from jitsi/rtcp-from-default-sender-ssrcs

ref(sdp): Do not add recvonly SSRC for muted video tracks.

modules/RTC/LocalSdpMunger.js

@@ -93,14 +93,6 @@ export default class LocalSdpMunger {
                 // eslint-disable-next-line no-continue
                 continue;
             }
-            if (!videoMLine.getSSRCCount()) {
-                logger.error(
-                    `${this.tpc} - no video SSRCs found`
-                        + '(should be at least the recv-only one)');
-
-                // eslint-disable-next-line no-continue
-                continue;
-            }
 
             modified = true;
 

modules/xmpp/SdpConsistency.js

@@ -35,6 +35,7 @@ export default class SdpConsistency {
      */
     clearVideoSsrcCache() {
         this.cachedPrimarySsrc = null;
+        this.injectRecvOnly = false;
     }
 
     /**
@@ -81,17 +82,18 @@ export default class SdpConsistency {
 
             return sdpStr;
         }
+
         if (videoMLine.direction === 'recvonly') {
             // If the mline is recvonly, we'll add the primary
             //  ssrc as a recvonly ssrc
-            if (this.cachedPrimarySsrc) {
+            if (this.cachedPrimarySsrc && this.injectRecvOnly) {
                 videoMLine.addSSRCAttribute({
                     id: this.cachedPrimarySsrc,
                     attribute: 'cname',
                     value: `recvonly-${this.cachedPrimarySsrc}`
                 });
             } else {
-                logger.error(
+                logger.info(
                     `${this.logPrefix} no SSRC found for the recvonly video`
                         + 'stream!');
             }
@@ -129,6 +131,8 @@ export default class SdpConsistency {
                     `${this.logPrefix} sdp-consistency caching primary ssrc`
                         + `${this.cachedPrimarySsrc}`);
             }
+
+            this.injectRecvOnly = true;
         }
 
         return sdpTransformer.toRawSDP();