From 314da91f1dff5e4c3921b06180110e7a15b38f22 Mon Sep 17 00:00:00 2001 From: Suvid Sahay <42059321+suvidsahay@users.noreply.github.com> Date: Thu, 26 Mar 2020 15:27:18 +0530 Subject: [PATCH] Fixed Didn't call Close on UserStreamListener log messages when syncing (#940) Signed-off-by: Suvid Sahay Removing the extra imports --- syncapi/sync/userstream.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/syncapi/sync/userstream.go b/syncapi/sync/userstream.go index 6eef8644..88867005 100644 --- a/syncapi/sync/userstream.go +++ b/syncapi/sync/userstream.go @@ -21,7 +21,6 @@ import ( "time" "github.com/matrix-org/dendrite/syncapi/types" - "github.com/matrix-org/util" ) // UserStream represents a communication mechanism between the /sync request goroutine @@ -76,7 +75,6 @@ func (s *UserStream) GetListener(ctx context.Context) UserStreamListener { // Lets be a bit paranoid here and check that Close() is being called runtime.SetFinalizer(&listener, func(l *UserStreamListener) { if !l.hasClosed { - util.GetLogger(ctx).Warn("Didn't call Close on UserStreamListener") l.Close() } })