From 447793d077d2d49cfac57330c590310dc335c28a Mon Sep 17 00:00:00 2001 From: Raja Subramanian Date: Wed, 29 May 2024 11:26:30 +0530 Subject: [PATCH] Move RTT errors to Debugw. (#2742) With the move to forwarding NTP timestamp as is, we get a bunch more of this error logged as the remote is basing it off of previous report and local (i. e. server-side) bases it off of a more recent report. Anyhow, this code has been around for a long time and there is nothing new to learn from those errors. Just log it at Debugw in case we can learn something from it for specific projects or environments where Debugw is okay. --- pkg/sfu/buffer/rtpstats_sender.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/pkg/sfu/buffer/rtpstats_sender.go b/pkg/sfu/buffer/rtpstats_sender.go index 64aca2fa8..a1b209a80 100644 --- a/pkg/sfu/buffer/rtpstats_sender.go +++ b/pkg/sfu/buffer/rtpstats_sender.go @@ -491,9 +491,7 @@ func (r *RTPStatsSender) UpdateFromReceiverReport(rr rtcp.ReceptionReport) (rtt if err == nil { isRttChanged = rtt != r.rtt } else { - if !errors.Is(err, mediatransportutil.ErrRttNotLastSenderReport) && !errors.Is(err, mediatransportutil.ErrRttNoLastSenderReport) { - r.logger.Warnw("error getting rtt", err) - } + r.logger.Debugw("error getting rtt", "error", err) } }