feat: add jellyfin/emby quick connect authentication #2212
+624
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Implements a quick connect authentication flow for jellyfin and emby servers. fix #1595
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds Jellyfin/Emby Quick Connect authentication support, providing users with an alternative PIN-based login method. The implementation includes a modal-based UI flow, backend API endpoints, and proper session management.
Key Changes:
- Frontend modal component for Quick Connect with polling-based authorization checking
- Three new backend API endpoints: initiate, check status, and authenticate
- Integration with existing Jellyfin API client with new Quick Connect methods
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
src/components/Login/JellyfinQuickConnectModal.tsx |
New modal component handling Quick Connect flow with code display and polling |
src/components/Login/JellyfinLogin.tsx |
Added Quick Connect button and modal integration |
server/routes/auth.ts |
Three new endpoints for Quick Connect initiation, status checking, and authentication |
server/api/jellyfin.ts |
Added Quick Connect API methods and TypeScript interfaces |
seerr-api.yml |
OpenAPI documentation for new Quick Connect endpoints |
src/i18n/locale/en.json |
Translation keys for Quick Connect UI elements |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
fallenbagel
force-pushed
the
fallenbagel/feat-jellyfin-quick-connect
branch
from
777f0b7 to
025f6e3
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Adds support for Jellyfin/Emby Quick Connect authentication, providing users with an alternative login method. Jellyfin Quick Connect button is handled as a modal (see screenshots below).
How Has This Been Tested?
Screenshots / Logs (if applicable)
Login Page:
Quick Connect Modal:
Checklist:
pnpm buildpnpm i18n:extract