Skip to content

Commit

Permalink
limits warnings about packet being dropped at the track projection le…
Browse files Browse the repository at this point in the history
…vel to VP8
  • Loading branch information
gpolitis committed Oct 4, 2019
1 parent 3736d35 commit c29af04
Showing 1 changed file with 23 additions and 17 deletions.
40 changes: 23 additions & 17 deletions src/main/java/org/jitsi/videobridge/cc/AdaptiveTrackProjection.java
Original file line number Diff line number Diff line change
Expand Up @@ -191,25 +191,31 @@ public boolean accept(@NotNull RawPacket rtpPacket)
if (encoding == null)
{
MediaStreamTrackDesc sourceTrack = getSource();
if (sourceTrack != null)

// We're exclusively interesting in finding issues with VP8 support
// at the moment.
if (Constants.VP8.equalsIgnoreCase(contextCopy.getFormat().getEncoding()))
{
MediaStreamTrackReceiver mediaStreamTrackReceiver
= sourceTrack.getMediaStreamTrackReceiver();

logger.warn(
"Dropping an RTP packet, because egress was unable " +
"to find an encoding for raw packet " +
mediaStreamTrackReceiver
.getStream().packetToString(rtpPacket)
+ ". Ingress is aware of these tracks: "
+ Arrays.toString(
if (sourceTrack != null)
{
MediaStreamTrackReceiver mediaStreamTrackReceiver
= sourceTrack.getMediaStreamTrackReceiver();

logger.warn(
"Dropping an RTP packet, because egress was unable " +
"to find an encoding for raw packet " +
mediaStreamTrackReceiver
.getStream().packetToString(rtpPacket)
+ ". Ingress is aware of these tracks: "
+ Arrays.toString(
mediaStreamTrackReceiver.getMediaStreamTracks()));
}
else
{
logger.warn(
"Dropping an RTP packet, because egress was unable " +
"to find an encoding for raw packet " + rtpPacket);
}
else
{
logger.warn(
"Dropping an RTP packet, because egress was unable " +
"to find an encoding for raw packet " + rtpPacket);
}
}

return false;
Expand Down

0 comments on commit c29af04

Please sign in to comment.