From 420e2fb22bc7aa4967ddadb11e444079efdf5117 Mon Sep 17 00:00:00 2001 From: kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com> Date: Thu, 28 Apr 2022 13:23:11 +0100 Subject: replace async client API / federator msg processing with worker pools (#497) * replace async client API / federator msg processing with worker pools * appease our lord-and-saviour, the linter --- internal/processing/federation/federation.go | 21 +++++++++------------ internal/processing/federation/postinbox.go | 6 +----- 2 files changed, 10 insertions(+), 17 deletions(-) (limited to 'internal/processing/federation') diff --git a/internal/processing/federation/federation.go b/internal/processing/federation/federation.go index 0d8d47c14..adf7b0437 100644 --- a/internal/processing/federation/federation.go +++ b/internal/processing/federation/federation.go @@ -27,7 +27,6 @@ import ( "github.com/superseriousbusiness/gotosocial/internal/db" "github.com/superseriousbusiness/gotosocial/internal/federation" "github.com/superseriousbusiness/gotosocial/internal/gtserror" - "github.com/superseriousbusiness/gotosocial/internal/messages" "github.com/superseriousbusiness/gotosocial/internal/typeutils" "github.com/superseriousbusiness/gotosocial/internal/visibility" ) @@ -81,20 +80,18 @@ type Processor interface { } type processor struct { - db db.DB - federator federation.Federator - tc typeutils.TypeConverter - filter visibility.Filter - fromFederator chan messages.FromFederator + db db.DB + federator federation.Federator + tc typeutils.TypeConverter + filter visibility.Filter } // New returns a new federation processor. -func New(db db.DB, tc typeutils.TypeConverter, federator federation.Federator, fromFederator chan messages.FromFederator) Processor { +func New(db db.DB, tc typeutils.TypeConverter, federator federation.Federator) Processor { return &processor{ - db: db, - federator: federator, - tc: tc, - filter: visibility.NewFilter(db), - fromFederator: fromFederator, + db: db, + federator: federator, + tc: tc, + filter: visibility.NewFilter(db), } } diff --git a/internal/processing/federation/postinbox.go b/internal/processing/federation/postinbox.go index ef09de498..b1ba4e1e0 100644 --- a/internal/processing/federation/postinbox.go +++ b/internal/processing/federation/postinbox.go @@ -21,12 +21,8 @@ package federation import ( "context" "net/http" - - "github.com/superseriousbusiness/gotosocial/internal/ap" ) func (p *processor) PostInbox(ctx context.Context, w http.ResponseWriter, r *http.Request) (bool, error) { - // pass the fromFederator channel through to postInbox, since it'll be needed later - contextWithChannel := context.WithValue(ctx, ap.ContextFromFederatorChan, p.fromFederator) - return p.federator.FederatingActor().PostInbox(contextWithChannel, w, r) + return p.federator.FederatingActor().PostInbox(ctx, w, r) } -- cgit v1.2.3