aboutsummaryrefslogtreecommitdiff
path: root/Jellyfin.Api/Auth
diff options
context:
space:
mode:
authorCody Robibero <cody@robibe.ro>2023-02-27 05:48:37 -0700
committerGitHub <noreply@github.com>2023-02-27 05:48:37 -0700
commit4873d2a54db7327b3cc021cc587d31d932b57f2e (patch)
treedc0a8614434c50244d69a7a2c1dc852ea1e3eb9e /Jellyfin.Api/Auth
parentedc627fd5b5a5bd19c843dd9e2970b1ebce3fbfd (diff)
Fix auth endpoints using api key (#9408)
Diffstat (limited to 'Jellyfin.Api/Auth')
-rw-r--r--Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationHandler.cs9
1 files changed, 8 insertions, 1 deletions
diff --git a/Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationHandler.cs b/Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationHandler.cs
index b1d97e4a1..de271ab64 100644
--- a/Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationHandler.cs
+++ b/Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationHandler.cs
@@ -46,6 +46,13 @@ namespace Jellyfin.Api.Auth.DefaultAuthorizationPolicy
return Task.CompletedTask;
}
+ if (isApiKey)
+ {
+ // Api keys are unrestricted.
+ context.Succeed(requirement);
+ return Task.CompletedTask;
+ }
+
var isInLocalNetwork = _httpContextAccessor.HttpContext is not null
&& _networkManager.IsInLocalNetwork(_httpContextAccessor.HttpContext.GetNormalizedRemoteIp());
var user = _userManager.GetUserById(userId);
@@ -62,7 +69,7 @@ namespace Jellyfin.Api.Auth.DefaultAuthorizationPolicy
}
// Admins can do everything
- if (isApiKey || context.User.IsInRole(UserRoles.Administrator))
+ if (context.User.IsInRole(UserRoles.Administrator))
{
context.Succeed(requirement);
return Task.CompletedTask;