mirror of
https://github.com/navidrome/navidrome.git
synced 2026-06-02 07:01:36 +00:00
* feat(plugins): add PlaybackReport to Scrobbler interface and all implementations * feat(plugins): add PlaybackReport worker and dispatch in PlayTracker * feat(plugins): add PlaybackReportRequest to plugin scrobbler capability * chore(plugins): regenerate PDK files with PlaybackReport * feat(plugins): add PlaybackReport to test scrobbler plugin * feat(plugins): add PlaybackReport to plugin scrobbler adapter * refactor(plugins): fix double DB fetch in StateStopped and batch getActiveScrobblers - Hoist mf from scrobble branch so PlaybackReport reuses it instead of fetching again from DB - Call getActiveScrobblers once per drain batch instead of per-entry * chore(plugins): include generated scrobbler schema with PlaybackReport * fix(plugins): skip PlaybackReport for plugins that don't export it Plugins detected as scrobblers only need to export one scrobbler function. Older plugins that don't export nd_scrobbler_playback_report would cause noisy error logs on every reportPlayback call. Now errFunctionNotFound and errNotImplemented are treated as no-ops. * refactor: rename NowPlayingInfo to PlaybackReport Signed-off-by: Deluan <deluan@navidrome.org> * refactor: rename stopNowPlayingWorker to stopBackgroundWorkers Signed-off-by: Deluan <deluan@navidrome.org> * refactor: move NowPlaying and PlaybackReport logic to separate worker files Signed-off-by: Deluan <deluan@navidrome.org> * refactor(scrobbler): rename NowPlayingInfo to PlaybackSession and add expired state Rename NowPlayingInfo struct to PlaybackSession to better reflect its role as a complete playback session representation. Add UserId field to make sessions self-contained, removing redundant userId parameters from PlaybackReport interface method and internal dispatch functions. Introduce StateExpired internal state that fires when a session cache entry expires without an explicit stop, ensuring plugins always receive a terminal event regardless of client behavior. * fix(scrobbler): update playback state description to include 'expired' Signed-off-by: Deluan <deluan@navidrome.org> * fix(scrobbler): resolve data race in OnExpiration callback Capture conf.Server.EnableNowPlaying at construction time instead of reading it from the background ttlcache eviction goroutine. The previous code raced with test config cleanup that writes to the same field concurrently. * fix(scrobbler): return error when media file lookup fails in StateStopped Simplify the MediaFile population logic in the stopped case to return an error if the track cannot be found. A stop report with an empty MediaFile is useless to plugins, and returning the error allows clients to retry or alert the user when auto-scrobble is enabled. * refactor(scrobbler): use session data directly in PlaybackReport adapter Use info.Username from PlaybackSession instead of extracting it from context in the plugin adapter, since the session is now self-contained. Add debug/trace logging for session expiration and enqueue the expired report with a user-enriched context so downstream handlers can identify the user. --------- Signed-off-by: Deluan <deluan@navidrome.org>
366 lines
11 KiB
Go
366 lines
11 KiB
Go
//go:build !windows
|
|
|
|
package plugins
|
|
|
|
import (
|
|
"context"
|
|
"errors"
|
|
"time"
|
|
|
|
"github.com/navidrome/navidrome/core/scrobbler"
|
|
"github.com/navidrome/navidrome/model"
|
|
"github.com/navidrome/navidrome/model/request"
|
|
. "github.com/onsi/ginkgo/v2"
|
|
. "github.com/onsi/gomega"
|
|
)
|
|
|
|
// ctxWithUser returns a fresh context with the test user.
|
|
// Must be called within each test, not in BeforeAll, because the context
|
|
// from BeforeAll gets cancelled before tests run.
|
|
func ctxWithUser() context.Context {
|
|
return request.WithUser(GinkgoT().Context(), model.User{ID: "user-1", UserName: "testuser"})
|
|
}
|
|
|
|
var _ = Describe("ScrobblerPlugin", Ordered, func() {
|
|
var (
|
|
scrobblerManager *Manager
|
|
s scrobbler.Scrobbler
|
|
)
|
|
|
|
BeforeAll(func() {
|
|
// Load the scrobbler via a new manager with the test-scrobbler plugin
|
|
scrobblerManager, _ = createTestManagerWithPlugins(nil, "test-scrobbler"+PackageExtension)
|
|
|
|
var ok bool
|
|
s, ok = scrobblerManager.LoadScrobbler("test-scrobbler")
|
|
Expect(ok).To(BeTrue())
|
|
})
|
|
|
|
Describe("LoadScrobbler", func() {
|
|
It("returns a scrobbler for a plugin with Scrobbler capability", func() {
|
|
Expect(s).ToNot(BeNil())
|
|
})
|
|
|
|
It("returns false for a plugin without Scrobbler capability", func() {
|
|
_, ok := testManager.LoadScrobbler("test-metadata-agent")
|
|
Expect(ok).To(BeFalse())
|
|
})
|
|
|
|
It("returns false for non-existent plugin", func() {
|
|
_, ok := scrobblerManager.LoadScrobbler("non-existent")
|
|
Expect(ok).To(BeFalse())
|
|
})
|
|
})
|
|
|
|
Describe("IsAuthorized", func() {
|
|
It("returns true when plugin is configured to authorize", func() {
|
|
result := s.IsAuthorized(ctxWithUser(), "user-1")
|
|
Expect(result).To(BeTrue())
|
|
})
|
|
|
|
Context("when plugin is configured to not authorize", Ordered, func() {
|
|
var notAuthScrobbler scrobbler.Scrobbler
|
|
|
|
BeforeAll(func() {
|
|
mgr, _ := createTestManagerWithPlugins(map[string]map[string]string{
|
|
"test-scrobbler": {"authorized": "false"},
|
|
}, "test-scrobbler"+PackageExtension)
|
|
|
|
var ok bool
|
|
notAuthScrobbler, ok = mgr.LoadScrobbler("test-scrobbler")
|
|
Expect(ok).To(BeTrue())
|
|
})
|
|
|
|
It("returns false", func() {
|
|
result := notAuthScrobbler.IsAuthorized(ctxWithUser(), "user-1")
|
|
Expect(result).To(BeFalse())
|
|
})
|
|
})
|
|
})
|
|
|
|
Describe("isUserAllowed", func() {
|
|
It("returns true when allUsers is true", func() {
|
|
sp := &ScrobblerPlugin{allUsers: true}
|
|
Expect(sp.isUserAllowed("any-user")).To(BeTrue())
|
|
})
|
|
|
|
It("returns false when allowedUserIDs is empty and allUsers is false", func() {
|
|
sp := &ScrobblerPlugin{allUsers: false, allowedUserIDs: []string{}}
|
|
Expect(sp.isUserAllowed("user-1")).To(BeFalse())
|
|
})
|
|
|
|
It("returns false when allowedUserIDs is nil and allUsers is false", func() {
|
|
sp := &ScrobblerPlugin{allUsers: false}
|
|
Expect(sp.isUserAllowed("user-1")).To(BeFalse())
|
|
})
|
|
|
|
It("returns true when user is in allowedUserIDs", func() {
|
|
sp := &ScrobblerPlugin{
|
|
allUsers: false,
|
|
allowedUserIDs: []string{"user-1", "user-2"},
|
|
userIDMap: map[string]struct{}{"user-1": {}, "user-2": {}},
|
|
}
|
|
Expect(sp.isUserAllowed("user-1")).To(BeTrue())
|
|
})
|
|
|
|
It("returns false when user is not in allowedUserIDs", func() {
|
|
sp := &ScrobblerPlugin{
|
|
allUsers: false,
|
|
allowedUserIDs: []string{"user-1", "user-2"},
|
|
userIDMap: map[string]struct{}{"user-1": {}, "user-2": {}},
|
|
}
|
|
Expect(sp.isUserAllowed("user-3")).To(BeFalse())
|
|
})
|
|
})
|
|
|
|
Describe("NowPlaying", func() {
|
|
It("successfully calls the plugin", func() {
|
|
track := &model.MediaFile{
|
|
ID: "track-1",
|
|
Title: "Test Song",
|
|
Album: "Test Album",
|
|
Artist: "Test Artist",
|
|
AlbumArtist: "Test Album Artist",
|
|
Duration: 180,
|
|
TrackNumber: 1,
|
|
DiscNumber: 1,
|
|
Participants: model.Participants{
|
|
model.RoleArtist: {{Artist: model.Artist{ID: "artist-1", Name: "Test Artist"}}},
|
|
model.RoleAlbumArtist: {{Artist: model.Artist{ID: "album-artist-1", Name: "Test Album Artist"}}},
|
|
},
|
|
}
|
|
|
|
err := s.NowPlaying(ctxWithUser(), "user-1", track, 30)
|
|
Expect(err).ToNot(HaveOccurred())
|
|
})
|
|
|
|
Context("when plugin returns error", Ordered, func() {
|
|
var retryScrobbler scrobbler.Scrobbler
|
|
|
|
BeforeAll(func() {
|
|
mgr, _ := createTestManagerWithPlugins(map[string]map[string]string{
|
|
"test-scrobbler": {"error": "service unavailable", "error_type": "scrobbler(retry_later)"},
|
|
}, "test-scrobbler"+PackageExtension)
|
|
|
|
var ok bool
|
|
retryScrobbler, ok = mgr.LoadScrobbler("test-scrobbler")
|
|
Expect(ok).To(BeTrue())
|
|
})
|
|
|
|
It("returns ErrRetryLater", func() {
|
|
track := &model.MediaFile{ID: "track-1", Title: "Test Song"}
|
|
err := retryScrobbler.NowPlaying(ctxWithUser(), "user-1", track, 30)
|
|
Expect(err).To(HaveOccurred())
|
|
Expect(err).To(MatchError(scrobbler.ErrRetryLater))
|
|
})
|
|
})
|
|
})
|
|
|
|
Describe("Scrobble", func() {
|
|
It("successfully calls the plugin", func() {
|
|
sc := scrobbler.Scrobble{
|
|
MediaFile: model.MediaFile{
|
|
ID: "track-1",
|
|
Title: "Test Song",
|
|
Album: "Test Album",
|
|
Artist: "Test Artist",
|
|
AlbumArtist: "Test Album Artist",
|
|
Duration: 180,
|
|
TrackNumber: 1,
|
|
DiscNumber: 1,
|
|
Participants: model.Participants{
|
|
model.RoleArtist: {{Artist: model.Artist{ID: "artist-1", Name: "Test Artist"}}},
|
|
model.RoleAlbumArtist: {{Artist: model.Artist{ID: "album-artist-1", Name: "Test Album Artist"}}},
|
|
},
|
|
},
|
|
TimeStamp: time.Now(),
|
|
}
|
|
|
|
err := s.Scrobble(ctxWithUser(), "user-1", sc)
|
|
Expect(err).ToNot(HaveOccurred())
|
|
})
|
|
|
|
Context("when plugin returns not_authorized error", Ordered, func() {
|
|
var notAuthScrobbler scrobbler.Scrobbler
|
|
|
|
BeforeAll(func() {
|
|
mgr, _ := createTestManagerWithPlugins(map[string]map[string]string{
|
|
"test-scrobbler": {"error": "user not linked", "error_type": "scrobbler(not_authorized)"},
|
|
}, "test-scrobbler"+PackageExtension)
|
|
|
|
var ok bool
|
|
notAuthScrobbler, ok = mgr.LoadScrobbler("test-scrobbler")
|
|
Expect(ok).To(BeTrue())
|
|
})
|
|
|
|
It("returns ErrNotAuthorized", func() {
|
|
scrobble := scrobbler.Scrobble{
|
|
MediaFile: model.MediaFile{ID: "track-1", Title: "Test Song"},
|
|
TimeStamp: time.Now(),
|
|
}
|
|
err := notAuthScrobbler.Scrobble(ctxWithUser(), "user-1", scrobble)
|
|
Expect(err).To(HaveOccurred())
|
|
Expect(err).To(MatchError(scrobbler.ErrNotAuthorized))
|
|
})
|
|
})
|
|
|
|
Context("when plugin returns unrecoverable error", Ordered, func() {
|
|
var unrecoverableScrobbler scrobbler.Scrobbler
|
|
|
|
BeforeAll(func() {
|
|
mgr, _ := createTestManagerWithPlugins(map[string]map[string]string{
|
|
"test-scrobbler": {"error": "track rejected", "error_type": "scrobbler(unrecoverable)"},
|
|
}, "test-scrobbler"+PackageExtension)
|
|
|
|
var ok bool
|
|
unrecoverableScrobbler, ok = mgr.LoadScrobbler("test-scrobbler")
|
|
Expect(ok).To(BeTrue())
|
|
})
|
|
|
|
It("returns ErrUnrecoverable", func() {
|
|
scrobble := scrobbler.Scrobble{
|
|
MediaFile: model.MediaFile{ID: "track-1", Title: "Test Song"},
|
|
TimeStamp: time.Now(),
|
|
}
|
|
err := unrecoverableScrobbler.Scrobble(ctxWithUser(), "user-1", scrobble)
|
|
Expect(err).To(HaveOccurred())
|
|
Expect(err).To(MatchError(scrobbler.ErrUnrecoverable))
|
|
})
|
|
})
|
|
})
|
|
|
|
Describe("PlaybackReport", func() {
|
|
It("successfully calls the plugin", func() {
|
|
info := scrobbler.PlaybackSession{
|
|
MediaFile: model.MediaFile{
|
|
ID: "track-1",
|
|
Title: "Test Song",
|
|
Album: "Test Album",
|
|
Artist: "Test Artist",
|
|
AlbumArtist: "Test Album Artist",
|
|
Duration: 180,
|
|
TrackNumber: 1,
|
|
DiscNumber: 1,
|
|
Participants: model.Participants{
|
|
model.RoleArtist: {{Artist: model.Artist{ID: "artist-1", Name: "Test Artist"}}},
|
|
model.RoleAlbumArtist: {{Artist: model.Artist{ID: "album-artist-1", Name: "Test Album Artist"}}},
|
|
},
|
|
},
|
|
Username: "testuser",
|
|
PlayerId: "player-1",
|
|
PlayerName: "Test Player",
|
|
State: "playing",
|
|
PositionMs: 30000,
|
|
PlaybackRate: 1.0,
|
|
LastReport: time.Now(),
|
|
}
|
|
|
|
err := s.PlaybackReport(ctxWithUser(), info)
|
|
Expect(err).ToNot(HaveOccurred())
|
|
})
|
|
|
|
Context("when plugin returns error", Ordered, func() {
|
|
var retryScrobbler scrobbler.Scrobbler
|
|
|
|
BeforeAll(func() {
|
|
mgr, _ := createTestManagerWithPlugins(map[string]map[string]string{
|
|
"test-scrobbler": {"error": "service unavailable", "error_type": "scrobbler(retry_later)"},
|
|
}, "test-scrobbler"+PackageExtension)
|
|
|
|
var ok bool
|
|
retryScrobbler, ok = mgr.LoadScrobbler("test-scrobbler")
|
|
Expect(ok).To(BeTrue())
|
|
})
|
|
|
|
It("returns ErrRetryLater", func() {
|
|
info := scrobbler.PlaybackSession{
|
|
MediaFile: model.MediaFile{ID: "track-1", Title: "Test Song"},
|
|
State: "playing",
|
|
LastReport: time.Now(),
|
|
}
|
|
err := retryScrobbler.PlaybackReport(ctxWithUser(), info)
|
|
Expect(err).To(HaveOccurred())
|
|
Expect(err).To(MatchError(scrobbler.ErrRetryLater))
|
|
})
|
|
})
|
|
})
|
|
|
|
Describe("PluginNames", func() {
|
|
It("returns plugin names with Scrobbler capability", func() {
|
|
names := scrobblerManager.PluginNames("Scrobbler")
|
|
Expect(names).To(ContainElement("test-scrobbler"))
|
|
})
|
|
|
|
It("does not return metadata agent plugins for Scrobbler capability", func() {
|
|
names := testManager.PluginNames("Scrobbler")
|
|
Expect(names).ToNot(ContainElement("test-metadata-agent"))
|
|
})
|
|
})
|
|
|
|
Describe("mediaFileToTrackInfo", func() {
|
|
var track *model.MediaFile
|
|
|
|
BeforeEach(func() {
|
|
track = &model.MediaFile{
|
|
ID: "track-1",
|
|
Title: "Test Song",
|
|
Path: "/music/test.flac",
|
|
LibraryID: 1,
|
|
}
|
|
})
|
|
|
|
fsManifest := &Manifest{
|
|
Permissions: &Permissions{
|
|
Library: &LibraryPermission{Filesystem: true},
|
|
},
|
|
}
|
|
|
|
It("includes LibraryID and Path when the plugin has filesystem access to the track's library", func() {
|
|
p := &plugin{manifest: fsManifest, libraries: newLibraryAccess([]int{1}, false)}
|
|
ti := mediaFileToTrackInfo(p, track)
|
|
Expect(ti.LibraryID).To(Equal(int32(1)))
|
|
Expect(ti.Path).To(Equal("/music/test.flac"))
|
|
})
|
|
|
|
It("omits LibraryID and Path when the plugin lacks filesystem permission", func() {
|
|
p := &plugin{manifest: &Manifest{}, libraries: newLibraryAccess([]int{1}, false)}
|
|
ti := mediaFileToTrackInfo(p, track)
|
|
Expect(ti.LibraryID).To(BeZero())
|
|
Expect(ti.Path).To(BeEmpty())
|
|
})
|
|
|
|
It("omits LibraryID and Path when the track's library is not in the allowed set", func() {
|
|
p := &plugin{manifest: fsManifest, libraries: newLibraryAccess([]int{2}, false)}
|
|
ti := mediaFileToTrackInfo(p, track)
|
|
Expect(ti.LibraryID).To(BeZero())
|
|
Expect(ti.Path).To(BeEmpty())
|
|
})
|
|
})
|
|
})
|
|
|
|
var _ = Describe("mapScrobblerError", func() {
|
|
It("returns nil for nil error", func() {
|
|
Expect(mapScrobblerError(nil)).ToNot(HaveOccurred())
|
|
})
|
|
|
|
It("returns ErrNotAuthorized for error containing 'not_authorized'", func() {
|
|
err := mapScrobblerError(errors.New("plugin error: scrobbler(not_authorized)"))
|
|
Expect(err).To(MatchError(scrobbler.ErrNotAuthorized))
|
|
})
|
|
|
|
It("returns ErrRetryLater for error containing 'retry_later'", func() {
|
|
err := mapScrobblerError(errors.New("temporary failure: scrobbler(retry_later)"))
|
|
Expect(err).To(MatchError(scrobbler.ErrRetryLater))
|
|
})
|
|
|
|
It("returns ErrUnrecoverable for error containing 'unrecoverable'", func() {
|
|
err := mapScrobblerError(errors.New("fatal error: scrobbler(unrecoverable)"))
|
|
Expect(err).To(MatchError(scrobbler.ErrUnrecoverable))
|
|
})
|
|
|
|
It("returns ErrUnrecoverable for unknown error", func() {
|
|
err := mapScrobblerError(errors.New("some unknown error"))
|
|
Expect(err).To(MatchError(scrobbler.ErrUnrecoverable))
|
|
})
|
|
})
|