aboutsummaryrefslogtreecommitdiff
path: root/Jellyfin.Api/Controllers/SessionController.cs
diff options
context:
space:
mode:
authorCody Robibero <cody@robibe.ro>2024-04-21 10:54:49 -0600
committerGitHub <noreply@github.com>2024-04-21 10:54:49 -0600
commit27fae3dd040b7e0a23f79e74bc1ee4c99595e5b8 (patch)
treec0cd9bac646b02c1cdd2f0ca18400e4d222a52ec /Jellyfin.Api/Controllers/SessionController.cs
parent43569082f9447413ce42cb251fbe528133a9837c (diff)
Limit sessions per user (#11370)
Diffstat (limited to 'Jellyfin.Api/Controllers/SessionController.cs')
-rw-r--r--Jellyfin.Api/Controllers/SessionController.cs8
1 files changed, 7 insertions, 1 deletions
diff --git a/Jellyfin.Api/Controllers/SessionController.cs b/Jellyfin.Api/Controllers/SessionController.cs
index 52b58b8f1..60de66ab0 100644
--- a/Jellyfin.Api/Controllers/SessionController.cs
+++ b/Jellyfin.Api/Controllers/SessionController.cs
@@ -84,7 +84,8 @@ public class SessionController : BaseJellyfinApiController
if (!user.HasPermission(PermissionKind.EnableRemoteControlOfOtherUsers))
{
- result = result.Where(i => i.UserId.IsEmpty() || i.ContainsUser(controllableByUserId.Value));
+ // User cannot control other user's sessions, validate user id.
+ result = result.Where(i => i.UserId.IsEmpty() || i.ContainsUser(RequestHelpers.GetUserId(User, controllableByUserId)));
}
if (!user.HasPermission(PermissionKind.EnableSharedDeviceControl))
@@ -105,6 +106,11 @@ public class SessionController : BaseJellyfinApiController
return true;
});
}
+ else if (!User.IsInRole(UserRoles.Administrator))
+ {
+ // Request isn't from administrator, limit to "own" sessions.
+ result = result.Where(i => i.UserId.IsEmpty() || i.ContainsUser(User.GetUserId()));
+ }
if (activeWithinSeconds.HasValue && activeWithinSeconds.Value > 0)
{