jesseward / jellyfin-plugin-lastfm

LastFM plugin for the Jellyfin media system. Fork of the Emby Last.FM plug-in
178 stars 11 forks source link

Scrobble request should use time the track started playing, not the current time #54

Open csaavedra opened 10 months ago

csaavedra commented 10 months ago

From the spec (https://www.last.fm/api/show/track.scrobble):

timestamp[i] (Required) : The time the track started playing, in UNIX timestamp format (integer number of seconds since 00:00:00, January 1st 1970 UTC). This must be in the UTC time zone.

However the plugin is just using the current time, instead, see https://github.com/jesseward/jellyfin-plugin-lastfm/blob/6c401ee29226e7279f13807e3028f9976a559006/Jellyfin.Plugin.Lastfm/Api/LastfmApiClient.cs#L54