jellyfin-mpv-shim icon indicating copy to clipboard operation
jellyfin-mpv-shim copied to clipboard

mpv shim was able to transcode when it was turned off for an user

Open d3lac opened this issue 4 years ago • 7 comments

Describe the bug

mpv shim was able to transcode a video when it was disabled for that particular user.

System (please complete the following information):

  • OS: Debian 10
  • Virtualization: KVM
  • Clients: mpv shim
  • Jellyfin Version: 10.6.4
  • Playback: transcode
  • Reverse Proxy: nginx
  • Base URL: none
  • Networking: Host
  • Storage: cloud

To Reproduce

  1. Create an user with transcode disabled and login to mpv shim with that user
  2. Open mpv shim conf.json (I'm using Windows 10 - %appdata%\jellyfin-mpv-shim\conf.json)
  3. Check the line "always_transcode": false, change it to true and restart mpv shim
  4. Now open and try playing any video, it will transcode.

Logs

https://del.dog/raw/vafisappeg

Screenshots

https://imgur.com/a/hsEnys2

d3lac avatar Dec 18 '20 10:12 d3lac

Hi, I am also able to reproduce this issue.

my administrator has disabled trans-coding for my profile and I am able to transcode by changing to config property to true in my Linux MPV Shim config ( ~/.config/jellyfin-mpv-shim/conf.json )

ByteOPCode avatar Dec 18 '20 11:12 ByteOPCode

-codec:v:0 copy -codec:a:0 copy means Remuxing(video repackage). mpv shim needs to update the descriptions.

nyanmisaka avatar Dec 18 '20 11:12 nyanmisaka

CC @iwalton3

nielsvanvelzen avatar Dec 18 '20 11:12 nielsvanvelzen

I don’t know that there is much I can do about this. That option drops all of the direct play profiles, but the server can still decide to direct stream if it is in the confines of the bitrate and codec limits.

iwalton3 avatar Dec 18 '20 14:12 iwalton3

The actual play method can be obtained from the session. https://github.com/jellyfin/jellyfin-web/blob/e7521162097fdb5b2731510a634339be9a690add/src/components/playback/playmethodhelper.js#L1-L17

nyanmisaka avatar Dec 18 '20 14:12 nyanmisaka

Ok so I can more accurately display what happens, but there isn’t anything I can do about the “always transcode” option not actually always transcoding.

iwalton3 avatar Dec 18 '20 14:12 iwalton3

Since this option Allow video playback that requires conversion without re-encoding. is checked, video remux is taking place as expected. The user was misled by "Transcode" description.

nyanmisaka avatar Dec 18 '20 15:12 nyanmisaka