Skip to content

Conversation

@herrrta
Copy link

@herrrta herrrta commented Dec 20, 2025

Description

  • updates jellyfin auth endpoints to:
    • persist jellyfin access token along user login
    • optionally allow login with existing jellyifn access token to prevent exceeding any admin defined user session limits
  • updates auth middleware to also acccept api keys for jellyfin user auth
  • adds Ability to use jellyfin access token for user authentication #2243

How Has This Been Tested?

Ran dev locally and tested endpoints using postman. I made sure i can login and use the provided jellyfinAuthToken in the login response.

I used this auth token to update my X-API-Key header to then access /api/v1/auth/me while also ensuring cookies were empty to make sure this authenticated using only the key. It properly authenticated to the right user each time.

Screenshots / Logs (if applicable)

image image

Checklist:

  • I have read and followed the contribution guidelines.
  • Disclosed any use of AI (see our policy)
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • Successful build pnpm build
  • Translation keys pnpm i18n:extract
  • Database migration (if required)

@herrrta herrrta requested a review from a team as a code owner December 20, 2025 14:29
@herrrta herrrta force-pushed the herrrta/feat/persist-jellyfin-token branch 3 times, most recently from 7cca6cd to 5c8772c Compare December 20, 2025 15:53
@herrrta herrrta changed the title feat: ability to use jellyfin access token for authentication feat: ability to use jellyfin access token for user authentication Dec 20, 2025
@herrrta herrrta force-pushed the herrrta/feat/persist-jellyfin-token branch from 5c8772c to 9900c53 Compare December 20, 2025 16:20
@herrrta herrrta force-pushed the herrrta/feat/persist-jellyfin-token branch 3 times, most recently from bea3b1e to ca29914 Compare December 29, 2025 14:08
@herrrta herrrta force-pushed the herrrta/feat/persist-jellyfin-token branch from ca29914 to b94eb36 Compare December 29, 2025 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant