[PR #4935] [MERGED] Remove used quick connect tokens #10333

Closed
opened 2026-02-07 06:18:23 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4935
Author: @ConfusedPolarBear
Created: 1/3/2021
Status: Merged
Merged: 2/8/2021
Merged by: @dkanada

Base: masterHead: quickconnect-cleanup


📝 Commits (1)

  • 8dd8332 Remove quick connect tokens after usage

📊 Changes

1 file changed (+6 additions, -1 deletions)

View changed files

📝 Emby.Server.Implementations/Session/SessionManager.cs (+6 -1)

📄 Description

After using quick connect to login, two device entries are created in the Devices page: one with the server's name labeled QuickConnect, and another for the connecting device. The first entry is used to associate the user id to the login request and the second entry is created by AuthenticateNewSessionInternal().

Changes
After authenticating the incoming quick connect request, delete the initial quick connect token from the authentication repository to avoid adding a second device to the list.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/4935 **Author:** [@ConfusedPolarBear](https://github.com/ConfusedPolarBear) **Created:** 1/3/2021 **Status:** ✅ Merged **Merged:** 2/8/2021 **Merged by:** [@dkanada](https://github.com/dkanada) **Base:** `master` ← **Head:** `quickconnect-cleanup` --- ### 📝 Commits (1) - [`8dd8332`](https://github.com/jellyfin/jellyfin/commit/8dd83327b5b9b77912b3b69ec00dc31898a86bc7) Remove quick connect tokens after usage ### 📊 Changes **1 file changed** (+6 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/Session/SessionManager.cs` (+6 -1) </details> ### 📄 Description After using quick connect to login, two device entries are created in the Devices page: one with the server's name labeled `QuickConnect`, and another for the connecting device. The first entry is used to associate the user id to the login request and the second entry is created by `AuthenticateNewSessionInternal()`. **Changes** After authenticating the incoming quick connect request, delete the initial quick connect token from the authentication repository to avoid adding a second device to the list. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:18:23 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#10333