TekH
28df3f4ec1
Refactored the handling of `senderDoc.ByteData` by replacing the ternary operator with an explicit `if` statement for better readability. Added a `Content-Disposition` header to ensure the file is displayed inline with a proper filename. Updated the MIME type of the file response from `application/octet-stream` to `application/pdf` to reflect the expected content type.
97 lines
3.6 KiB
C#
97 lines
3.6 KiB
C#
using DigitalData.Auth.Claims;
|
|
using EnvelopeGenerator.API.Controllers.Interfaces;
|
|
using EnvelopeGenerator.API.Extensions;
|
|
using EnvelopeGenerator.Application.Documents.Queries;
|
|
using EnvelopeGenerator.Domain.Constants;
|
|
using MediatR;
|
|
using Microsoft.AspNetCore.Authorization;
|
|
using Microsoft.AspNetCore.Mvc;
|
|
using System.Threading.Channels;
|
|
|
|
namespace EnvelopeGenerator.API.Controllers;
|
|
|
|
/// <summary>
|
|
/// Provides access to envelope documents for authenticated receivers.
|
|
/// </summary>
|
|
/// <remarks>
|
|
/// Initializes a new instance of the <see cref="DocumentController"/> class.
|
|
/// </remarks>
|
|
[Authorize]
|
|
[ApiController]
|
|
[Route("api/[controller]")]
|
|
public class DocumentController(IMediator mediator, IAuthorizationService authService, ILogger<DocumentController> logger) : ControllerBase, IAuthController
|
|
{
|
|
/// <summary>
|
|
///
|
|
/// </summary>
|
|
public IAuthorizationService AuthService => authService;
|
|
|
|
/// <summary>
|
|
/// Returns the document bytes receiver.
|
|
/// </summary>
|
|
/// <param name="query">Encoded envelope key.</param>
|
|
/// <param name="cancel">Cancellation token.</param>
|
|
[HttpGet]
|
|
[Authorize(Policy = AuthPolicy.SenderOrReceiver)]
|
|
public async Task<IActionResult> GetDocument(CancellationToken cancel, [FromQuery] ReadDocumentQuery? query = null)
|
|
{
|
|
// Sender: expects query with envelope key
|
|
if (await this.IsUserInPolicyAsync(AuthPolicy.Sender))
|
|
{
|
|
if (query is null)
|
|
return BadRequest("Missing document query.");
|
|
|
|
var senderDoc = await mediator.Send(query, cancel);
|
|
return senderDoc.ByteData is byte[] senderDocByte
|
|
? File(senderDocByte, "application/octet-stream")
|
|
: NotFound("Document is empty.");
|
|
}
|
|
|
|
// Receiver: resolve envelope id from claims
|
|
if (await this.IsUserInPolicyAsync(AuthPolicy.Receiver))
|
|
{
|
|
if (query is not null)
|
|
return BadRequest("Query parameters are not allowed for receiver role.");
|
|
|
|
var envelopeId = User.GetEnvelopeIdOfReceiver();
|
|
var receiverDoc = await mediator.Send(new ReadDocumentQuery { EnvelopeId = envelopeId }, cancel);
|
|
return receiverDoc.ByteData is byte[] receiverDocByte
|
|
? File(receiverDocByte, "application/octet-stream")
|
|
: NotFound("Document is empty.");
|
|
}
|
|
|
|
return Unauthorized();
|
|
}
|
|
|
|
/// <summary>
|
|
///
|
|
/// </summary>
|
|
/// <param name="envelopeKey"></param>
|
|
/// <param name="cancel"></param>
|
|
/// <returns></returns>
|
|
[HttpGet("{envelopeKey}")]
|
|
[Authorize(Policy = AuthPolicy.Receiver)]
|
|
public async Task<IActionResult> GetDocumentOfReceiver(string envelopeKey, CancellationToken cancel)
|
|
{
|
|
var envelopeIdStr = User.FindFirst(EnvelopeClaimNames.EnvelopeId)?.Value;
|
|
|
|
if (!int.TryParse(envelopeIdStr, out int envelopeId))
|
|
{
|
|
logger.LogError(
|
|
"Inner service error: Failed to parse Envelope ID from claims. Claim '{ClaimName}' had an invalid or missing value: '{ClaimValue}'.",
|
|
EnvelopeClaimNames.EnvelopeId,
|
|
envelopeIdStr ?? "null");
|
|
|
|
return StatusCode(StatusCodes.Status500InternalServerError, "Inner service error: Invalid envelope claim.");
|
|
}
|
|
|
|
var senderDoc = await mediator.Send(new ReadDocumentQuery() { EnvelopeId = envelopeId }, cancel);
|
|
|
|
if (senderDoc.ByteData is not byte[] senderDocByte)
|
|
return NotFound("Document is empty.");
|
|
|
|
Response.Headers.ContentDisposition = $"inline; filename=\"{envelopeKey}.pdf\"";
|
|
return File(senderDocByte, "application/pdf");
|
|
}
|
|
}
|