From 5a30f6ed87a1d49ac0842acdce52c3653f2d59ea Mon Sep 17 00:00:00 2001 From: Raja Subramanian Date: Sat, 2 Jul 2022 14:10:24 +0530 Subject: [PATCH] Add logging for permissions debugging (#799) * Add logging for permissions debugging * Remove debug --- pkg/rtc/mediatracksubscriptions.go | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/pkg/rtc/mediatracksubscriptions.go b/pkg/rtc/mediatracksubscriptions.go index bcfc9dc3a..34c63d19f 100644 --- a/pkg/rtc/mediatracksubscriptions.go +++ b/pkg/rtc/mediatracksubscriptions.go @@ -869,9 +869,9 @@ func (t *MediaTrackSubscriptions) downTrackClosed( if sender == nil { return } - t.params.Logger.Debugw("removing peerconnection track", - "subscriber", sub.Identity(), - "subscriberID", subscriberID, + sub.GetLogger().Infow("removing peerconnection track", + "publisher", subTrack.PublisherIdentity(), + "publisherID", subTrack.PublisherID(), "kind", t.params.MediaTrack.Kind(), ) if err := sub.SubscriberPC().RemoveTrack(sender); err != nil { @@ -882,10 +882,10 @@ func (t *MediaTrackSubscriptions) downTrackClosed( if _, ok := err.(*rtcerr.InvalidStateError); !ok { // most of these are safe to ignore, since the track state might have already // been set to Inactive - t.params.Logger.Debugw("could not remove remoteTrack from forwarder", + sub.GetLogger().Infow("could not remove remoteTrack from forwarder", "error", err, - "subscriber", sub.Identity(), - "subscriberID", subscriberID, + "publisher", subTrack.PublisherIdentity(), + "publisherID", subTrack.PublisherID(), ) } }