New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Open

csaavedra opened this issue Oct 9, 2023 · 0 comments

Labels

bug good first issue

csaavedra commented Oct 9, 2023

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

jellyfin-plugin-lastfm/Jellyfin.Plugin.Lastfm/Api/LastfmApiClient.cs

Line 54 in 6c401ee

Timestamp = Helpers.CurrentTimestamp(),

jesseward added bug good first issue labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment