aboutsummaryrefslogtreecommitdiff
path: root/federationapi
diff options
context:
space:
mode:
authorTill <2353100+S7evinK@users.noreply.github.com>2023-03-22 09:21:32 +0100
committerGitHub <noreply@github.com>2023-03-22 09:21:32 +0100
commit5e85a00cb36c3d343cd5b6f6a18435989724a135 (patch)
treeb22c34dd0a6cdc04025b90086843f9084a876412 /federationapi
parentec6879e5ae2919c903707475ce8d72244b2a6847 (diff)
Remove `BaseDendrite` (#3023)
Removes `BaseDendrite` to, hopefully, make testing and composing of components easier in the future.
Diffstat (limited to 'federationapi')
-rw-r--r--federationapi/federationapi.go51
-rw-r--r--federationapi/federationapi_keys_test.go11
-rw-r--r--federationapi/federationapi_test.go75
-rw-r--r--federationapi/queue/queue_test.go14
-rw-r--r--federationapi/routing/profile_test.go18
-rw-r--r--federationapi/routing/query_test.go18
-rw-r--r--federationapi/routing/routing.go16
-rw-r--r--federationapi/routing/send_test.go18
-rw-r--r--federationapi/storage/storage_test.go2
9 files changed, 129 insertions, 94 deletions
diff --git a/federationapi/federationapi.go b/federationapi/federationapi.go
index 8a4237ba..3b5394a1 100644
--- a/federationapi/federationapi.go
+++ b/federationapi/federationapi.go
@@ -17,6 +17,10 @@ package federationapi
import (
"time"
+ "github.com/matrix-org/dendrite/internal/httputil"
+ "github.com/matrix-org/dendrite/internal/sqlutil"
+ "github.com/matrix-org/dendrite/setup/config"
+ "github.com/matrix-org/dendrite/setup/process"
"github.com/sirupsen/logrus"
"github.com/matrix-org/dendrite/federationapi/api"
@@ -29,7 +33,6 @@ import (
"github.com/matrix-org/dendrite/federationapi/storage"
"github.com/matrix-org/dendrite/internal/caching"
roomserverAPI "github.com/matrix-org/dendrite/roomserver/api"
- "github.com/matrix-org/dendrite/setup/base"
"github.com/matrix-org/dendrite/setup/jetstream"
userapi "github.com/matrix-org/dendrite/userapi/api"
@@ -40,17 +43,21 @@ import (
// AddPublicRoutes sets up and registers HTTP handlers on the base API muxes for the FederationAPI component.
func AddPublicRoutes(
- base *base.BaseDendrite,
+ processContext *process.ProcessContext,
+ routers httputil.Routers,
+ dendriteConfig *config.Dendrite,
+ natsInstance *jetstream.NATSInstance,
userAPI userapi.FederationUserAPI,
federation *gomatrixserverlib.FederationClient,
keyRing gomatrixserverlib.JSONVerifier,
rsAPI roomserverAPI.FederationRoomserverAPI,
fedAPI federationAPI.FederationInternalAPI,
servers federationAPI.ServersInRoomProvider,
+ enableMetrics bool,
) {
- cfg := &base.Cfg.FederationAPI
- mscCfg := &base.Cfg.MSCs
- js, _ := base.NATS.Prepare(base.ProcessContext, &cfg.Matrix.JetStream)
+ cfg := &dendriteConfig.FederationAPI
+ mscCfg := &dendriteConfig.MSCs
+ js, _ := natsInstance.Prepare(processContext, &cfg.Matrix.JetStream)
producer := &producers.SyncAPIProducer{
JetStream: js,
TopicReceiptEvent: cfg.Matrix.JetStream.Prefixed(jetstream.OutputReceiptEvent),
@@ -75,26 +82,30 @@ func AddPublicRoutes(
}
routing.Setup(
- base,
+ routers,
+ dendriteConfig,
rsAPI, f, keyRing,
federation, userAPI, mscCfg,
- servers, producer,
+ servers, producer, enableMetrics,
)
}
// NewInternalAPI returns a concerete implementation of the internal API. Callers
// can call functions directly on the returned API or via an HTTP interface using AddInternalRoutes.
func NewInternalAPI(
- base *base.BaseDendrite,
+ processContext *process.ProcessContext,
+ dendriteCfg *config.Dendrite,
+ cm sqlutil.Connections,
+ natsInstance *jetstream.NATSInstance,
federation api.FederationClient,
rsAPI roomserverAPI.FederationRoomserverAPI,
caches *caching.Caches,
keyRing *gomatrixserverlib.KeyRing,
resetBlacklist bool,
) api.FederationInternalAPI {
- cfg := &base.Cfg.FederationAPI
+ cfg := &dendriteCfg.FederationAPI
- federationDB, err := storage.NewDatabase(base.ProcessContext.Context(), base.ConnectionManager, &cfg.Database, caches, base.Cfg.Global.IsLocalServerName)
+ federationDB, err := storage.NewDatabase(processContext.Context(), cm, &cfg.Database, caches, dendriteCfg.Global.IsLocalServerName)
if err != nil {
logrus.WithError(err).Panic("failed to connect to federation sender db")
}
@@ -108,51 +119,51 @@ func NewInternalAPI(
cfg.FederationMaxRetries+1,
cfg.P2PFederationRetriesUntilAssumedOffline+1)
- js, nats := base.NATS.Prepare(base.ProcessContext, &cfg.Matrix.JetStream)
+ js, nats := natsInstance.Prepare(processContext, &cfg.Matrix.JetStream)
- signingInfo := base.Cfg.Global.SigningIdentities()
+ signingInfo := dendriteCfg.Global.SigningIdentities()
queues := queue.NewOutgoingQueues(
- federationDB, base.ProcessContext,
+ federationDB, processContext,
cfg.Matrix.DisableFederation,
cfg.Matrix.ServerName, federation, rsAPI, &stats,
signingInfo,
)
rsConsumer := consumers.NewOutputRoomEventConsumer(
- base.ProcessContext, cfg, js, nats, queues,
+ processContext, cfg, js, nats, queues,
federationDB, rsAPI,
)
if err = rsConsumer.Start(); err != nil {
logrus.WithError(err).Panic("failed to start room server consumer")
}
tsConsumer := consumers.NewOutputSendToDeviceConsumer(
- base.ProcessContext, cfg, js, queues, federationDB,
+ processContext, cfg, js, queues, federationDB,
)
if err = tsConsumer.Start(); err != nil {
logrus.WithError(err).Panic("failed to start send-to-device consumer")
}
receiptConsumer := consumers.NewOutputReceiptConsumer(
- base.ProcessContext, cfg, js, queues, federationDB,
+ processContext, cfg, js, queues, federationDB,
)
if err = receiptConsumer.Start(); err != nil {
logrus.WithError(err).Panic("failed to start receipt consumer")
}
typingConsumer := consumers.NewOutputTypingConsumer(
- base.ProcessContext, cfg, js, queues, federationDB,
+ processContext, cfg, js, queues, federationDB,
)
if err = typingConsumer.Start(); err != nil {
logrus.WithError(err).Panic("failed to start typing consumer")
}
keyConsumer := consumers.NewKeyChangeConsumer(
- base.ProcessContext, &base.Cfg.KeyServer, js, queues, federationDB, rsAPI,
+ processContext, &dendriteCfg.KeyServer, js, queues, federationDB, rsAPI,
)
if err = keyConsumer.Start(); err != nil {
logrus.WithError(err).Panic("failed to start key server consumer")
}
presenceConsumer := consumers.NewOutputPresenceConsumer(
- base.ProcessContext, cfg, js, queues, federationDB, rsAPI,
+ processContext, cfg, js, queues, federationDB, rsAPI,
)
if err = presenceConsumer.Start(); err != nil {
logrus.WithError(err).Panic("failed to start presence consumer")
@@ -161,7 +172,7 @@ func NewInternalAPI(
var cleanExpiredEDUs func()
cleanExpiredEDUs = func() {
logrus.Infof("Cleaning expired EDUs")
- if err := federationDB.DeleteExpiredEDUs(base.Context()); err != nil {
+ if err := federationDB.DeleteExpiredEDUs(processContext.Context()); err != nil {
logrus.WithError(err).Error("Failed to clean expired EDUs")
}
time.AfterFunc(time.Hour, cleanExpiredEDUs)
diff --git a/federationapi/federationapi_keys_test.go b/federationapi/federationapi_keys_test.go
index bb6ee893..425cdbb9 100644
--- a/federationapi/federationapi_keys_test.go
+++ b/federationapi/federationapi_keys_test.go
@@ -12,12 +12,14 @@ import (
"testing"
"time"
+ "github.com/matrix-org/dendrite/internal/sqlutil"
+ "github.com/matrix-org/dendrite/setup/jetstream"
+ "github.com/matrix-org/dendrite/setup/process"
"github.com/matrix-org/gomatrixserverlib"
"github.com/matrix-org/dendrite/federationapi/api"
"github.com/matrix-org/dendrite/federationapi/routing"
"github.com/matrix-org/dendrite/internal/caching"
- "github.com/matrix-org/dendrite/setup/base"
"github.com/matrix-org/dendrite/setup/config"
)
@@ -65,7 +67,7 @@ func TestMain(m *testing.M) {
// Create a new cache but don't enable prometheus!
s.cache = caching.NewRistrettoCache(8*1024*1024, time.Hour, false)
-
+ natsInstance := jetstream.NATSInstance{}
// Create a temporary directory for JetStream.
d, err := os.MkdirTemp("./", "jetstream*")
if err != nil {
@@ -109,8 +111,9 @@ func TestMain(m *testing.M) {
)
// Finally, build the server key APIs.
- sbase := base.NewBaseDendrite(cfg, base.DisableMetrics)
- s.api = NewInternalAPI(sbase, s.fedclient, nil, s.cache, nil, true)
+ processCtx := process.NewProcessContext()
+ cm := sqlutil.NewConnectionManager(processCtx, cfg.Global.DatabaseOptions)
+ s.api = NewInternalAPI(processCtx, cfg, cm, &natsInstance, s.fedclient, nil, s.cache, nil, true)
}
// Now that we have built our server key APIs, start the
diff --git a/federationapi/federationapi_test.go b/federationapi/federationapi_test.go
index 8aea96a7..d39f4051 100644
--- a/federationapi/federationapi_test.go
+++ b/federationapi/federationapi_test.go
@@ -11,6 +11,8 @@ import (
"time"
"github.com/matrix-org/dendrite/internal/caching"
+ "github.com/matrix-org/dendrite/internal/httputil"
+ "github.com/matrix-org/dendrite/internal/sqlutil"
"github.com/matrix-org/gomatrix"
"github.com/matrix-org/gomatrixserverlib"
"github.com/nats-io/nats.go"
@@ -19,8 +21,6 @@ import (
"github.com/matrix-org/dendrite/federationapi/api"
"github.com/matrix-org/dendrite/federationapi/internal"
rsapi "github.com/matrix-org/dendrite/roomserver/api"
- "github.com/matrix-org/dendrite/setup/base"
- "github.com/matrix-org/dendrite/setup/config"
"github.com/matrix-org/dendrite/setup/jetstream"
"github.com/matrix-org/dendrite/test"
"github.com/matrix-org/dendrite/test/testrig"
@@ -163,22 +163,24 @@ func TestFederationAPIJoinThenKeyUpdate(t *testing.T) {
}
func testFederationAPIJoinThenKeyUpdate(t *testing.T, dbType test.DBType) {
- base, close := testrig.CreateBaseDendrite(t, dbType)
- caches := caching.NewRistrettoCache(base.Cfg.Global.Cache.EstimatedMaxSize, base.Cfg.Global.Cache.MaxAge, caching.DisableMetrics)
- base.Cfg.FederationAPI.PreferDirectFetch = true
- base.Cfg.FederationAPI.KeyPerspectives = nil
+ cfg, processCtx, close := testrig.CreateConfig(t, dbType)
+ cm := sqlutil.NewConnectionManager(processCtx, cfg.Global.DatabaseOptions)
+ caches := caching.NewRistrettoCache(128*1024*1024, time.Hour, caching.DisableMetrics)
+ natsInstance := jetstream.NATSInstance{}
+ cfg.FederationAPI.PreferDirectFetch = true
+ cfg.FederationAPI.KeyPerspectives = nil
defer close()
- jsctx, _ := base.NATS.Prepare(base.ProcessContext, &base.Cfg.Global.JetStream)
- defer jetstream.DeleteAllStreams(jsctx, &base.Cfg.Global.JetStream)
+ jsctx, _ := natsInstance.Prepare(processCtx, &cfg.Global.JetStream)
+ defer jetstream.DeleteAllStreams(jsctx, &cfg.Global.JetStream)
serverA := gomatrixserverlib.ServerName("server.a")
serverAKeyID := gomatrixserverlib.KeyID("ed25519:servera")
serverAPrivKey := test.PrivateKeyA
creator := test.NewUser(t, test.WithSigningServer(serverA, serverAKeyID, serverAPrivKey))
- myServer := base.Cfg.Global.ServerName
- myServerKeyID := base.Cfg.Global.KeyID
- myServerPrivKey := base.Cfg.Global.PrivateKey
+ myServer := cfg.Global.ServerName
+ myServerKeyID := cfg.Global.KeyID
+ myServerPrivKey := cfg.Global.PrivateKey
joiningUser := test.NewUser(t, test.WithSigningServer(myServer, myServerKeyID, myServerPrivKey))
fmt.Printf("creator: %v joining user: %v\n", creator.ID, joiningUser.ID)
room := test.NewRoom(t, creator)
@@ -214,7 +216,7 @@ func testFederationAPIJoinThenKeyUpdate(t *testing.T, dbType test.DBType) {
},
},
}
- fsapi := federationapi.NewInternalAPI(base, fc, rsapi, caches, nil, false)
+ fsapi := federationapi.NewInternalAPI(processCtx, cfg, cm, &natsInstance, fc, rsapi, caches, nil, false)
var resp api.PerformJoinResponse
fsapi.PerformJoin(context.Background(), &api.PerformJoinRequest{
@@ -247,7 +249,7 @@ func testFederationAPIJoinThenKeyUpdate(t *testing.T, dbType test.DBType) {
}
msg := &nats.Msg{
- Subject: base.Cfg.Global.JetStream.Prefixed(jetstream.OutputKeyChangeEvent),
+ Subject: cfg.Global.JetStream.Prefixed(jetstream.OutputKeyChangeEvent),
Header: nats.Header{},
Data: b,
}
@@ -265,30 +267,6 @@ func testFederationAPIJoinThenKeyUpdate(t *testing.T, dbType test.DBType) {
// Tests that event IDs with '/' in them (escaped as %2F) are correctly passed to the right handler and don't 404.
// Relevant for v3 rooms and a cause of flakey sytests as the IDs are randomly generated.
func TestRoomsV3URLEscapeDoNot404(t *testing.T) {
- _, privKey, _ := ed25519.GenerateKey(nil)
- cfg := &config.Dendrite{}
- cfg.Defaults(config.DefaultOpts{
- Generate: true,
- SingleDatabase: false,
- })
- cfg.Global.KeyID = gomatrixserverlib.KeyID("ed25519:auto")
- cfg.Global.ServerName = gomatrixserverlib.ServerName("localhost")
- cfg.Global.PrivateKey = privKey
- cfg.Global.JetStream.InMemory = true
- b := base.NewBaseDendrite(cfg, base.DisableMetrics)
- keyRing := &test.NopJSONVerifier{}
- // TODO: This is pretty fragile, as if anything calls anything on these nils this test will break.
- // Unfortunately, it makes little sense to instantiate these dependencies when we just want to test routing.
- federationapi.AddPublicRoutes(b, nil, nil, keyRing, nil, &internal.FederationInternalAPI{}, nil)
- baseURL, cancel := test.ListenAndServe(t, b.Routers.Federation, true)
- defer cancel()
- serverName := gomatrixserverlib.ServerName(strings.TrimPrefix(baseURL, "https://"))
-
- fedCli := gomatrixserverlib.NewFederationClient(
- cfg.Global.SigningIdentities(),
- gomatrixserverlib.WithSkipVerify(true),
- )
-
testCases := []struct {
roomVer gomatrixserverlib.RoomVersion
eventJSON string
@@ -317,6 +295,29 @@ func TestRoomsV3URLEscapeDoNot404(t *testing.T) {
},
}
+ cfg, processCtx, close := testrig.CreateConfig(t, test.DBTypeSQLite)
+ defer close()
+ routers := httputil.NewRouters()
+
+ _, privKey, _ := ed25519.GenerateKey(nil)
+ cfg.Global.KeyID = gomatrixserverlib.KeyID("ed25519:auto")
+ cfg.Global.ServerName = gomatrixserverlib.ServerName("localhost")
+ cfg.Global.PrivateKey = privKey
+ cfg.Global.JetStream.InMemory = true
+ keyRing := &test.NopJSONVerifier{}
+ natsInstance := jetstream.NATSInstance{}
+ // TODO: This is pretty fragile, as if anything calls anything on these nils this test will break.
+ // Unfortunately, it makes little sense to instantiate these dependencies when we just want to test routing.
+ federationapi.AddPublicRoutes(processCtx, routers, cfg, &natsInstance, nil, nil, keyRing, nil, &internal.FederationInternalAPI{}, nil, caching.DisableMetrics)
+ baseURL, cancel := test.ListenAndServe(t, routers.Federation, true)
+ defer cancel()
+ serverName := gomatrixserverlib.ServerName(strings.TrimPrefix(baseURL, "https://"))
+
+ fedCli := gomatrixserverlib.NewFederationClient(
+ cfg.Global.SigningIdentities(),
+ gomatrixserverlib.WithSkipVerify(true),
+ )
+
for _, tc := range testCases {
ev, err := gomatrixserverlib.NewEventFromTrustedJSON([]byte(tc.eventJSON), false, tc.roomVer)
if err != nil {
diff --git a/federationapi/queue/queue_test.go b/federationapi/queue/queue_test.go
index c3fb1056..55d1df4a 100644
--- a/federationapi/queue/queue_test.go
+++ b/federationapi/queue/queue_test.go
@@ -22,6 +22,7 @@ import (
"time"
"github.com/matrix-org/dendrite/internal/caching"
+ "github.com/matrix-org/dendrite/internal/sqlutil"
"github.com/matrix-org/dendrite/test/testrig"
"go.uber.org/atomic"
"gotest.tools/v3/poll"
@@ -41,18 +42,19 @@ import (
func mustCreateFederationDatabase(t *testing.T, dbType test.DBType, realDatabase bool) (storage.Database, *process.ProcessContext, func()) {
if realDatabase {
// Real Database/s
- b, baseClose := testrig.CreateBaseDendrite(t, dbType)
- caches := caching.NewRistrettoCache(b.Cfg.Global.Cache.EstimatedMaxSize, b.Cfg.Global.Cache.MaxAge, caching.DisableMetrics)
+ cfg, processCtx, close := testrig.CreateConfig(t, dbType)
+ cm := sqlutil.NewConnectionManager(processCtx, cfg.Global.DatabaseOptions)
+ caches := caching.NewRistrettoCache(128*1024*1024, time.Hour, caching.DisableMetrics)
connStr, dbClose := test.PrepareDBConnectionString(t, dbType)
- db, err := storage.NewDatabase(b.ProcessContext.Context(), b.ConnectionManager, &config.DatabaseOptions{
+ db, err := storage.NewDatabase(processCtx.Context(), cm, &config.DatabaseOptions{
ConnectionString: config.DataSource(connStr),
- }, caches, b.Cfg.Global.IsLocalServerName)
+ }, caches, cfg.Global.IsLocalServerName)
if err != nil {
t.Fatalf("NewDatabase returned %s", err)
}
- return db, b.ProcessContext, func() {
+ return db, processCtx, func() {
+ close()
dbClose()
- baseClose()
}
} else {
// Fake Database
diff --git a/federationapi/routing/profile_test.go b/federationapi/routing/profile_test.go
index df494a74..d5e9997f 100644
--- a/federationapi/routing/profile_test.go
+++ b/federationapi/routing/profile_test.go
@@ -27,7 +27,10 @@ import (
fedAPI "github.com/matrix-org/dendrite/federationapi"
fedInternal "github.com/matrix-org/dendrite/federationapi/internal"
"github.com/matrix-org/dendrite/federationapi/routing"
+ "github.com/matrix-org/dendrite/internal/caching"
"github.com/matrix-org/dendrite/internal/httputil"
+ "github.com/matrix-org/dendrite/internal/sqlutil"
+ "github.com/matrix-org/dendrite/setup/jetstream"
"github.com/matrix-org/dendrite/test"
"github.com/matrix-org/dendrite/test/testrig"
userAPI "github.com/matrix-org/dendrite/userapi/api"
@@ -46,23 +49,26 @@ func (u *fakeUserAPI) QueryProfile(ctx context.Context, req *userAPI.QueryProfil
func TestHandleQueryProfile(t *testing.T) {
test.WithAllDatabases(t, func(t *testing.T, dbType test.DBType) {
- base, close := testrig.CreateBaseDendrite(t, dbType)
+ cfg, processCtx, close := testrig.CreateConfig(t, dbType)
+ cm := sqlutil.NewConnectionManager(processCtx, cfg.Global.DatabaseOptions)
+ routers := httputil.NewRouters()
defer close()
fedMux := mux.NewRouter().SkipClean(true).PathPrefix(httputil.PublicFederationPathPrefix).Subrouter().UseEncodedPath()
- base.Routers.Federation = fedMux
- base.Cfg.FederationAPI.Matrix.SigningIdentity.ServerName = testOrigin
- base.Cfg.FederationAPI.Matrix.Metrics.Enabled = false
+ natsInstance := jetstream.NATSInstance{}
+ routers.Federation = fedMux
+ cfg.FederationAPI.Matrix.SigningIdentity.ServerName = testOrigin
+ cfg.FederationAPI.Matrix.Metrics.Enabled = false
fedClient := fakeFedClient{}
serverKeyAPI := &signing.YggdrasilKeys{}
keyRing := serverKeyAPI.KeyRing()
- fedapi := fedAPI.NewInternalAPI(base, &fedClient, nil, nil, keyRing, true)
+ fedapi := fedAPI.NewInternalAPI(processCtx, cfg, cm, &natsInstance, &fedClient, nil, nil, keyRing, true)
userapi := fakeUserAPI{}
r, ok := fedapi.(*fedInternal.FederationInternalAPI)
if !ok {
panic("This is a programming error.")
}
- routing.Setup(base, nil, r, keyRing, &fedClient, &userapi, &base.Cfg.MSCs, nil, nil)
+ routing.Setup(routers, cfg, nil, r, keyRing, &fedClient, &userapi, &cfg.MSCs, nil, nil, caching.DisableMetrics)
handler := fedMux.Get(routing.QueryProfileRouteName).GetHandler().ServeHTTP
_, sk, _ := ed25519.GenerateKey(nil)
diff --git a/federationapi/routing/query_test.go b/federationapi/routing/query_test.go
index 69cf7047..6e702963 100644
--- a/federationapi/routing/query_test.go
+++ b/federationapi/routing/query_test.go
@@ -28,7 +28,10 @@ import (
fedclient "github.com/matrix-org/dendrite/federationapi/api"
fedInternal "github.com/matrix-org/dendrite/federationapi/internal"
"github.com/matrix-org/dendrite/federationapi/routing"
+ "github.com/matrix-org/dendrite/internal/caching"
"github.com/matrix-org/dendrite/internal/httputil"
+ "github.com/matrix-org/dendrite/internal/sqlutil"
+ "github.com/matrix-org/dendrite/setup/jetstream"
"github.com/matrix-org/dendrite/test"
"github.com/matrix-org/dendrite/test/testrig"
"github.com/matrix-org/gomatrixserverlib"
@@ -46,23 +49,26 @@ func (f *fakeFedClient) LookupRoomAlias(ctx context.Context, origin, s gomatrixs
func TestHandleQueryDirectory(t *testing.T) {
test.WithAllDatabases(t, func(t *testing.T, dbType test.DBType) {
- base, close := testrig.CreateBaseDendrite(t, dbType)
+ cfg, processCtx, close := testrig.CreateConfig(t, dbType)
+ cm := sqlutil.NewConnectionManager(processCtx, cfg.Global.DatabaseOptions)
+ routers := httputil.NewRouters()
defer close()
fedMux := mux.NewRouter().SkipClean(true).PathPrefix(httputil.PublicFederationPathPrefix).Subrouter().UseEncodedPath()
- base.Routers.Federation = fedMux
- base.Cfg.FederationAPI.Matrix.SigningIdentity.ServerName = testOrigin
- base.Cfg.FederationAPI.Matrix.Metrics.Enabled = false
+ natsInstance := jetstream.NATSInstance{}
+ routers.Federation = fedMux
+ cfg.FederationAPI.Matrix.SigningIdentity.ServerName = testOrigin
+ cfg.FederationAPI.Matrix.Metrics.Enabled = false
fedClient := fakeFedClient{}
serverKeyAPI := &signing.YggdrasilKeys{}
keyRing := serverKeyAPI.KeyRing()
- fedapi := fedAPI.NewInternalAPI(base, &fedClient, nil, nil, keyRing, true)
+ fedapi := fedAPI.NewInternalAPI(processCtx, cfg, cm, &natsInstance, &fedClient, nil, nil, keyRing, true)
userapi := fakeUserAPI{}
r, ok := fedapi.(*fedInternal.FederationInternalAPI)
if !ok {
panic("This is a programming error.")
}
- routing.Setup(base, nil, r, keyRing, &fedClient, &userapi, &base.Cfg.MSCs, nil, nil)
+ routing.Setup(routers, cfg, nil, r, keyRing, &fedClient, &userapi, &cfg.MSCs, nil, nil, caching.DisableMetrics)
handler := fedMux.Get(routing.QueryDirectoryRouteName).GetHandler().ServeHTTP
_, sk, _ := ed25519.GenerateKey(nil)
diff --git a/federationapi/routing/routing.go b/federationapi/routing/routing.go
index c86d18d2..a1f943e7 100644
--- a/federationapi/routing/routing.go
+++ b/federationapi/routing/routing.go
@@ -31,7 +31,6 @@ import (
"github.com/matrix-org/dendrite/internal/httputil"
"github.com/matrix-org/dendrite/roomserver/api"
roomserverAPI "github.com/matrix-org/dendrite/roomserver/api"
- "github.com/matrix-org/dendrite/setup/base"
"github.com/matrix-org/dendrite/setup/config"
userapi "github.com/matrix-org/dendrite/userapi/api"
"github.com/matrix-org/gomatrixserverlib"
@@ -55,7 +54,8 @@ const (
// applied:
// nolint: gocyclo
func Setup(
- base *base.BaseDendrite,
+ routers httputil.Routers,
+ dendriteCfg *config.Dendrite,
rsAPI roomserverAPI.FederationRoomserverAPI,
fsAPI *fedInternal.FederationInternalAPI,
keys gomatrixserverlib.JSONVerifier,
@@ -63,14 +63,14 @@ func Setup(
userAPI userapi.FederationUserAPI,
mscCfg *config.MSCs,
servers federationAPI.ServersInRoomProvider,
- producer *producers.SyncAPIProducer,
+ producer *producers.SyncAPIProducer, enableMetrics bool,
) {
- fedMux := base.Routers.Federation
- keyMux := base.Routers.Keys
- wkMux := base.Routers.WellKnown
- cfg := &base.Cfg.FederationAPI
+ fedMux := routers.Federation
+ keyMux := routers.Keys
+ wkMux := routers.WellKnown
+ cfg := &dendriteCfg.FederationAPI
- if base.EnableMetrics {
+ if enableMetrics {
prometheus.MustRegister(
internal.PDUCountTotal, internal.EDUCountTotal,
)
diff --git a/federationapi/routing/send_test.go b/federationapi/routing/send_test.go
index 53e1399a..28fa6d6d 100644
--- a/federationapi/routing/send_test.go
+++ b/federationapi/routing/send_test.go
@@ -25,7 +25,10 @@ import (
fedAPI "github.com/matrix-org/dendrite/federationapi"
fedInternal "github.com/matrix-org/dendrite/federationapi/internal"
"github.com/matrix-org/dendrite/federationapi/routing"
+ "github.com/matrix-org/dendrite/internal/caching"
"github.com/matrix-org/dendrite/internal/httputil"
+ "github.com/matrix-org/dendrite/internal/sqlutil"
+ "github.com/matrix-org/dendrite/setup/jetstream"
"github.com/matrix-org/dendrite/test"
"github.com/matrix-org/dendrite/test/testrig"
"github.com/matrix-org/gomatrixserverlib"
@@ -44,21 +47,24 @@ type sendContent struct {
func TestHandleSend(t *testing.T) {
test.WithAllDatabases(t, func(t *testing.T, dbType test.DBType) {
- base, close := testrig.CreateBaseDendrite(t, dbType)
+ cfg, processCtx, close := testrig.CreateConfig(t, dbType)
+ cm := sqlutil.NewConnectionManager(processCtx, cfg.Global.DatabaseOptions)
+ routers := httputil.NewRouters()
defer close()
fedMux := mux.NewRouter().SkipClean(true).PathPrefix(httputil.PublicFederationPathPrefix).Subrouter().UseEncodedPath()
- base.Routers.Federation = fedMux
- base.Cfg.FederationAPI.Matrix.SigningIdentity.ServerName = testOrigin
- base.Cfg.FederationAPI.Matrix.Metrics.Enabled = false
- fedapi := fedAPI.NewInternalAPI(base, nil, nil, nil, nil, true)
+ natsInstance := jetstream.NATSInstance{}
+ routers.Federation = fedMux
+ cfg.FederationAPI.Matrix.SigningIdentity.ServerName = testOrigin
+ cfg.FederationAPI.Matrix.Metrics.Enabled = false
+ fedapi := fedAPI.NewInternalAPI(processCtx, cfg, cm, &natsInstance, nil, nil, nil, nil, true)
serverKeyAPI := &signing.YggdrasilKeys{}
keyRing := serverKeyAPI.KeyRing()
r, ok := fedapi.(*fedInternal.FederationInternalAPI)
if !ok {
panic("This is a programming error.")
}
- routing.Setup(base, nil, r, keyRing, nil, nil, &base.Cfg.MSCs, nil, nil)
+ routing.Setup(routers, cfg, nil, r, keyRing, nil, nil, &cfg.MSCs, nil, nil, caching.DisableMetrics)
handler := fedMux.Get(routing.SendRouteName).GetHandler().ServeHTTP
_, sk, _ := ed25519.GenerateKey(nil)
diff --git a/federationapi/storage/storage_test.go b/federationapi/storage/storage_test.go
index a02b71b6..74863c07 100644
--- a/federationapi/storage/storage_test.go
+++ b/federationapi/storage/storage_test.go
@@ -20,7 +20,7 @@ func mustCreateFederationDatabase(t *testing.T, dbType test.DBType) (storage.Dat
caches := caching.NewRistrettoCache(8*1024*1024, time.Hour, false)
connStr, dbClose := test.PrepareDBConnectionString(t, dbType)
ctx := context.Background()
- cm := sqlutil.NewConnectionManager()
+ cm := sqlutil.NewConnectionManager(nil, config.DatabaseOptions{})
db, err := storage.NewDatabase(ctx, cm, &config.DatabaseOptions{
ConnectionString: config.DataSource(connStr),
}, caches, func(server gomatrixserverlib.ServerName) bool { return server == "localhost" })