-
-
Notifications
You must be signed in to change notification settings - Fork 290
Fix API parameter for Jellyfin 10.12 and up #1429
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This will definitely need additional consideration, because I'm on the latest stable Renaming I'll leave this PR up, but let's wait to see if this change makes it into Jellyfin's stable releases first. |
|
It is REALLY strange. I'm going to try and test some more things and see if I can find anything. |
|
So I was taking a look at the jellyfin versions and looks like Some links for reference: |
|
I downgraded my server from 10.12 to 10.11.5 and yeah that does still use |
This popped up just as I finished typing. Wow. |
|
Good find. For this PR, you should pin the version for the feature to be |
|
Fair enough. |
Change 10.11 version check to 10.12
|
I decided to simplify the approach here to just force the new |
Added a version check for Jellyfin servers 10.11+ to use the new "ApiKey" syntax.
Also added a API_KEY_CAMELCASE feature flag for this case.
Marked as draft because I've only tested it on my machine with my server.