diff options
Diffstat (limited to 'internal/federation')
| -rw-r--r-- | internal/federation/dereferencing/account.go | 6 | ||||
| -rw-r--r-- | internal/federation/dereferencing/collectionpage.go | 3 | ||||
| -rw-r--r-- | internal/federation/dereferencing/status.go | 18 | ||||
| -rw-r--r-- | internal/federation/dereferencing/status_test.go | 5 | ||||
| -rw-r--r-- | internal/federation/federatingdb/accept.go | 18 | ||||
| -rw-r--r-- | internal/federation/federatingdb/announce.go | 10 | ||||
| -rw-r--r-- | internal/federation/federatingdb/create.go | 38 | ||||
| -rw-r--r-- | internal/federation/federatingdb/delete.go | 16 | ||||
| -rw-r--r-- | internal/federation/federatingdb/undo.go | 9 | ||||
| -rw-r--r-- | internal/federation/federatingdb/update.go | 29 | ||||
| -rw-r--r-- | internal/federation/federatingdb/util.go | 15 | 
11 files changed, 89 insertions, 78 deletions
| diff --git a/internal/federation/dereferencing/account.go b/internal/federation/dereferencing/account.go index 8cae002e8..eb6a3a760 100644 --- a/internal/federation/dereferencing/account.go +++ b/internal/federation/dereferencing/account.go @@ -165,19 +165,19 @@ func (d *deref) dereferenceAccountable(ctx context.Context, username string, rem  	}  	switch t.GetTypeName() { -	case string(gtsmodel.ActivityStreamsPerson): +	case string(ap.ActorPerson):  		p, ok := t.(vocab.ActivityStreamsPerson)  		if !ok {  			return nil, errors.New("DereferenceAccountable: error resolving type as activitystreams person")  		}  		return p, nil -	case string(gtsmodel.ActivityStreamsApplication): +	case string(ap.ActorApplication):  		p, ok := t.(vocab.ActivityStreamsApplication)  		if !ok {  			return nil, errors.New("DereferenceAccountable: error resolving type as activitystreams application")  		}  		return p, nil -	case string(gtsmodel.ActivityStreamsService): +	case string(ap.ActorService):  		p, ok := t.(vocab.ActivityStreamsService)  		if !ok {  			return nil, errors.New("DereferenceAccountable: error resolving type as activitystreams service") diff --git a/internal/federation/dereferencing/collectionpage.go b/internal/federation/dereferencing/collectionpage.go index 6f0beeaf6..c5a54402c 100644 --- a/internal/federation/dereferencing/collectionpage.go +++ b/internal/federation/dereferencing/collectionpage.go @@ -28,7 +28,6 @@ import (  	"github.com/go-fed/activity/streams"  	"github.com/go-fed/activity/streams/vocab"  	"github.com/superseriousbusiness/gotosocial/internal/ap" -	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel"  )  // DereferenceCollectionPage returns the activitystreams CollectionPage at the specified IRI, or an error if something goes wrong. @@ -57,7 +56,7 @@ func (d *deref) DereferenceCollectionPage(ctx context.Context, username string,  		return nil, fmt.Errorf("DereferenceCollectionPage: error resolving json into ap vocab type: %s", err)  	} -	if t.GetTypeName() != gtsmodel.ActivityStreamsCollectionPage { +	if t.GetTypeName() != ap.ObjectCollectionPage {  		return nil, fmt.Errorf("DereferenceCollectionPage: type name %s not supported", t.GetTypeName())  	} diff --git a/internal/federation/dereferencing/status.go b/internal/federation/dereferencing/status.go index 7a7f928f1..b8f5bba3b 100644 --- a/internal/federation/dereferencing/status.go +++ b/internal/federation/dereferencing/status.go @@ -154,55 +154,55 @@ func (d *deref) dereferenceStatusable(ctx context.Context, username string, remo  	// Article, Document, Image, Video, Note, Page, Event, Place, Mention, Profile  	switch t.GetTypeName() { -	case gtsmodel.ActivityStreamsArticle: +	case ap.ObjectArticle:  		p, ok := t.(vocab.ActivityStreamsArticle)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsArticle")  		}  		return p, nil -	case gtsmodel.ActivityStreamsDocument: +	case ap.ObjectDocument:  		p, ok := t.(vocab.ActivityStreamsDocument)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsDocument")  		}  		return p, nil -	case gtsmodel.ActivityStreamsImage: +	case ap.ObjectImage:  		p, ok := t.(vocab.ActivityStreamsImage)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsImage")  		}  		return p, nil -	case gtsmodel.ActivityStreamsVideo: +	case ap.ObjectVideo:  		p, ok := t.(vocab.ActivityStreamsVideo)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsVideo")  		}  		return p, nil -	case gtsmodel.ActivityStreamsNote: +	case ap.ObjectNote:  		p, ok := t.(vocab.ActivityStreamsNote)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsNote")  		}  		return p, nil -	case gtsmodel.ActivityStreamsPage: +	case ap.ObjectPage:  		p, ok := t.(vocab.ActivityStreamsPage)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsPage")  		}  		return p, nil -	case gtsmodel.ActivityStreamsEvent: +	case ap.ObjectEvent:  		p, ok := t.(vocab.ActivityStreamsEvent)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsEvent")  		}  		return p, nil -	case gtsmodel.ActivityStreamsPlace: +	case ap.ObjectPlace:  		p, ok := t.(vocab.ActivityStreamsPlace)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsPlace")  		}  		return p, nil -	case gtsmodel.ActivityStreamsProfile: +	case ap.ObjectProfile:  		p, ok := t.(vocab.ActivityStreamsProfile)  		if !ok {  			return nil, errors.New("DereferenceStatusable: error resolving type as ActivityStreamsProfile") diff --git a/internal/federation/dereferencing/status_test.go b/internal/federation/dereferencing/status_test.go index 43732ac77..1ab4ade53 100644 --- a/internal/federation/dereferencing/status_test.go +++ b/internal/federation/dereferencing/status_test.go @@ -29,6 +29,7 @@ import (  	"github.com/go-fed/activity/streams"  	"github.com/stretchr/testify/suite" +	"github.com/superseriousbusiness/gotosocial/internal/ap"  	"github.com/superseriousbusiness/gotosocial/internal/db"  	"github.com/superseriousbusiness/gotosocial/internal/federation/dereferencing"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel" @@ -133,7 +134,7 @@ func (suite *StatusTestSuite) TestDereferenceSimpleStatus() {  	suite.False(status.Local)  	suite.Empty(status.ContentWarning)  	suite.Equal(gtsmodel.VisibilityPublic, status.Visibility) -	suite.Equal(gtsmodel.ActivityStreamsNote, status.ActivityStreamsType) +	suite.Equal(ap.ObjectNote, status.ActivityStreamsType)  	// status should be in the database  	dbStatus, err := suite.db.GetStatusByURI(context.Background(), status.URI) @@ -171,7 +172,7 @@ func (suite *StatusTestSuite) TestDereferenceStatusWithMention() {  	suite.False(status.Local)  	suite.Empty(status.ContentWarning)  	suite.Equal(gtsmodel.VisibilityPublic, status.Visibility) -	suite.Equal(gtsmodel.ActivityStreamsNote, status.ActivityStreamsType) +	suite.Equal(ap.ObjectNote, status.ActivityStreamsType)  	// status should be in the database  	dbStatus, err := suite.db.GetStatusByURI(context.Background(), status.URI) diff --git a/internal/federation/federatingdb/accept.go b/internal/federation/federatingdb/accept.go index 0b14e8a6a..ceaf4c4ef 100644 --- a/internal/federation/federatingdb/accept.go +++ b/internal/federation/federatingdb/accept.go @@ -27,8 +27,10 @@ import (  	"github.com/go-fed/activity/streams"  	"github.com/go-fed/activity/streams/vocab"  	"github.com/sirupsen/logrus" +	"github.com/superseriousbusiness/gotosocial/internal/ap"  	"github.com/superseriousbusiness/gotosocial/internal/db"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel" +	"github.com/superseriousbusiness/gotosocial/internal/messages"  	"github.com/superseriousbusiness/gotosocial/internal/util"  ) @@ -67,7 +69,7 @@ func (f *federatingDB) Accept(ctx context.Context, accept vocab.ActivityStreamsA  		l.Error("ACCEPT: from federator channel wasn't set on context")  		return nil  	} -	fromFederatorChan, ok := fromFederatorChanI.(chan gtsmodel.FromFederator) +	fromFederatorChan, ok := fromFederatorChanI.(chan messages.FromFederator)  	if !ok {  		l.Error("ACCEPT: from federator channel was set on context but couldn't be parsed")  		return nil @@ -99,9 +101,9 @@ func (f *federatingDB) Accept(ctx context.Context, accept vocab.ActivityStreamsA  					return err  				} -				fromFederatorChan <- gtsmodel.FromFederator{ -					APObjectType:     gtsmodel.ActivityStreamsFollow, -					APActivityType:   gtsmodel.ActivityStreamsAccept, +				fromFederatorChan <- messages.FromFederator{ +					APObjectType:     ap.ActivityFollow, +					APActivityType:   ap.ActivityAccept,  					GTSModel:         follow,  					ReceivingAccount: targetAcct,  				} @@ -116,7 +118,7 @@ func (f *federatingDB) Accept(ctx context.Context, accept vocab.ActivityStreamsA  		}  		switch iter.GetType().GetTypeName() {  		// we have the whole object so we can figure out what we're accepting -		case string(gtsmodel.ActivityStreamsFollow): +		case string(ap.ActivityFollow):  			// ACCEPT FOLLOW  			asFollow, ok := iter.GetType().(vocab.ActivityStreamsFollow)  			if !ok { @@ -136,9 +138,9 @@ func (f *federatingDB) Accept(ctx context.Context, accept vocab.ActivityStreamsA  				return err  			} -			fromFederatorChan <- gtsmodel.FromFederator{ -				APObjectType:     gtsmodel.ActivityStreamsFollow, -				APActivityType:   gtsmodel.ActivityStreamsAccept, +			fromFederatorChan <- messages.FromFederator{ +				APObjectType:     ap.ActivityFollow, +				APActivityType:   ap.ActivityAccept,  				GTSModel:         follow,  				ReceivingAccount: targetAcct,  			} diff --git a/internal/federation/federatingdb/announce.go b/internal/federation/federatingdb/announce.go index 5cd34285e..7d7b12cbc 100644 --- a/internal/federation/federatingdb/announce.go +++ b/internal/federation/federatingdb/announce.go @@ -26,7 +26,9 @@ import (  	"github.com/go-fed/activity/streams"  	"github.com/go-fed/activity/streams/vocab"  	"github.com/sirupsen/logrus" +	"github.com/superseriousbusiness/gotosocial/internal/ap"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel" +	"github.com/superseriousbusiness/gotosocial/internal/messages"  	"github.com/superseriousbusiness/gotosocial/internal/util"  ) @@ -65,7 +67,7 @@ func (f *federatingDB) Announce(ctx context.Context, announce vocab.ActivityStre  		l.Error("ANNOUNCE: from federator channel wasn't set on context")  		return nil  	} -	fromFederatorChan, ok := fromFederatorChanI.(chan gtsmodel.FromFederator) +	fromFederatorChan, ok := fromFederatorChanI.(chan messages.FromFederator)  	if !ok {  		l.Error("ANNOUNCE: from federator channel was set on context but couldn't be parsed")  		return nil @@ -82,9 +84,9 @@ func (f *federatingDB) Announce(ctx context.Context, announce vocab.ActivityStre  	}  	// it's a new announce so pass it back to the processor async for dereferencing etc -	fromFederatorChan <- gtsmodel.FromFederator{ -		APObjectType:     gtsmodel.ActivityStreamsAnnounce, -		APActivityType:   gtsmodel.ActivityStreamsCreate, +	fromFederatorChan <- messages.FromFederator{ +		APObjectType:     ap.ActivityAnnounce, +		APActivityType:   ap.ActivityCreate,  		GTSModel:         boost,  		ReceivingAccount: targetAcct,  	} diff --git a/internal/federation/federatingdb/create.go b/internal/federation/federatingdb/create.go index 8ea549c5a..88b0d1e8b 100644 --- a/internal/federation/federatingdb/create.go +++ b/internal/federation/federatingdb/create.go @@ -27,9 +27,11 @@ import (  	"github.com/go-fed/activity/streams"  	"github.com/go-fed/activity/streams/vocab"  	"github.com/sirupsen/logrus" +	"github.com/superseriousbusiness/gotosocial/internal/ap"  	"github.com/superseriousbusiness/gotosocial/internal/db"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel"  	"github.com/superseriousbusiness/gotosocial/internal/id" +	"github.com/superseriousbusiness/gotosocial/internal/messages"  	"github.com/superseriousbusiness/gotosocial/internal/util"  ) @@ -81,14 +83,14 @@ func (f *federatingDB) Create(ctx context.Context, asType vocab.Type) error {  		l.Error("CREATE: from federator channel wasn't set on context")  		return nil  	} -	fromFederatorChan, ok := fromFederatorChanI.(chan gtsmodel.FromFederator) +	fromFederatorChan, ok := fromFederatorChanI.(chan messages.FromFederator)  	if !ok {  		l.Error("CREATE: from federator channel was set on context but couldn't be parsed")  		return nil  	}  	switch asType.GetTypeName() { -	case gtsmodel.ActivityStreamsCreate: +	case ap.ActivityCreate:  		// CREATE SOMETHING  		create, ok := asType.(vocab.ActivityStreamsCreate)  		if !ok { @@ -97,7 +99,7 @@ func (f *federatingDB) Create(ctx context.Context, asType vocab.Type) error {  		object := create.GetActivityStreamsObject()  		for objectIter := object.Begin(); objectIter != object.End(); objectIter = objectIter.Next() {  			switch objectIter.GetType().GetTypeName() { -			case gtsmodel.ActivityStreamsNote: +			case ap.ObjectNote:  				// CREATE A NOTE  				note := objectIter.GetActivityStreamsNote()  				status, err := f.typeConverter.ASStatusToStatus(ctx, note) @@ -122,15 +124,15 @@ func (f *federatingDB) Create(ctx context.Context, asType vocab.Type) error {  					return fmt.Errorf("CREATE: database error inserting status: %s", err)  				} -				fromFederatorChan <- gtsmodel.FromFederator{ -					APObjectType:     gtsmodel.ActivityStreamsNote, -					APActivityType:   gtsmodel.ActivityStreamsCreate, +				fromFederatorChan <- messages.FromFederator{ +					APObjectType:     ap.ObjectNote, +					APActivityType:   ap.ActivityCreate,  					GTSModel:         status,  					ReceivingAccount: targetAcct,  				}  			}  		} -	case gtsmodel.ActivityStreamsFollow: +	case ap.ActivityFollow:  		// FOLLOW SOMETHING  		follow, ok := asType.(vocab.ActivityStreamsFollow)  		if !ok { @@ -152,13 +154,13 @@ func (f *federatingDB) Create(ctx context.Context, asType vocab.Type) error {  			return fmt.Errorf("CREATE: database error inserting follow request: %s", err)  		} -		fromFederatorChan <- gtsmodel.FromFederator{ -			APObjectType:     gtsmodel.ActivityStreamsFollow, -			APActivityType:   gtsmodel.ActivityStreamsCreate, +		fromFederatorChan <- messages.FromFederator{ +			APObjectType:     ap.ActivityFollow, +			APActivityType:   ap.ActivityCreate,  			GTSModel:         followRequest,  			ReceivingAccount: targetAcct,  		} -	case gtsmodel.ActivityStreamsLike: +	case ap.ActivityLike:  		// LIKE SOMETHING  		like, ok := asType.(vocab.ActivityStreamsLike)  		if !ok { @@ -180,13 +182,13 @@ func (f *federatingDB) Create(ctx context.Context, asType vocab.Type) error {  			return fmt.Errorf("CREATE: database error inserting fave: %s", err)  		} -		fromFederatorChan <- gtsmodel.FromFederator{ -			APObjectType:     gtsmodel.ActivityStreamsLike, -			APActivityType:   gtsmodel.ActivityStreamsCreate, +		fromFederatorChan <- messages.FromFederator{ +			APObjectType:     ap.ActivityLike, +			APActivityType:   ap.ActivityCreate,  			GTSModel:         fave,  			ReceivingAccount: targetAcct,  		} -	case gtsmodel.ActivityStreamsBlock: +	case ap.ActivityBlock:  		// BLOCK SOMETHING  		blockable, ok := asType.(vocab.ActivityStreamsBlock)  		if !ok { @@ -208,9 +210,9 @@ func (f *federatingDB) Create(ctx context.Context, asType vocab.Type) error {  			return fmt.Errorf("CREATE: database error inserting block: %s", err)  		} -		fromFederatorChan <- gtsmodel.FromFederator{ -			APObjectType:     gtsmodel.ActivityStreamsBlock, -			APActivityType:   gtsmodel.ActivityStreamsCreate, +		fromFederatorChan <- messages.FromFederator{ +			APObjectType:     ap.ActivityBlock, +			APActivityType:   ap.ActivityCreate,  			GTSModel:         block,  			ReceivingAccount: targetAcct,  		} diff --git a/internal/federation/federatingdb/delete.go b/internal/federation/federatingdb/delete.go index 11b818168..abc3715da 100644 --- a/internal/federation/federatingdb/delete.go +++ b/internal/federation/federatingdb/delete.go @@ -24,7 +24,9 @@ import (  	"net/url"  	"github.com/sirupsen/logrus" +	"github.com/superseriousbusiness/gotosocial/internal/ap"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel" +	"github.com/superseriousbusiness/gotosocial/internal/messages"  	"github.com/superseriousbusiness/gotosocial/internal/util"  ) @@ -61,7 +63,7 @@ func (f *federatingDB) Delete(ctx context.Context, id *url.URL) error {  		l.Error("DELETE: from federator channel wasn't set on context")  		return nil  	} -	fromFederatorChan, ok := fromFederatorChanI.(chan gtsmodel.FromFederator) +	fromFederatorChan, ok := fromFederatorChanI.(chan messages.FromFederator)  	if !ok {  		l.Error("DELETE: from federator channel was set on context but couldn't be parsed")  		return nil @@ -76,9 +78,9 @@ func (f *federatingDB) Delete(ctx context.Context, id *url.URL) error {  		if err := f.db.DeleteByID(ctx, s.ID, >smodel.Status{}); err != nil {  			return fmt.Errorf("DELETE: err deleting status: %s", err)  		} -		fromFederatorChan <- gtsmodel.FromFederator{ -			APObjectType:     gtsmodel.ActivityStreamsNote, -			APActivityType:   gtsmodel.ActivityStreamsDelete, +		fromFederatorChan <- messages.FromFederator{ +			APObjectType:     ap.ObjectNote, +			APActivityType:   ap.ActivityDelete,  			GTSModel:         s,  			ReceivingAccount: targetAcct,  		} @@ -91,9 +93,9 @@ func (f *federatingDB) Delete(ctx context.Context, id *url.URL) error {  		if err := f.db.DeleteByID(ctx, a.ID, >smodel.Account{}); err != nil {  			return fmt.Errorf("DELETE: err deleting account: %s", err)  		} -		fromFederatorChan <- gtsmodel.FromFederator{ -			APObjectType:     gtsmodel.ActivityStreamsProfile, -			APActivityType:   gtsmodel.ActivityStreamsDelete, +		fromFederatorChan <- messages.FromFederator{ +			APObjectType:     ap.ObjectProfile, +			APActivityType:   ap.ActivityDelete,  			GTSModel:         a,  			ReceivingAccount: targetAcct,  		} diff --git a/internal/federation/federatingdb/undo.go b/internal/federation/federatingdb/undo.go index 0fa38114d..7b49815df 100644 --- a/internal/federation/federatingdb/undo.go +++ b/internal/federation/federatingdb/undo.go @@ -27,6 +27,7 @@ import (  	"github.com/go-fed/activity/streams"  	"github.com/go-fed/activity/streams/vocab"  	"github.com/sirupsen/logrus" +	"github.com/superseriousbusiness/gotosocial/internal/ap"  	"github.com/superseriousbusiness/gotosocial/internal/db"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel"  	"github.com/superseriousbusiness/gotosocial/internal/util" @@ -72,7 +73,7 @@ func (f *federatingDB) Undo(ctx context.Context, undo vocab.ActivityStreamsUndo)  			continue  		}  		switch iter.GetType().GetTypeName() { -		case string(gtsmodel.ActivityStreamsFollow): +		case string(ap.ActivityFollow):  			// UNDO FOLLOW  			ASFollow, ok := iter.GetType().(vocab.ActivityStreamsFollow)  			if !ok { @@ -101,11 +102,11 @@ func (f *federatingDB) Undo(ctx context.Context, undo vocab.ActivityStreamsUndo)  			}  			l.Debug("follow undone")  			return nil -		case string(gtsmodel.ActivityStreamsLike): +		case string(ap.ActivityLike):  			// UNDO LIKE -		case string(gtsmodel.ActivityStreamsAnnounce): +		case string(ap.ActivityAnnounce):  			// UNDO BOOST/REBLOG/ANNOUNCE -		case string(gtsmodel.ActivityStreamsBlock): +		case string(ap.ActivityBlock):  			// UNDO BLOCK  			ASBlock, ok := iter.GetType().(vocab.ActivityStreamsBlock)  			if !ok { diff --git a/internal/federation/federatingdb/update.go b/internal/federation/federatingdb/update.go index e9dfe5315..2bcf2533c 100644 --- a/internal/federation/federatingdb/update.go +++ b/internal/federation/federatingdb/update.go @@ -29,6 +29,7 @@ import (  	"github.com/sirupsen/logrus"  	"github.com/superseriousbusiness/gotosocial/internal/ap"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel" +	"github.com/superseriousbusiness/gotosocial/internal/messages"  	"github.com/superseriousbusiness/gotosocial/internal/util"  ) @@ -84,50 +85,50 @@ func (f *federatingDB) Update(ctx context.Context, asType vocab.Type) error {  	if fromFederatorChanI == nil {  		l.Error("UPDATE: from federator channel wasn't set on context")  	} -	fromFederatorChan, ok := fromFederatorChanI.(chan gtsmodel.FromFederator) +	fromFederatorChan, ok := fromFederatorChanI.(chan messages.FromFederator)  	if !ok {  		l.Error("UPDATE: from federator channel was set on context but couldn't be parsed")  	}  	typeName := asType.GetTypeName() -	if typeName == gtsmodel.ActivityStreamsApplication || -		typeName == gtsmodel.ActivityStreamsGroup || -		typeName == gtsmodel.ActivityStreamsOrganization || -		typeName == gtsmodel.ActivityStreamsPerson || -		typeName == gtsmodel.ActivityStreamsService { +	if typeName == ap.ActorApplication || +		typeName == ap.ActorGroup || +		typeName == ap.ActorOrganization || +		typeName == ap.ActorPerson || +		typeName == ap.ActorService {  		// it's an UPDATE to some kind of account  		var accountable ap.Accountable  		switch asType.GetTypeName() { -		case gtsmodel.ActivityStreamsApplication: +		case ap.ActorApplication:  			l.Debug("got update for APPLICATION")  			i, ok := asType.(vocab.ActivityStreamsApplication)  			if !ok {  				return errors.New("UPDATE: could not convert type to application")  			}  			accountable = i -		case gtsmodel.ActivityStreamsGroup: +		case ap.ActorGroup:  			l.Debug("got update for GROUP")  			i, ok := asType.(vocab.ActivityStreamsGroup)  			if !ok {  				return errors.New("UPDATE: could not convert type to group")  			}  			accountable = i -		case gtsmodel.ActivityStreamsOrganization: +		case ap.ActorOrganization:  			l.Debug("got update for ORGANIZATION")  			i, ok := asType.(vocab.ActivityStreamsOrganization)  			if !ok {  				return errors.New("UPDATE: could not convert type to organization")  			}  			accountable = i -		case gtsmodel.ActivityStreamsPerson: +		case ap.ActorPerson:  			l.Debug("got update for PERSON")  			i, ok := asType.(vocab.ActivityStreamsPerson)  			if !ok {  				return errors.New("UPDATE: could not convert type to person")  			}  			accountable = i -		case gtsmodel.ActivityStreamsService: +		case ap.ActorService:  			l.Debug("got update for SERVICE")  			i, ok := asType.(vocab.ActivityStreamsService)  			if !ok { @@ -157,9 +158,9 @@ func (f *federatingDB) Update(ctx context.Context, asType vocab.Type) error {  			return fmt.Errorf("UPDATE: database error inserting updated account: %s", err)  		} -		fromFederatorChan <- gtsmodel.FromFederator{ -			APObjectType:     gtsmodel.ActivityStreamsProfile, -			APActivityType:   gtsmodel.ActivityStreamsUpdate, +		fromFederatorChan <- messages.FromFederator{ +			APObjectType:     ap.ObjectProfile, +			APActivityType:   ap.ActivityUpdate,  			GTSModel:         updatedAcct,  			ReceivingAccount: targetAcct,  		} diff --git a/internal/federation/federatingdb/util.go b/internal/federation/federatingdb/util.go index b5befc613..d8c7d8e8a 100644 --- a/internal/federation/federatingdb/util.go +++ b/internal/federation/federatingdb/util.go @@ -28,6 +28,7 @@ import (  	"github.com/go-fed/activity/streams"  	"github.com/go-fed/activity/streams/vocab"  	"github.com/sirupsen/logrus" +	"github.com/superseriousbusiness/gotosocial/internal/ap"  	"github.com/superseriousbusiness/gotosocial/internal/db"  	"github.com/superseriousbusiness/gotosocial/internal/gtsmodel"  	"github.com/superseriousbusiness/gotosocial/internal/id" @@ -78,7 +79,7 @@ func (f *federatingDB) NewID(ctx context.Context, t vocab.Type) (idURL *url.URL,  	l.Debugf("received NEWID request for asType %s", string(b))  	switch t.GetTypeName() { -	case gtsmodel.ActivityStreamsFollow: +	case ap.ActivityFollow:  		// FOLLOW  		// ID might already be set on a follow we've created, so check it here and return it if it is  		follow, ok := t.(vocab.ActivityStreamsFollow) @@ -108,7 +109,7 @@ func (f *federatingDB) NewID(ctx context.Context, t vocab.Type) (idURL *url.URL,  				}  			}  		} -	case gtsmodel.ActivityStreamsNote: +	case ap.ObjectNote:  		// NOTE aka STATUS  		// ID might already be set on a note we've created, so check it here and return it if it is  		note, ok := t.(vocab.ActivityStreamsNote) @@ -121,7 +122,7 @@ func (f *federatingDB) NewID(ctx context.Context, t vocab.Type) (idURL *url.URL,  				return idProp.GetIRI(), nil  			}  		} -	case gtsmodel.ActivityStreamsLike: +	case ap.ActivityLike:  		// LIKE aka FAVE  		// ID might already be set on a fave we've created, so check it here and return it if it is  		fave, ok := t.(vocab.ActivityStreamsLike) @@ -134,7 +135,7 @@ func (f *federatingDB) NewID(ctx context.Context, t vocab.Type) (idURL *url.URL,  				return idProp.GetIRI(), nil  			}  		} -	case gtsmodel.ActivityStreamsAnnounce: +	case ap.ActivityAnnounce:  		// ANNOUNCE aka BOOST  		// ID might already be set on an announce we've created, so check it here and return it if it is  		announce, ok := t.(vocab.ActivityStreamsAnnounce) @@ -147,7 +148,7 @@ func (f *federatingDB) NewID(ctx context.Context, t vocab.Type) (idURL *url.URL,  				return idProp.GetIRI(), nil  			}  		} -	case gtsmodel.ActivityStreamsUpdate: +	case ap.ActivityUpdate:  		// UPDATE  		// ID might already be set on an update we've created, so check it here and return it if it is  		update, ok := t.(vocab.ActivityStreamsUpdate) @@ -160,7 +161,7 @@ func (f *federatingDB) NewID(ctx context.Context, t vocab.Type) (idURL *url.URL,  				return idProp.GetIRI(), nil  			}  		} -	case gtsmodel.ActivityStreamsBlock: +	case ap.ActivityBlock:  		// BLOCK  		// ID might already be set on a block we've created, so check it here and return it if it is  		block, ok := t.(vocab.ActivityStreamsBlock) @@ -173,7 +174,7 @@ func (f *federatingDB) NewID(ctx context.Context, t vocab.Type) (idURL *url.URL,  				return idProp.GetIRI(), nil  			}  		} -	case gtsmodel.ActivityStreamsUndo: +	case ap.ActivityUndo:  		// UNDO  		// ID might already be set on an undo we've created, so check it here and return it if it is  		undo, ok := t.(vocab.ActivityStreamsUndo) | 
