mirror of
https://github.com/superseriousbusiness/gotosocial.git
synced 2024-12-27 09:18:52 +01:00
87cff71af9
* persist queued worker tasks to database on shutdown, fill worker queues from database on startup
* ensure the tasks are sorted by creation time before pushing them
* add migration to insert WorkerTask{} into database, add test for worker task persistence
* add test for recovering worker queues from database
* quick tweak
* whoops we ended up with double cleaner job scheduling
* insert each task separately, because bun is throwing some reflection error??
* add specific checking of cancelled worker contexts
* add http request signing to deliveries recovered from database
* add test for outgoing public key ID being correctly set on delivery
* replace select with Queue.PopCtx()
* get rid of loop now we don't use it
* remove field now we don't use it
* ensure that signing func is set
* header values weren't being copied over 🤦
* use ptr for httpclient.Request in delivery
* move worker queue filling to later in server init process
* fix rebase issues
* make logging less shouty
* use slices.Delete() instead of copying / reslicing
* have database return tasks in ascending order instead of sorting them
* add a 1 minute timeout to persisting worker queues
422 lines
11 KiB
Go
422 lines
11 KiB
Go
// GoToSocial
|
|
// Copyright (C) GoToSocial Authors admin@gotosocial.org
|
|
// SPDX-License-Identifier: AGPL-3.0-or-later
|
|
//
|
|
// This program is free software: you can redistribute it and/or modify
|
|
// it under the terms of the GNU Affero General Public License as published by
|
|
// the Free Software Foundation, either version 3 of the License, or
|
|
// (at your option) any later version.
|
|
//
|
|
// This program is distributed in the hope that it will be useful,
|
|
// but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
// GNU Affero General Public License for more details.
|
|
//
|
|
// You should have received a copy of the GNU Affero General Public License
|
|
// along with this program. If not, see <http://www.gnu.org/licenses/>.
|
|
|
|
package admin_test
|
|
|
|
import (
|
|
"bytes"
|
|
"context"
|
|
"encoding/json"
|
|
"fmt"
|
|
"io"
|
|
"net/http"
|
|
"net/url"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/suite"
|
|
"github.com/superseriousbusiness/gotosocial/internal/ap"
|
|
"github.com/superseriousbusiness/gotosocial/internal/gtscontext"
|
|
"github.com/superseriousbusiness/gotosocial/internal/gtsmodel"
|
|
"github.com/superseriousbusiness/gotosocial/internal/httpclient"
|
|
"github.com/superseriousbusiness/gotosocial/internal/messages"
|
|
"github.com/superseriousbusiness/gotosocial/internal/transport/delivery"
|
|
"github.com/superseriousbusiness/gotosocial/testrig"
|
|
)
|
|
|
|
var (
|
|
// TODO: move these test values into
|
|
// the testrig test models area. They'll
|
|
// need to be as both WorkerTask and as
|
|
// the raw types themselves.
|
|
|
|
testDeliveries = []*delivery.Delivery{
|
|
{
|
|
ObjectID: "https://google.com/users/bigboy/follow/1",
|
|
TargetID: "https://askjeeves.com/users/smallboy",
|
|
Request: toRequest("POST", "https://askjeeves.com/users/smallboy/inbox", []byte("data!"), http.Header{"Host": {"https://askjeeves.com"}}),
|
|
},
|
|
{
|
|
Request: toRequest("GET", "https://google.com", []byte("uwu im just a wittle seawch engwin"), http.Header{"Host": {"https://google.com"}}),
|
|
},
|
|
}
|
|
|
|
testFederatorMsgs = []*messages.FromFediAPI{
|
|
{
|
|
APObjectType: ap.ObjectNote,
|
|
APActivityType: ap.ActivityCreate,
|
|
TargetURI: "https://gotosocial.org",
|
|
Requesting: >smodel.Account{ID: "654321"},
|
|
Receiving: >smodel.Account{ID: "123456"},
|
|
},
|
|
{
|
|
APObjectType: ap.ObjectProfile,
|
|
APActivityType: ap.ActivityUpdate,
|
|
TargetURI: "https://uk-queen-is-dead.org",
|
|
Requesting: >smodel.Account{ID: "123456"},
|
|
Receiving: >smodel.Account{ID: "654321"},
|
|
},
|
|
}
|
|
|
|
testClientMsgs = []*messages.FromClientAPI{
|
|
{
|
|
APObjectType: ap.ObjectNote,
|
|
APActivityType: ap.ActivityCreate,
|
|
TargetURI: "https://gotosocial.org",
|
|
Origin: >smodel.Account{ID: "654321"},
|
|
Target: >smodel.Account{ID: "123456"},
|
|
},
|
|
{
|
|
APObjectType: ap.ObjectProfile,
|
|
APActivityType: ap.ActivityUpdate,
|
|
TargetURI: "https://uk-queen-is-dead.org",
|
|
Origin: >smodel.Account{ID: "123456"},
|
|
Target: >smodel.Account{ID: "654321"},
|
|
},
|
|
}
|
|
)
|
|
|
|
type WorkerTaskTestSuite struct {
|
|
AdminStandardTestSuite
|
|
}
|
|
|
|
func (suite *WorkerTaskTestSuite) TestFillWorkerQueues() {
|
|
ctx, cncl := context.WithCancel(context.Background())
|
|
defer cncl()
|
|
|
|
var tasks []*gtsmodel.WorkerTask
|
|
|
|
for _, dlv := range testDeliveries {
|
|
// Serialize all test deliveries.
|
|
data, err := dlv.Serialize()
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
|
|
// Append each serialized delivery to tasks.
|
|
tasks = append(tasks, >smodel.WorkerTask{
|
|
WorkerType: gtsmodel.DeliveryWorker,
|
|
TaskData: data,
|
|
})
|
|
}
|
|
|
|
for _, msg := range testFederatorMsgs {
|
|
// Serialize all test messages.
|
|
data, err := msg.Serialize()
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
|
|
if msg.Receiving != nil {
|
|
// Quick hack to bypass database errors for non-existing
|
|
// accounts, instead we just insert this into cache ;).
|
|
suite.state.Caches.DB.Account.Put(msg.Receiving)
|
|
suite.state.Caches.DB.AccountSettings.Put(>smodel.AccountSettings{
|
|
AccountID: msg.Receiving.ID,
|
|
})
|
|
}
|
|
|
|
if msg.Requesting != nil {
|
|
// Quick hack to bypass database errors for non-existing
|
|
// accounts, instead we just insert this into cache ;).
|
|
suite.state.Caches.DB.Account.Put(msg.Requesting)
|
|
suite.state.Caches.DB.AccountSettings.Put(>smodel.AccountSettings{
|
|
AccountID: msg.Requesting.ID,
|
|
})
|
|
}
|
|
|
|
// Append each serialized message to tasks.
|
|
tasks = append(tasks, >smodel.WorkerTask{
|
|
WorkerType: gtsmodel.FederatorWorker,
|
|
TaskData: data,
|
|
})
|
|
}
|
|
|
|
for _, msg := range testClientMsgs {
|
|
// Serialize all test messages.
|
|
data, err := msg.Serialize()
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
|
|
if msg.Origin != nil {
|
|
// Quick hack to bypass database errors for non-existing
|
|
// accounts, instead we just insert this into cache ;).
|
|
suite.state.Caches.DB.Account.Put(msg.Origin)
|
|
suite.state.Caches.DB.AccountSettings.Put(>smodel.AccountSettings{
|
|
AccountID: msg.Origin.ID,
|
|
})
|
|
}
|
|
|
|
if msg.Target != nil {
|
|
// Quick hack to bypass database errors for non-existing
|
|
// accounts, instead we just insert this into cache ;).
|
|
suite.state.Caches.DB.Account.Put(msg.Target)
|
|
suite.state.Caches.DB.AccountSettings.Put(>smodel.AccountSettings{
|
|
AccountID: msg.Target.ID,
|
|
})
|
|
}
|
|
|
|
// Append each serialized message to tasks.
|
|
tasks = append(tasks, >smodel.WorkerTask{
|
|
WorkerType: gtsmodel.ClientWorker,
|
|
TaskData: data,
|
|
})
|
|
}
|
|
|
|
// Persist all test worker tasks to the database.
|
|
err := suite.state.DB.PutWorkerTasks(ctx, tasks)
|
|
suite.NoError(err)
|
|
|
|
// Fill the worker queues from persisted task data.
|
|
err = suite.adminProcessor.FillWorkerQueues(ctx)
|
|
suite.NoError(err)
|
|
|
|
var (
|
|
// Recovered
|
|
// task counts.
|
|
ndelivery int
|
|
nfederator int
|
|
nclient int
|
|
)
|
|
|
|
// Fetch current gotosocial instance account, for later checks.
|
|
instanceAcc, err := suite.state.DB.GetInstanceAccount(ctx, "")
|
|
suite.NoError(err)
|
|
|
|
for {
|
|
// Pop all queued delivery tasks from worker queue.
|
|
dlv, ok := suite.state.Workers.Delivery.Queue.Pop()
|
|
if !ok {
|
|
break
|
|
}
|
|
|
|
// Incr count.
|
|
ndelivery++
|
|
|
|
// Check that we have this message in slice.
|
|
err = containsSerializable(testDeliveries, dlv)
|
|
suite.NoError(err)
|
|
|
|
// Check that delivery request context has instance account pubkey.
|
|
pubKeyID := gtscontext.OutgoingPublicKeyID(dlv.Request.Context())
|
|
suite.Equal(instanceAcc.PublicKeyURI, pubKeyID)
|
|
signfn := gtscontext.HTTPClientSignFunc(dlv.Request.Context())
|
|
suite.NotNil(signfn)
|
|
}
|
|
|
|
for {
|
|
// Pop all queued federator messages from worker queue.
|
|
msg, ok := suite.state.Workers.Federator.Queue.Pop()
|
|
if !ok {
|
|
break
|
|
}
|
|
|
|
// Incr count.
|
|
nfederator++
|
|
|
|
// Check that we have this message in slice.
|
|
err = containsSerializable(testFederatorMsgs, msg)
|
|
suite.NoError(err)
|
|
}
|
|
|
|
for {
|
|
// Pop all queued client messages from worker queue.
|
|
msg, ok := suite.state.Workers.Client.Queue.Pop()
|
|
if !ok {
|
|
break
|
|
}
|
|
|
|
// Incr count.
|
|
nclient++
|
|
|
|
// Check that we have this message in slice.
|
|
err = containsSerializable(testClientMsgs, msg)
|
|
suite.NoError(err)
|
|
}
|
|
|
|
// Ensure recovered task counts as expected.
|
|
suite.Equal(len(testDeliveries), ndelivery)
|
|
suite.Equal(len(testFederatorMsgs), nfederator)
|
|
suite.Equal(len(testClientMsgs), nclient)
|
|
}
|
|
|
|
func (suite *WorkerTaskTestSuite) TestPersistWorkerQueues() {
|
|
ctx, cncl := context.WithCancel(context.Background())
|
|
defer cncl()
|
|
|
|
// Push all test worker tasks to their respective queues.
|
|
suite.state.Workers.Delivery.Queue.Push(testDeliveries...)
|
|
suite.state.Workers.Federator.Queue.Push(testFederatorMsgs...)
|
|
suite.state.Workers.Client.Queue.Push(testClientMsgs...)
|
|
|
|
// Persist the worker queued tasks to database.
|
|
err := suite.adminProcessor.PersistWorkerQueues(ctx)
|
|
suite.NoError(err)
|
|
|
|
// Fetch all the persisted tasks from database.
|
|
tasks, err := suite.state.DB.GetWorkerTasks(ctx)
|
|
suite.NoError(err)
|
|
|
|
var (
|
|
// Persisted
|
|
// task counts.
|
|
ndelivery int
|
|
nfederator int
|
|
nclient int
|
|
)
|
|
|
|
// Check persisted task data.
|
|
for _, task := range tasks {
|
|
switch task.WorkerType {
|
|
case gtsmodel.DeliveryWorker:
|
|
var dlv delivery.Delivery
|
|
|
|
// Incr count.
|
|
ndelivery++
|
|
|
|
// Deserialize the persisted task data.
|
|
err := dlv.Deserialize(task.TaskData)
|
|
suite.NoError(err)
|
|
|
|
// Check that we have this delivery in slice.
|
|
err = containsSerializable(testDeliveries, &dlv)
|
|
suite.NoError(err)
|
|
|
|
case gtsmodel.FederatorWorker:
|
|
var msg messages.FromFediAPI
|
|
|
|
// Incr count.
|
|
nfederator++
|
|
|
|
// Deserialize the persisted task data.
|
|
err := msg.Deserialize(task.TaskData)
|
|
suite.NoError(err)
|
|
|
|
// Check that we have this message in slice.
|
|
err = containsSerializable(testFederatorMsgs, &msg)
|
|
suite.NoError(err)
|
|
|
|
case gtsmodel.ClientWorker:
|
|
var msg messages.FromClientAPI
|
|
|
|
// Incr count.
|
|
nclient++
|
|
|
|
// Deserialize the persisted task data.
|
|
err := msg.Deserialize(task.TaskData)
|
|
suite.NoError(err)
|
|
|
|
// Check that we have this message in slice.
|
|
err = containsSerializable(testClientMsgs, &msg)
|
|
suite.NoError(err)
|
|
|
|
default:
|
|
suite.T().Errorf("unexpected worker type: %d", task.WorkerType)
|
|
}
|
|
}
|
|
|
|
// Ensure persisted task counts as expected.
|
|
suite.Equal(len(testDeliveries), ndelivery)
|
|
suite.Equal(len(testFederatorMsgs), nfederator)
|
|
suite.Equal(len(testClientMsgs), nclient)
|
|
}
|
|
|
|
func (suite *WorkerTaskTestSuite) SetupTest() {
|
|
suite.AdminStandardTestSuite.SetupTest()
|
|
// we don't want workers running
|
|
testrig.StopWorkers(&suite.state)
|
|
}
|
|
|
|
func TestWorkerTaskTestSuite(t *testing.T) {
|
|
suite.Run(t, new(WorkerTaskTestSuite))
|
|
}
|
|
|
|
// containsSerializeable returns whether slice of serializables contains given serializable entry.
|
|
func containsSerializable[T interface{ Serialize() ([]byte, error) }](expect []T, have T) error {
|
|
// Serialize wanted value.
|
|
bh, err := have.Serialize()
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
|
|
var strings []string
|
|
|
|
for _, t := range expect {
|
|
// Serialize expected value.
|
|
be, err := t.Serialize()
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
|
|
// Alloc as string.
|
|
se := string(be)
|
|
|
|
if se == string(bh) {
|
|
// We have this entry!
|
|
return nil
|
|
}
|
|
|
|
// Add to serialized strings.
|
|
strings = append(strings, se)
|
|
}
|
|
|
|
return fmt.Errorf("could not find %s in %s", string(bh), strings)
|
|
}
|
|
|
|
// urlStr simply returns u.String() or "" if nil.
|
|
func urlStr(u *url.URL) string {
|
|
if u == nil {
|
|
return ""
|
|
}
|
|
return u.String()
|
|
}
|
|
|
|
// accountID simply returns account.ID or "" if nil.
|
|
func accountID(account *gtsmodel.Account) string {
|
|
if account == nil {
|
|
return ""
|
|
}
|
|
return account.ID
|
|
}
|
|
|
|
// toRequest creates httpclient.Request from HTTP method, URL and body data.
|
|
func toRequest(method string, url string, body []byte, hdr http.Header) *httpclient.Request {
|
|
var rbody io.Reader
|
|
if body != nil {
|
|
rbody = bytes.NewReader(body)
|
|
}
|
|
req, err := http.NewRequest(method, url, rbody)
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
for key, values := range hdr {
|
|
for _, value := range values {
|
|
req.Header.Add(key, value)
|
|
}
|
|
}
|
|
return httpclient.WrapRequest(req)
|
|
}
|
|
|
|
// toJSON marshals input type as JSON data.
|
|
func toJSON(a any) []byte {
|
|
b, err := json.Marshal(a)
|
|
if err != nil {
|
|
panic(err)
|
|
}
|
|
return b
|
|
}
|