jellyfin-media-player icon indicating copy to clipboard operation
jellyfin-media-player copied to clipboard

JMP 1.11.1 removes trailing slash which makes it unable to connect to my Jellyfin server

Open fredguile opened this issue 5 months ago • 0 comments

Describe the bug On my shared Jellyfin server, everything is secured via basic auth with exception of that URL "/username/jellyfin/" - pay attention to that trailing slash - and sadly I don't have any way to change that (hint: it's a well-known service used by many).

As since this commit, JMP 1.11.1 will strip out the trailing slash, it makes it unable to connect, as basic auth isn't supported AFAIK.

I would like to open a tiny fix PR touching "find-webclient.js" in order to unblock this.

To Reproduce You need to assume that your Jellyfin server is secured with basic auth on pretty much every URL except "/jellyfin/*" (that trailing slash is of a utmost importance).

Steps to reproduce the behavior:

  1. Just log-in as usual, the login process wouldn't succeed as failing on a 401 auth required as seen in the dev tools

Expected behavior We could just not slash that trailing slash, this would solve the issue.

Desktop (please complete the following information):

  • not OS-specific
  • I've been testing a fix on version 1.11.1 and linux, I can open a fix PR shortly

Additional context That bug seems low priority, but it probably affects many JMP users utilizing a well-known app sharing provider.

fredguile avatar Sep 02 '24 13:09 fredguile