michael.heier/jellyfin
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Fix Authorize attributes

Browse Source
This commit is contained in:
crobibero 2020-04-23 10:03:54 -06:00
parent a06d271725
commit bb8e738a08
1 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
Jellyfin.Api/Controllers/ConfigurationController.cs
View File

@ -1,12 +1,13 @@
#nullable enable #nullable enable
using System.Threading.Tasks; using System.Threading.Tasks;
using Jellyfin.Api.Constants;
using Jellyfin.Api.Models.ConfigurationDtos; using Jellyfin.Api.Models.ConfigurationDtos;
using MediaBrowser.Controller.Configuration; using MediaBrowser.Controller.Configuration;
using MediaBrowser.Controller.MediaEncoding; using MediaBrowser.Controller.MediaEncoding;
using MediaBrowser.Controller.Net;
using MediaBrowser.Model.Configuration; using MediaBrowser.Model.Configuration;
using MediaBrowser.Model.Serialization; using MediaBrowser.Model.Serialization;
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Http; using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.Mvc; using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.ModelBinding; using Microsoft.AspNetCore.Mvc.ModelBinding;
@ -17,7 +18,7 @@ namespace Jellyfin.Api.Controllers
/// Configuration Controller. /// Configuration Controller.
/// </summary> /// </summary>
[Route("System")] [Route("System")]
[Authenticated] [Authorize]
public class ConfigurationController : BaseJellyfinApiController public class ConfigurationController : BaseJellyfinApiController
{ {
private readonly IServerConfigurationManager _configurationManager; private readonly IServerConfigurationManager _configurationManager;
@ -48,7 +49,7 @@ namespace Jellyfin.Api.Controllers
[ProducesResponseType(StatusCodes.Status200OK)] [ProducesResponseType(StatusCodes.Status200OK)]
public ActionResult<ServerConfiguration> GetConfiguration() public ActionResult<ServerConfiguration> GetConfiguration()
{ {
return Ok(_configurationManager.Configuration); return _configurationManager.Configuration;
} }
/// <summary> /// <summary>
@ -57,7 +58,7 @@ namespace Jellyfin.Api.Controllers
/// <param name="configuration">Configuration.</param> /// <param name="configuration">Configuration.</param>
/// <returns>Status.</returns> /// <returns>Status.</returns>
[HttpPost("Configuration")] [HttpPost("Configuration")]
[Authenticated(Roles = "Admin")] [Authorize(Policy = Policies.RequiresElevation)]
[ProducesResponseType(StatusCodes.Status200OK)] [ProducesResponseType(StatusCodes.Status200OK)]
public ActionResult UpdateConfiguration([FromBody, BindRequired] ServerConfiguration configuration) public ActionResult UpdateConfiguration([FromBody, BindRequired] ServerConfiguration configuration)
{ {
@ -74,7 +75,7 @@ namespace Jellyfin.Api.Controllers
[ProducesResponseType(StatusCodes.Status200OK)] [ProducesResponseType(StatusCodes.Status200OK)]
public ActionResult<object> GetNamedConfiguration([FromRoute] string key) public ActionResult<object> GetNamedConfiguration([FromRoute] string key)
{ {
return Ok(_configurationManager.GetConfiguration(key)); return _configurationManager.GetConfiguration(key);
} }
/// <summary> /// <summary>
@ -83,7 +84,7 @@ namespace Jellyfin.Api.Controllers
/// <param name="key">Configuration key.</param> /// <param name="key">Configuration key.</param>
/// <returns>Status.</returns> /// <returns>Status.</returns>
[HttpPost("Configuration/{Key}")] [HttpPost("Configuration/{Key}")]
[Authenticated(Roles = "Admin")] [Authorize(Policy = Policies.RequiresElevation)]
[ProducesResponseType(StatusCodes.Status200OK)] [ProducesResponseType(StatusCodes.Status200OK)]
public async Task<ActionResult> UpdateNamedConfiguration([FromRoute] string key) public async Task<ActionResult> UpdateNamedConfiguration([FromRoute] string key)
{ {
@ -104,11 +105,11 @@ namespace Jellyfin.Api.Controllers
/// </summary> /// </summary>
/// <returns>MetadataOptions.</returns> /// <returns>MetadataOptions.</returns>
[HttpGet("Configuration/MetadataOptions/Default")] [HttpGet("Configuration/MetadataOptions/Default")]
[Authenticated(Roles = "Admin")] [Authorize(Policy = Policies.RequiresElevation)]
[ProducesResponseType(StatusCodes.Status200OK)] [ProducesResponseType(StatusCodes.Status200OK)]
public ActionResult<MetadataOptions> GetDefaultMetadataOptions() public ActionResult<MetadataOptions> GetDefaultMetadataOptions()
{ {
return Ok(new MetadataOptions()); return new MetadataOptions();
} }
/// <summary> /// <summary>
@ -117,7 +118,7 @@ namespace Jellyfin.Api.Controllers
/// <param name="mediaEncoderPath">Media encoder path form body.</param> /// <param name="mediaEncoderPath">Media encoder path form body.</param>
/// <returns>Status.</returns> /// <returns>Status.</returns>
[HttpPost("MediaEncoder/Path")] [HttpPost("MediaEncoder/Path")]
[Authenticated(Roles = "Admin", AllowBeforeStartupWizard = true)] [Authorize(Policy = Policies.FirstTimeSetupOrElevated)]
[ProducesResponseType(StatusCodes.Status200OK)] [ProducesResponseType(StatusCodes.Status200OK)]
public ActionResult UpdateMediaEncoderPath([FromForm, BindRequired] MediaEncoderPathDto mediaEncoderPath) public ActionResult UpdateMediaEncoderPath([FromForm, BindRequired] MediaEncoderPathDto mediaEncoderPath)
{ {