[Jellyfin playback reporting] Reported playtime is less than song length

[internetfreak]

Another report from me, it's like there's an endless cycle of them? But we're getting there. It's not that I'm too dependent on that feature, but it never hurts to be precise. Also, I'll use this as a way to thank you for fixing my previous issues, it works really well.

Describe the bug Now that we've fixed the previous reporting issues and I've tested them, I noticed that the playtime logged by the playback reporting plugin seems to be consistently 15-30s off compared to the original song length. Tracks played via jellyfin web UI are logged properly

To Reproduce Steps to reproduce the behavior:

1. Play any song, wait about 30s for the plugin to notice (that happens via web UI as well, maybe jellyfin needs some time to fire the event)
2. Wait for the song to end
3. Open the reporting interface, set the end date to one day further than the current date, so you see what got logged today.
4. Compare the duration reported by the plugin to the length of the song. Notice how it's different (I've seen 15-30s depending on the song)

Expected behavior Playback time is reported accurately

Screenshots As always, I have a few screenshots prepared. First comes the reports of the plugin from today: You can see, I have played a few songs via Sonixd and one via Web for comparision. Now let's see the lengths of each song as displayed by jellyfin This one is reported 21s too short 29 seconds missing 15s missing Correct length, this was the "control" song

Desktop (please complete the following information):

• OS: Windows
• Version: 0.12.1

Additional context Status reported via dashboard seems to look good, the played song is displayed immediately and the position also matches closely enough, so jellyfin does in fact know what is being played

[jeffvli]

The missing length is probably due to the combination of the 5s delayed play signal and sending the stop signal when the track hits the 90% / 4 minute mark. I'll see if I can more closely match the times but I'm going to push this issue back since it's at least somewhat stable now and I have a few other features that I want to address.