Skip to content
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.

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

Roku client appears to NOT update NFO file after video finishes playing through to the very end #1814

Closed
thecodygriffin opened this issue Jul 3, 2024 · 10 comments · Fixed by #1941
Labels
bug Something isn't working

Comments

@thecodygriffin
Copy link

Software Versions

  • Jellyfin Server Version: 10.9.7
  • Roku Client Version: 2.0.7

Describe the bug

Upon allowing a video to play through to the very end, the Roku client appears to NOT prompt the update of the corresponding NFO file.

To be very specific, "watched" is NOT updating from "false" to "true" in the corresponding NFO file.

Prerequisites To Reproduce

  1. Navigate to the Libraries > Libraries section of the JellyFin server from the Dashboard in the main navigation
  2. Select the "Manage library" option in the kebab menu for the testing library
  3. Select the "Nfo" checkbox in the "Metadata savers" section for the testing library
  4. Select "OK" to save the setting
  5. Navigate to the Libraries > NFO Settings section of the JellyFin server in the main navigation menu
  6. Select the the user to be used for testing in the "Save user watch data to NFO files for" field
  7. Select "Save" to save the setting

How To Reproduce

  1. Open the JellyFin Roku client
  2. Ensure you are logged in with the same user selected in Step 6 of the "Prerequisites To Reproduce" section
  3. Navigate to the library selected in Step 2 of the "Prerequisites To Reproduce" section
  4. Select any video that has not yet been watched
  5. Watch the video through to the very end when the stream closes and returns you to the library

Expected behavior

The NFO file is updated including "playcount" and "watched" among others when applicable

Logs

There are no [ERR] messages in the logs for my testing time range. There was an instance of the following [WRN] message, but it does not appear to correlate with the behavior experienced.

[2024-07-03 10:52:25.892 -04:00] [WRN] WS "10.16.8.1" error receiving data: "The remote party closed the WebSocket connection without completing the close handshake."

Connection Information

  • Is server local or remote? LOCAL
  • Is server connection HTTP or HTTPS? HTTP

Additional context

If the video is exited by the user prior to conclusion, the NFO file is updated as expected whether that be the "watched", "playcount","lastplayed", or "resume" values.

Furthermore, the behavior is NOT reproducible from the Android or Web clients.

@thecodygriffin thecodygriffin added the bug Something isn't working label Jul 3, 2024
@thecodygriffin thecodygriffin changed the title Roku client does NOT update NFO file after video finishes playing through to the very end Roku client appears to NOT update NFO file after video finishes playing through to the very end Jul 3, 2024
@roidy
Copy link

roidy commented Sep 13, 2024

Maybe related to the following issue:- #1567

The app handles the stopped state which happens when the user ends a file by pressing back on the remote but fails to handle the finished state which is fired when the media naturally stops at the end with no user interaction.

@cewert
Copy link
Member

cewert commented Sep 28, 2024

@thecodygriffin can you give #1941 a test when you get a chance?

@thecodygriffin
Copy link
Author

@cewert Is it fair to assume that I will need to build the Roku app with the #1941 pull request and then side load it to test?

If so, I may not be able to do that myself for a few weeks. Otherwise, if there is an existing test build you can point me to, I can side load it sometime this weekend and test.

@cewert
Copy link
Member

cewert commented Sep 28, 2024

@cewert Is it fair to assume that I will need to build the Roku app with the #1941 pull request and then side load it to test?

Yes, you will need to sideload the app to test but you don't have to build it unless you want to use the debugger and see debugging output. Our CI automatically creates a build zip for all PRs https://github.com/jellyfin/jellyfin-roku/actions/runs/11086513250?pr=1941

@roidy
Copy link

roidy commented Sep 28, 2024

Unfortunately that doesn't fix my problem of the video player not returning to SDR mode after playing a HDR/DV file to completion. In fact it's still not registering a finished state.

@roidy
Copy link

roidy commented Sep 28, 2024

Here's a video showing the issue:- https://streamable.com/d4wqgk

Watching the activity in the Jellyfin server you can see that when stopping media using the back button on the remote the server registers a finished state, however when the media naturally stops at the end with no user interaction no finished state is being sent to the server and the server still thinks the video is active.

@thecodygriffin
Copy link
Author

@cewert similar to @roidy findings, the build from https://github.com/jellyfin/jellyfin-roku/actions/runs/11086513250?pr=1941 did not resolve this issue. The NFO file is still not updating upon the "natural" conclusion of a video.

@cewert
Copy link
Member

cewert commented Sep 29, 2024

@thecodygriffin There was a bug in that build. Sorry for not giving you the new link. Can you try this build as well? That's the current build for #1941

@thecodygriffin
Copy link
Author

@cewert Success! The latest build from https://github.com/jellyfin/jellyfin-roku/actions/runs/11087871463?pr=1941 resolves the issue.

I confirmed that the NFO file was updated in the following scenarios:

  • Upon a video "naturally" concluding and returning me back to the library
  • Upon a video "naturally" concluding and auto playing the next video in a season

I also verified that the NFO file is still updated when existing it early prior to the conclusion.

Thank you for your work on this. I can't wait until the fix makes it into an upcoming release.

@cewert cewert linked a pull request Sep 29, 2024 that will close this issue
@cewert
Copy link
Member

cewert commented Sep 29, 2024

@cewert cewert closed this as completed Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants