From 209edd38a4163a8cf4abd5e47bfe0ea1a100f351 Mon Sep 17 00:00:00 2001 From: cvium Date: Wed, 8 Feb 2023 23:55:26 +0100 Subject: refactor: simplify authz --- .../FirstTimeSetupRequirement.cs | 26 ++++++++++++++++++++++ 1 file changed, 26 insertions(+) create mode 100644 Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs (limited to 'Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs') diff --git a/Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs b/Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs new file mode 100644 index 000000000..8b7a94954 --- /dev/null +++ b/Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs @@ -0,0 +1,26 @@ +using Jellyfin.Api.Auth.DefaultAuthorizationPolicy; + +namespace Jellyfin.Api.Auth.FirstTimeSetupPolicy +{ + /// + /// The authorization requirement, requiring incomplete first time setup or default privileges, for the authorization handler. + /// + public class FirstTimeSetupRequirement : DefaultAuthorizationRequirement + { + /// + /// Initializes a new instance of the class. + /// + /// A value indicating whether to ignore parental schedule. + /// A value indicating whether administrator role is required. + public FirstTimeSetupRequirement(bool validateParentalSchedule = false, bool requireAdmin = true) + { + ValidateParentalSchedule = validateParentalSchedule; + RequireAdmin = requireAdmin; + } + + /// + /// Gets a value indicating whether administrator role is required. + /// + public bool RequireAdmin { get; } + } +} -- cgit v1.2.3 From ac118e10f04de5489e6d0ac17c8ca16bdf6d0dc3 Mon Sep 17 00:00:00 2001 From: cvium Date: Thu, 9 Feb 2023 15:01:04 +0100 Subject: remove unnecessary init --- .../Auth/DefaultAuthorizationPolicy/DefaultAuthorizationRequirement.cs | 2 +- Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) (limited to 'Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs') diff --git a/Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationRequirement.cs b/Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationRequirement.cs index 0846e7515..5ba1bc330 100644 --- a/Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationRequirement.cs +++ b/Jellyfin.Api/Auth/DefaultAuthorizationPolicy/DefaultAuthorizationRequirement.cs @@ -19,6 +19,6 @@ namespace Jellyfin.Api.Auth.DefaultAuthorizationPolicy /// /// Gets a value indicating whether to ignore parental schedule. /// - public bool ValidateParentalSchedule { get; init; } + public bool ValidateParentalSchedule { get; } } } diff --git a/Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs b/Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs index 8b7a94954..6252a2feb 100644 --- a/Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs +++ b/Jellyfin.Api/Auth/FirstTimeSetupPolicy/FirstTimeSetupRequirement.cs @@ -12,9 +12,8 @@ namespace Jellyfin.Api.Auth.FirstTimeSetupPolicy /// /// A value indicating whether to ignore parental schedule. /// A value indicating whether administrator role is required. - public FirstTimeSetupRequirement(bool validateParentalSchedule = false, bool requireAdmin = true) + public FirstTimeSetupRequirement(bool validateParentalSchedule = false, bool requireAdmin = true) : base(validateParentalSchedule) { - ValidateParentalSchedule = validateParentalSchedule; RequireAdmin = requireAdmin; } -- cgit v1.2.3