summaryrefslogtreecommitdiff
path: root/internal/api/middleware.go
diff options
context:
space:
mode:
authorPaul Buetow <paul@buetow.org>2026-05-04 00:10:32 +0300
committerPaul Buetow <paul@buetow.org>2026-05-04 00:10:32 +0300
commitc68c54b0212ebc8d111b11c6edff82942c3a4463 (patch)
treefaab06668883d4977aebe4885d61cb0e00644009 /internal/api/middleware.go
parent2774c4706214be226d822ab296ac062f5212f740 (diff)
task 6: narrow service interfaces in Server and Middleware
Split Server struct to accept narrow service interfaces instead of fat composites (MediaService, AdminService). Each handler now depends only on its specific slice (MediaBrowseService, MediaWriteService, etc.). Split Middleware to depend on a narrow UserStore interface instead of full repository.Store. Updated all constructors, call sites, and tests. Added negative tests for nil AdminService and ProgressService returning 501 Not Implemented. References task 6.
Diffstat (limited to 'internal/api/middleware.go')
-rw-r--r--internal/api/middleware.go11
1 files changed, 8 insertions, 3 deletions
diff --git a/internal/api/middleware.go b/internal/api/middleware.go
index bc19fc8..c6cc303 100644
--- a/internal/api/middleware.go
+++ b/internal/api/middleware.go
@@ -7,7 +7,6 @@ import (
"codeberg.org/snonux/player/internal/auth"
"codeberg.org/snonux/player/internal/model"
- "codeberg.org/snonux/player/internal/repository"
)
type ctxKey int
@@ -17,14 +16,20 @@ const (
userCtxKey
)
+// UserStore is the narrow interface Middleware needs for user lookups.
+type UserStore interface {
+ CountUsers(ctx context.Context) (int, error)
+ GetUserByID(ctx context.Context, id int64) (*model.User, error)
+}
+
// Middleware holds dependencies for middleware constructors.
type Middleware struct {
- store repository.Store
+ store UserStore
sm *auth.SessionManager
}
// NewMiddleware creates middleware handlers.
-func NewMiddleware(store repository.Store, sm *auth.SessionManager) *Middleware {
+func NewMiddleware(store UserStore, sm *auth.SessionManager) *Middleware {
return &Middleware{store: store, sm: sm}
}