From 1078d96ff587b5ef74ebb79c1440b8cdaf3acf19 Mon Sep 17 00:00:00 2001 From: Raja Subramanian Date: Sat, 19 Aug 2023 12:06:09 +0530 Subject: [PATCH] Fix log tag (#1981) - Using "ts" collided with logger built-in tag and causes confusion - Log a bit more to understand large jumps --- pkg/sfu/buffer/rtpstats.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/pkg/sfu/buffer/rtpstats.go b/pkg/sfu/buffer/rtpstats.go index 8f2b54f7d..d99954bb0 100644 --- a/pkg/sfu/buffer/rtpstats.go +++ b/pkg/sfu/buffer/rtpstats.go @@ -812,7 +812,8 @@ func (r *RTPStats) maybeAdjustFirstPacketTime(ts uint32) { return } samplesDuration := time.Duration(float64(samplesDiff) / float64(r.params.ClockRate) * float64(time.Second)) - firstTime := time.Now().Add(-samplesDuration) + now := time.Now() + firstTime := now.Add(-samplesDuration) if firstTime.Before(r.firstTime) { r.logger.Debugw( "adjusting first packet time", @@ -821,10 +822,12 @@ func (r *RTPStats) maybeAdjustFirstPacketTime(ts uint32) { ) if r.firstTime.Sub(firstTime) > firstPacketTimeAdjustThreshold { r.logger.Infow("first packet time adjustment too big, ignoring", - "adjustment", r.firstTime.Sub(firstTime), + "startTime", r.startTime.String(), + "nowTime", now.String(), "before", r.firstTime.String(), "after", firstTime.String(), - "ts", ts, + "adjustment", r.firstTime.Sub(firstTime), + "nowTS", ts, "extStartTS", r.extStartTS, ) } else {