diff options
Diffstat (limited to 'internal/processing')
| -rw-r--r-- | internal/processing/account/account.go | 9 | ||||
| -rw-r--r-- | internal/processing/account/account_test.go | 6 | ||||
| -rw-r--r-- | internal/processing/account/create.go | 3 | ||||
| -rw-r--r-- | internal/processing/account/delete.go | 2 | ||||
| -rw-r--r-- | internal/processing/account/update.go | 3 | ||||
| -rw-r--r-- | internal/processing/admin/admin.go | 5 | ||||
| -rw-r--r-- | internal/processing/admin/createdomainblock.go | 2 | ||||
| -rw-r--r-- | internal/processing/fromfederator.go | 2 | ||||
| -rw-r--r-- | internal/processing/media/media.go | 5 | ||||
| -rw-r--r-- | internal/processing/notification.go | 3 | ||||
| -rw-r--r-- | internal/processing/processor.go | 26 | ||||
| -rw-r--r-- | internal/processing/processor_test.go | 15 | ||||
| -rw-r--r-- | internal/processing/search.go | 4 | ||||
| -rw-r--r-- | internal/processing/status/status.go | 9 | ||||
| -rw-r--r-- | internal/processing/status/status_test.go | 2 | ||||
| -rw-r--r-- | internal/processing/status/util_test.go | 4 | ||||
| -rw-r--r-- | internal/processing/streaming/openstream.go | 2 | ||||
| -rw-r--r-- | internal/processing/streaming/streaming.go | 5 | ||||
| -rw-r--r-- | internal/processing/streaming/streaming_test.go | 6 | ||||
| -rw-r--r-- | internal/processing/timeline.go | 5 | 
20 files changed, 44 insertions, 74 deletions
| diff --git a/internal/processing/account/account.go b/internal/processing/account/account.go index 831607d94..e88cd3a94 100644 --- a/internal/processing/account/account.go +++ b/internal/processing/account/account.go @@ -22,7 +22,6 @@ import (  	"context"  	"mime/multipart" -	"github.com/sirupsen/logrus"  	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"  	"github.com/superseriousbusiness/gotosocial/internal/config"  	"github.com/superseriousbusiness/gotosocial/internal/db" @@ -87,21 +86,19 @@ type processor struct {  	formatter     text.Formatter  	db            db.DB  	federator     federation.Federator -	log           *logrus.Logger  }  // New returns a new account processor. -func New(db db.DB, tc typeutils.TypeConverter, mediaHandler media.Handler, oauthServer oauth.Server, fromClientAPI chan messages.FromClientAPI, federator federation.Federator, config *config.Config, log *logrus.Logger) Processor { +func New(db db.DB, tc typeutils.TypeConverter, mediaHandler media.Handler, oauthServer oauth.Server, fromClientAPI chan messages.FromClientAPI, federator federation.Federator, config *config.Config) Processor {  	return &processor{  		tc:            tc,  		config:        config,  		mediaHandler:  mediaHandler,  		fromClientAPI: fromClientAPI,  		oauthServer:   oauthServer, -		filter:        visibility.NewFilter(db, log), -		formatter:     text.NewFormatter(config, db, log), +		filter:        visibility.NewFilter(db), +		formatter:     text.NewFormatter(config, db),  		db:            db,  		federator:     federator, -		log:           log,  	}  } diff --git a/internal/processing/account/account_test.go b/internal/processing/account/account_test.go index cfc7130ca..f18290944 100644 --- a/internal/processing/account/account_test.go +++ b/internal/processing/account/account_test.go @@ -21,7 +21,6 @@ package account_test  import (  	"git.iim.gay/grufwub/go-store/kv"  	"github.com/go-fed/activity/pub" -	"github.com/sirupsen/logrus"  	"github.com/stretchr/testify/suite"  	"github.com/superseriousbusiness/gotosocial/internal/config"  	"github.com/superseriousbusiness/gotosocial/internal/db" @@ -41,7 +40,6 @@ type AccountStandardTestSuite struct {  	suite.Suite  	config              *config.Config  	db                  db.DB -	log                 *logrus.Logger  	tc                  typeutils.TypeConverter  	storage             *kv.KVStore  	mediaHandler        media.Handler @@ -77,7 +75,7 @@ func (suite *AccountStandardTestSuite) SetupSuite() {  func (suite *AccountStandardTestSuite) SetupTest() {  	suite.config = testrig.NewTestConfig()  	suite.db = testrig.NewTestDB() -	suite.log = testrig.NewTestLog() +	testrig.InitTestLog()  	suite.tc = testrig.NewTestTypeConverter(suite.db)  	suite.storage = testrig.NewTestStorage()  	suite.mediaHandler = testrig.NewTestMediaHandler(suite.db, suite.storage) @@ -86,7 +84,7 @@ func (suite *AccountStandardTestSuite) SetupTest() {  	suite.httpClient = testrig.NewMockHTTPClient(nil)  	suite.transportController = testrig.NewTestTransportController(suite.httpClient, suite.db)  	suite.federator = testrig.NewTestFederator(suite.db, suite.transportController, suite.storage) -	suite.accountProcessor = account.New(suite.db, suite.tc, suite.mediaHandler, suite.oauthServer, suite.fromClientAPIChan, suite.federator, suite.config, suite.log) +	suite.accountProcessor = account.New(suite.db, suite.tc, suite.mediaHandler, suite.oauthServer, suite.fromClientAPIChan, suite.federator, suite.config)  	testrig.StandardDBSetup(suite.db, nil)  	testrig.StandardStorageSetup(suite.storage, "../../../testrig/media")  } diff --git a/internal/processing/account/create.go b/internal/processing/account/create.go index 9eb618994..56557af42 100644 --- a/internal/processing/account/create.go +++ b/internal/processing/account/create.go @@ -21,6 +21,7 @@ package account  import (  	"context"  	"fmt" +	"github.com/sirupsen/logrus"  	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel" @@ -29,7 +30,7 @@ import (  )  func (p *processor) Create(ctx context.Context, applicationToken oauth2.TokenInfo, application *gtsmodel.Application, form *apimodel.AccountCreateRequest) (*apimodel.Token, error) { -	l := p.log.WithField("func", "accountCreate") +	l := logrus.WithField("func", "accountCreate")  	emailAvailable, err := p.db.IsEmailAvailable(ctx, form.Email)  	if err != nil { diff --git a/internal/processing/account/delete.go b/internal/processing/account/delete.go index 632f49751..4be0c859e 100644 --- a/internal/processing/account/delete.go +++ b/internal/processing/account/delete.go @@ -58,7 +58,7 @@ func (p *processor) Delete(ctx context.Context, account *gtsmodel.Account, origi  	if account.Domain != "" {  		fields["domain"] = account.Domain  	} -	l := p.log.WithFields(fields) +	l := logrus.WithFields(fields)  	l.Debug("beginning account delete process") diff --git a/internal/processing/account/update.go b/internal/processing/account/update.go index 1ab25787f..54bf372e7 100644 --- a/internal/processing/account/update.go +++ b/internal/processing/account/update.go @@ -23,6 +23,7 @@ import (  	"context"  	"errors"  	"fmt" +	"github.com/sirupsen/logrus"  	"io"  	"mime/multipart" @@ -37,7 +38,7 @@ import (  )  func (p *processor) Update(ctx context.Context, account *gtsmodel.Account, form *apimodel.UpdateCredentialsRequest) (*apimodel.Account, error) { -	l := p.log.WithField("func", "AccountUpdate") +	l := logrus.WithField("func", "AccountUpdate")  	if form.Discoverable != nil {  		account.Discoverable = *form.Discoverable diff --git a/internal/processing/admin/admin.go b/internal/processing/admin/admin.go index 92f69f06b..1aaaced5e 100644 --- a/internal/processing/admin/admin.go +++ b/internal/processing/admin/admin.go @@ -22,7 +22,6 @@ import (  	"context"  	"mime/multipart" -	"github.com/sirupsen/logrus"  	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"  	"github.com/superseriousbusiness/gotosocial/internal/config"  	"github.com/superseriousbusiness/gotosocial/internal/db" @@ -49,17 +48,15 @@ type processor struct {  	mediaHandler  media.Handler  	fromClientAPI chan messages.FromClientAPI  	db            db.DB -	log           *logrus.Logger  }  // New returns a new admin processor. -func New(db db.DB, tc typeutils.TypeConverter, mediaHandler media.Handler, fromClientAPI chan messages.FromClientAPI, config *config.Config, log *logrus.Logger) Processor { +func New(db db.DB, tc typeutils.TypeConverter, mediaHandler media.Handler, fromClientAPI chan messages.FromClientAPI, config *config.Config) Processor {  	return &processor{  		tc:            tc,  		config:        config,  		mediaHandler:  mediaHandler,  		fromClientAPI: fromClientAPI,  		db:            db, -		log:           log,  	}  } diff --git a/internal/processing/admin/createdomainblock.go b/internal/processing/admin/createdomainblock.go index 399007fe0..f544a84bd 100644 --- a/internal/processing/admin/createdomainblock.go +++ b/internal/processing/admin/createdomainblock.go @@ -87,7 +87,7 @@ func (p *processor) DomainBlockCreate(ctx context.Context, account *gtsmodel.Acc  // 2. Delete the instance account for that instance if it exists.  // 3. Select all accounts from this instance and pass them through the delete functionality of the processor.  func (p *processor) initiateDomainBlockSideEffects(ctx context.Context, account *gtsmodel.Account, block *gtsmodel.DomainBlock) { -	l := p.log.WithFields(logrus.Fields{ +	l := logrus.WithFields(logrus.Fields{  		"func":   "domainBlockProcessSideEffects",  		"domain": block.Domain,  	}) diff --git a/internal/processing/fromfederator.go b/internal/processing/fromfederator.go index 449cc6f08..8a36fb9b7 100644 --- a/internal/processing/fromfederator.go +++ b/internal/processing/fromfederator.go @@ -35,7 +35,7 @@ import (  // and directs the message into the appropriate side effect handler function, or simply does nothing if there's  // no handler function defined for the combination of Activity and Object.  func (p *processor) ProcessFromFederator(ctx context.Context, federatorMsg messages.FromFederator) error { -	l := p.log.WithFields(logrus.Fields{ +	l := logrus.WithFields(logrus.Fields{  		"func":           "processFromFederator",  		"APActivityType": federatorMsg.APActivityType,  		"APObjectType":   federatorMsg.APObjectType, diff --git a/internal/processing/media/media.go b/internal/processing/media/media.go index 4c8416483..e7c9a6e0f 100644 --- a/internal/processing/media/media.go +++ b/internal/processing/media/media.go @@ -22,7 +22,6 @@ import (  	"context"  	"git.iim.gay/grufwub/go-store/kv" -	"github.com/sirupsen/logrus"  	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"  	"github.com/superseriousbusiness/gotosocial/internal/config"  	"github.com/superseriousbusiness/gotosocial/internal/db" @@ -49,17 +48,15 @@ type processor struct {  	mediaHandler media.Handler  	storage      *kv.KVStore  	db           db.DB -	log          *logrus.Logger  }  // New returns a new media processor. -func New(db db.DB, tc typeutils.TypeConverter, mediaHandler media.Handler, storage *kv.KVStore, config *config.Config, log *logrus.Logger) Processor { +func New(db db.DB, tc typeutils.TypeConverter, mediaHandler media.Handler, storage *kv.KVStore, config *config.Config) Processor {  	return &processor{  		tc:           tc,  		config:       config,  		mediaHandler: mediaHandler,  		storage:      storage,  		db:           db, -		log:          log,  	}  } diff --git a/internal/processing/notification.go b/internal/processing/notification.go index 27280a973..0710f2266 100644 --- a/internal/processing/notification.go +++ b/internal/processing/notification.go @@ -20,6 +20,7 @@ package processing  import (  	"context" +	"github.com/sirupsen/logrus"  	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"  	"github.com/superseriousbusiness/gotosocial/internal/gtserror" @@ -27,7 +28,7 @@ import (  )  func (p *processor) NotificationsGet(ctx context.Context, authed *oauth.Auth, limit int, maxID string, sinceID string) ([]*apimodel.Notification, gtserror.WithCode) { -	l := p.log.WithField("func", "NotificationsGet") +	l := logrus.WithField("func", "NotificationsGet")  	notifs, err := p.db.GetNotifications(ctx, authed.Account.ID, limit, maxID, sinceID)  	if err != nil { diff --git a/internal/processing/processor.go b/internal/processing/processor.go index c643738c2..5732ad092 100644 --- a/internal/processing/processor.go +++ b/internal/processing/processor.go @@ -229,7 +229,6 @@ type processor struct {  	fromFederator   chan messages.FromFederator  	federator       federation.Federator  	stop            chan interface{} -	log             *logrus.Logger  	config          *config.Config  	tc              typeutils.TypeConverter  	oauthServer     oauth.Server @@ -250,23 +249,22 @@ type processor struct {  	mediaProcessor     mediaProcessor.Processor  } -// NewProcessor returns a new Processor that uses the given federator and logger -func NewProcessor(config *config.Config, tc typeutils.TypeConverter, federator federation.Federator, oauthServer oauth.Server, mediaHandler media.Handler, storage *kv.KVStore, timelineManager timeline.Manager, db db.DB, log *logrus.Logger) Processor { +// NewProcessor returns a new Processor that uses the given federator +func NewProcessor(config *config.Config, tc typeutils.TypeConverter, federator federation.Federator, oauthServer oauth.Server, mediaHandler media.Handler, storage *kv.KVStore, timelineManager timeline.Manager, db db.DB) Processor {  	fromClientAPI := make(chan messages.FromClientAPI, 1000)  	fromFederator := make(chan messages.FromFederator, 1000) -	statusProcessor := status.New(db, tc, config, fromClientAPI, log) -	streamingProcessor := streaming.New(db, oauthServer, log) -	accountProcessor := account.New(db, tc, mediaHandler, oauthServer, fromClientAPI, federator, config, log) -	adminProcessor := admin.New(db, tc, mediaHandler, fromClientAPI, config, log) -	mediaProcessor := mediaProcessor.New(db, tc, mediaHandler, storage, config, log) +	statusProcessor := status.New(db, tc, config, fromClientAPI) +	streamingProcessor := streaming.New(db, oauthServer) +	accountProcessor := account.New(db, tc, mediaHandler, oauthServer, fromClientAPI, federator, config) +	adminProcessor := admin.New(db, tc, mediaHandler, fromClientAPI, config) +	mediaProcessor := mediaProcessor.New(db, tc, mediaHandler, storage, config)  	return &processor{  		fromClientAPI:   fromClientAPI,  		fromFederator:   fromFederator,  		federator:       federator,  		stop:            make(chan interface{}), -		log:             log,  		config:          config,  		tc:              tc,  		oauthServer:     oauthServer, @@ -274,7 +272,7 @@ func NewProcessor(config *config.Config, tc typeutils.TypeConverter, federator f  		storage:         storage,  		timelineManager: timelineManager,  		db:              db, -		filter:          visibility.NewFilter(db, log), +		filter:          visibility.NewFilter(db),  		accountProcessor:   accountProcessor,  		adminProcessor:     adminProcessor, @@ -291,17 +289,17 @@ func (p *processor) Start(ctx context.Context) error {  		for {  			select {  			case clientMsg := <-p.fromClientAPI: -				p.log.Tracef("received message FROM client API: %+v", clientMsg) +				logrus.Tracef("received message FROM client API: %+v", clientMsg)  				go func() {  					if err := p.ProcessFromClientAPI(ctx, clientMsg); err != nil { -						p.log.Error(err) +						logrus.Error(err)  					}  				}()  			case federatorMsg := <-p.fromFederator: -				p.log.Tracef("received message FROM federator: %+v", federatorMsg) +				logrus.Tracef("received message FROM federator: %+v", federatorMsg)  				go func() {  					if err := p.ProcessFromFederator(ctx, federatorMsg); err != nil { -						p.log.Error(err) +						logrus.Error(err)  					}  				}()  			case <-p.stop: diff --git a/internal/processing/processor_test.go b/internal/processing/processor_test.go index 7a1db442e..f2a9e455e 100644 --- a/internal/processing/processor_test.go +++ b/internal/processing/processor_test.go @@ -28,7 +28,6 @@ import (  	"git.iim.gay/grufwub/go-store/kv"  	"github.com/go-fed/activity/streams" -	"github.com/sirupsen/logrus"  	"github.com/stretchr/testify/suite"  	"github.com/superseriousbusiness/gotosocial/internal/config"  	"github.com/superseriousbusiness/gotosocial/internal/db" @@ -48,7 +47,6 @@ type ProcessingStandardTestSuite struct {  	suite.Suite  	config              *config.Config  	db                  db.DB -	log                 *logrus.Logger  	storage             *kv.KVStore  	typeconverter       typeutils.TypeConverter  	transportController transport.Controller @@ -100,7 +98,7 @@ func (suite *ProcessingStandardTestSuite) SetupSuite() {  func (suite *ProcessingStandardTestSuite) SetupTest() {  	suite.config = testrig.NewTestConfig()  	suite.db = testrig.NewTestDB() -	suite.log = testrig.NewTestLog() +	testrig.InitTestLog()  	suite.storage = testrig.NewTestStorage()  	suite.typeconverter = testrig.NewTestTypeConverter(suite.db) @@ -190,16 +188,7 @@ func (suite *ProcessingStandardTestSuite) SetupTest() {  	suite.mediaHandler = testrig.NewTestMediaHandler(suite.db, suite.storage)  	suite.timelineManager = testrig.NewTestTimelineManager(suite.db) -	suite.processor = processing.NewProcessor( -		suite.config, -		suite.typeconverter, -		suite.federator, -		suite.oauthServer, -		suite.mediaHandler, -		suite.storage, -		suite.timelineManager, -		suite.db, -		suite.log) +	suite.processor = processing.NewProcessor(suite.config, suite.typeconverter, suite.federator, suite.oauthServer, suite.mediaHandler, suite.storage, suite.timelineManager, suite.db)  	testrig.StandardDBSetup(suite.db, suite.testAccounts)  	testrig.StandardStorageSetup(suite.storage, "../../testrig/media") diff --git a/internal/processing/search.go b/internal/processing/search.go index c5bfd722b..a73315396 100644 --- a/internal/processing/search.go +++ b/internal/processing/search.go @@ -34,7 +34,7 @@ import (  )  func (p *processor) SearchGet(ctx context.Context, authed *oauth.Auth, searchQuery *apimodel.SearchQuery) (*apimodel.SearchResult, gtserror.WithCode) { -	l := p.log.WithFields(logrus.Fields{ +	l := logrus.WithFields(logrus.Fields{  		"func":  "SearchGet",  		"query": searchQuery.Query,  	}) @@ -116,7 +116,7 @@ func (p *processor) SearchGet(ctx context.Context, authed *oauth.Auth, searchQue  }  func (p *processor) searchStatusByURI(ctx context.Context, authed *oauth.Auth, uri *url.URL, resolve bool) (*gtsmodel.Status, error) { -	l := p.log.WithFields(logrus.Fields{ +	l := logrus.WithFields(logrus.Fields{  		"func":    "searchStatusByURI",  		"uri":     uri.String(),  		"resolve": resolve, diff --git a/internal/processing/status/status.go b/internal/processing/status/status.go index 10faa5696..666c237b7 100644 --- a/internal/processing/status/status.go +++ b/internal/processing/status/status.go @@ -21,7 +21,6 @@ package status  import (  	"context" -	"github.com/sirupsen/logrus"  	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"  	"github.com/superseriousbusiness/gotosocial/internal/config"  	"github.com/superseriousbusiness/gotosocial/internal/db" @@ -77,18 +76,16 @@ type processor struct {  	filter        visibility.Filter  	formatter     text.Formatter  	fromClientAPI chan messages.FromClientAPI -	log           *logrus.Logger  }  // New returns a new status processor. -func New(db db.DB, tc typeutils.TypeConverter, config *config.Config, fromClientAPI chan messages.FromClientAPI, log *logrus.Logger) Processor { +func New(db db.DB, tc typeutils.TypeConverter, config *config.Config, fromClientAPI chan messages.FromClientAPI) Processor {  	return &processor{  		tc:            tc,  		config:        config,  		db:            db, -		filter:        visibility.NewFilter(db, log), -		formatter:     text.NewFormatter(config, db, log), +		filter:        visibility.NewFilter(db), +		formatter:     text.NewFormatter(config, db),  		fromClientAPI: fromClientAPI, -		log:           log,  	}  } diff --git a/internal/processing/status/status_test.go b/internal/processing/status/status_test.go index 707a4843b..c5c439057 100644 --- a/internal/processing/status/status_test.go +++ b/internal/processing/status/status_test.go @@ -19,7 +19,6 @@  package status_test  import ( -	"github.com/sirupsen/logrus"  	"github.com/stretchr/testify/suite"  	"github.com/superseriousbusiness/gotosocial/internal/config"  	"github.com/superseriousbusiness/gotosocial/internal/db" @@ -34,7 +33,6 @@ type StatusStandardTestSuite struct {  	suite.Suite  	config            *config.Config  	db                db.DB -	log               *logrus.Logger  	typeConverter     typeutils.TypeConverter  	fromClientAPIChan chan messages.FromClientAPI diff --git a/internal/processing/status/util_test.go b/internal/processing/status/util_test.go index f80cf9342..37e2f2dfc 100644 --- a/internal/processing/status/util_test.go +++ b/internal/processing/status/util_test.go @@ -67,10 +67,10 @@ func (suite *UtilTestSuite) SetupSuite() {  func (suite *UtilTestSuite) SetupTest() {  	suite.config = testrig.NewTestConfig()  	suite.db = testrig.NewTestDB() -	suite.log = testrig.NewTestLog() +	testrig.InitTestLog()  	suite.typeConverter = testrig.NewTestTypeConverter(suite.db)  	suite.fromClientAPIChan = make(chan messages.FromClientAPI, 100) -	suite.status = status.New(suite.db, suite.typeConverter, suite.config, suite.fromClientAPIChan, suite.log) +	suite.status = status.New(suite.db, suite.typeConverter, suite.config, suite.fromClientAPIChan)  	testrig.StandardDBSetup(suite.db, nil)  } diff --git a/internal/processing/streaming/openstream.go b/internal/processing/streaming/openstream.go index 74b6486f5..706cc0675 100644 --- a/internal/processing/streaming/openstream.go +++ b/internal/processing/streaming/openstream.go @@ -31,7 +31,7 @@ import (  )  func (p *processor) OpenStreamForAccount(ctx context.Context, account *gtsmodel.Account, streamType string) (*stream.Stream, gtserror.WithCode) { -	l := p.log.WithFields(logrus.Fields{ +	l := logrus.WithFields(logrus.Fields{  		"func":       "OpenStreamForAccount",  		"account":    account.ID,  		"streamType": streamType, diff --git a/internal/processing/streaming/streaming.go b/internal/processing/streaming/streaming.go index abce30cd1..604ec1267 100644 --- a/internal/processing/streaming/streaming.go +++ b/internal/processing/streaming/streaming.go @@ -22,7 +22,6 @@ import (  	"context"  	"sync" -	"github.com/sirupsen/logrus"  	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"  	"github.com/superseriousbusiness/gotosocial/internal/db"  	"github.com/superseriousbusiness/gotosocial/internal/gtserror" @@ -47,16 +46,14 @@ type Processor interface {  type processor struct {  	db          db.DB -	log         *logrus.Logger  	oauthServer oauth.Server  	streamMap   *sync.Map  }  // New returns a new status processor. -func New(db db.DB, oauthServer oauth.Server, log *logrus.Logger) Processor { +func New(db db.DB, oauthServer oauth.Server) Processor {  	return &processor{  		db:          db, -		log:         log,  		oauthServer: oauthServer,  		streamMap:   &sync.Map{},  	} diff --git a/internal/processing/streaming/streaming_test.go b/internal/processing/streaming/streaming_test.go index acc090b06..cbb899d12 100644 --- a/internal/processing/streaming/streaming_test.go +++ b/internal/processing/streaming/streaming_test.go @@ -19,7 +19,6 @@  package streaming_test  import ( -	"github.com/sirupsen/logrus"  	"github.com/stretchr/testify/suite"  	"github.com/superseriousbusiness/gotosocial/internal/db"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel" @@ -34,7 +33,6 @@ type StreamingTestSuite struct {  	testTokens   map[string]*gtsmodel.Token  	db           db.DB  	oauthServer  oauth.Server -	log          *logrus.Logger  	streamingProcessor streaming.Processor  } @@ -44,8 +42,8 @@ func (suite *StreamingTestSuite) SetupTest() {  	suite.testTokens = testrig.NewTestTokens()  	suite.db = testrig.NewTestDB()  	suite.oauthServer = testrig.NewTestOauthServer(suite.db) -	suite.log = testrig.NewTestLog() -	suite.streamingProcessor = streaming.New(suite.db, suite.oauthServer, suite.log) +	testrig.InitTestLog() +	suite.streamingProcessor = streaming.New(suite.db, suite.oauthServer)  	testrig.StandardDBSetup(suite.db, suite.testAccounts)  } diff --git a/internal/processing/timeline.go b/internal/processing/timeline.go index 249ef542d..64238225f 100644 --- a/internal/processing/timeline.go +++ b/internal/processing/timeline.go @@ -21,6 +21,7 @@ package processing  import (  	"context"  	"fmt" +	"github.com/sirupsen/logrus"  	"net/url"  	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model" @@ -129,7 +130,7 @@ func (p *processor) FavedTimelineGet(ctx context.Context, authed *oauth.Auth, ma  }  func (p *processor) filterPublicStatuses(ctx context.Context, authed *oauth.Auth, statuses []*gtsmodel.Status) ([]*apimodel.Status, error) { -	l := p.log.WithField("func", "filterPublicStatuses") +	l := logrus.WithField("func", "filterPublicStatuses")  	apiStatuses := []*apimodel.Status{}  	for _, s := range statuses { @@ -164,7 +165,7 @@ func (p *processor) filterPublicStatuses(ctx context.Context, authed *oauth.Auth  }  func (p *processor) filterFavedStatuses(ctx context.Context, authed *oauth.Auth, statuses []*gtsmodel.Status) ([]*apimodel.Status, error) { -	l := p.log.WithField("func", "filterFavedStatuses") +	l := logrus.WithField("func", "filterFavedStatuses")  	apiStatuses := []*apimodel.Status{}  	for _, s := range statuses { | 
