Use response dto

This commit is contained in:
Cody Robibero 2021-10-29 06:33:34 -06:00
parent 0e584f6840
commit bcb1c9b652
2 changed files with 26 additions and 4 deletions

View File

@ -98,14 +98,14 @@ namespace Jellyfin.Api.Controllers
/// <response code="200">Document saved.</response>
/// <response code="403">Event logging disabled.</response>
/// <response code="413">Upload size too large.</response>
/// <returns>Created file name.</returns>
/// <returns>Create response.</returns>
[HttpPost("Document")]
[ProducesResponseType(typeof(string), StatusCodes.Status200OK)]
[ProducesResponseType(typeof(ClientLogDocumentResponseDto), StatusCodes.Status200OK)]
[ProducesResponseType(StatusCodes.Status403Forbidden)]
[ProducesResponseType(StatusCodes.Status413PayloadTooLarge)]
[AcceptsFile(MediaTypeNames.Text.Plain)]
[RequestSizeLimit(MaxDocumentSize)]
public async Task<ActionResult<string>> LogFile()
public async Task<ActionResult<ClientLogDocumentResponseDto>> LogFile()
{
if (!_serverConfigurationManager.Configuration.AllowClientLogUpload)
{
@ -123,7 +123,7 @@ namespace Jellyfin.Api.Controllers
var fileName = await _clientEventLogger.WriteDocumentAsync(authorizationInfo, Request.Body)
.ConfigureAwait(false);
return Ok(fileName);
return Ok(new ClientLogDocumentResponseDto(fileName));
}
private void Log(ClientLogEventDto dto, AuthorizationInfo authorizationInfo)

View File

@ -0,0 +1,22 @@
namespace Jellyfin.Api.Models.ClientLogDtos
{
/// <summary>
/// Client log document response dto.
/// </summary>
public class ClientLogDocumentResponseDto
{
/// <summary>
/// Initializes a new instance of the <see cref="ClientLogDocumentResponseDto"/> class.
/// </summary>
/// <param name="filename">The file name.</param>
public ClientLogDocumentResponseDto(string filename)
{
Filename = filename;
}
/// <summary>
/// Gets the resulting filename.
/// </summary>
public string Filename { get; }
}
}