diff options
| author | Cody Robibero <cody@robibe.ro> | 2026-09-05 10:57:55 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2026-09-05 10:57:55 -0400 |
| commit | 2b0ccf2a3010bab8b4bae2085ab7ececddb157c0 (patch) | |
| tree | 2e7b09841ac2fe771e37ff405ef822d11449e8ae /Emby.Server.Implementations | |
| parent | d4a23cd438b68ea22869bfbd67c626d7746d5c2a (diff) | |
| parent | 51a7d5d08a3dc00ec3c83f4491be0436a4098096 (diff) | |
Merge pull request #17800 from fmarcac/fix/syncplay-active-session-counter
Fix SyncPlay active session counter leaking on rejoin
Diffstat (limited to 'Emby.Server.Implementations')
| -rw-r--r-- | Emby.Server.Implementations/SyncPlay/SyncPlayManager.cs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/Emby.Server.Implementations/SyncPlay/SyncPlayManager.cs b/Emby.Server.Implementations/SyncPlay/SyncPlayManager.cs index b45d754554..a2b9088708 100644 --- a/Emby.Server.Implementations/SyncPlay/SyncPlayManager.cs +++ b/Emby.Server.Implementations/SyncPlay/SyncPlayManager.cs @@ -181,8 +181,8 @@ namespace Emby.Server.Implementations.SyncPlay { if (existingGroup.GroupId.Equals(request.GroupId)) { - // Restore session. - UpdateSessionsCounter(session.UserId, 1); + // Restore session. The session is already in the group and has already + // been counted, so the counter must not be incremented a second time. group.SessionJoin(session, request, cancellationToken); return; } @@ -400,7 +400,7 @@ namespace Emby.Server.Implementations.SyncPlay // Update sessions counter. var newSessionsCounter = _activeUsers.AddOrUpdate( userId, - 1, + toAdd, (_, sessionsCounter) => sessionsCounter + toAdd); // Should never happen. |