e95ed36647
Phases 1-10 of the planned refactoring:
Phase 1: rename shared-models -> common
- namespace Shared.Models -> Common throughout
- remove stale AspNetCore.Http.Features 5.0 reference
Phase 2: create shared-data with abstract BaseEntity
- BaseEntity: required string Id { get; init; } + DateTime CreatedAt { get; init; }
Phase 3: rename myai-models -> myai-data
- namespace MyAi.Models -> MyAi.Data
- MigrationsAssembly("myai-data")
Phase 4: rename cv-search-models -> cv-search-data
- namespace CvSearch.Models -> CvSearch.Data
- move JobSearchSettings to cv-matcher-api-models
- JobSearch*Entity now inherits BaseEntity
Phase 5: extract rag-data from rag-api
- new project: Apis/rag-data with RagDbContext + entities + migrations
- RagDocumentEntity inherits BaseEntity; cache entities use CacheKey PK
- fix duplicate AddHttpClient<RagAiClient>/AddScoped registrations in rag-api
- MigrationsAssembly("rag-data")
Phase 6: extract cv-matcher-data from cv-matcher-api
- new project: Apis/cv-matcher-data with CvMatcherDbContext + entities + migrations
- CvMatchResultEntity inherits BaseEntity; CvMatcherChatCacheEntity uses CacheKey PK
- MigrationsAssembly("cv-matcher-data")
Phase 7: create empty cv-cleanup-job-models and cv-search-job-models
Phase 8: update all 5 Dockerfiles for renamed/new projects
Phase 9: reorganise .sln virtual folders (Apis/Jobs/Models/Data/Helpers)
- update root CLAUDE.md with new project taxonomy and migration commands
- update cv-matcher-api/CLAUDE.md and cv-search-job/CLAUDE.md
Phase 10: add Directory.Packages.props for centralised NuGet versions
- remove Version= from all PackageReference elements in active .csproj files
No database changes. No runtime behaviour changes.
All MigrationId strings in __EFMigrationsHistory are unaffected.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
221 lines
10 KiB
C#
221 lines
10 KiB
C#
using Models.Settings;
|
|
using Api.Services.Contracts;
|
|
using Microsoft.AspNetCore.Cors;
|
|
using Microsoft.AspNetCore.Mvc;
|
|
using Microsoft.AspNetCore.StaticFiles;
|
|
using Microsoft.Extensions.Options;
|
|
using Microsoft.Net.Http.Headers;
|
|
using Swashbuckle.AspNetCore.Annotations;
|
|
using Common.Responses;
|
|
|
|
namespace Api.Controllers
|
|
{
|
|
/// <summary>
|
|
/// Controller for handling file downloads with support for resume and chunked transfers.
|
|
/// Routes are prefixed with "api/filedownload".
|
|
/// </summary>
|
|
[ApiController]
|
|
[Route("api/[controller]")]
|
|
[EnableCors("FrontendOnly")]
|
|
public sealed class FileDownloadController : ControllerBase
|
|
{
|
|
private readonly ILogger<FileDownloadController> _logger;
|
|
private readonly FileStorageSettings _fileStorageSettings;
|
|
private readonly IContentTypeProvider _contentTypeProvider;
|
|
private readonly IEmailSender _emailSender;
|
|
private const int BufferSize = 81920; // 80 KB buffer for optimal streaming performance
|
|
|
|
public FileDownloadController(
|
|
ILogger<FileDownloadController> logger,
|
|
IOptions<FileStorageSettings> fileStorageSettings,
|
|
IContentTypeProvider contentTypeProvider,
|
|
IEmailSender emailSender)
|
|
{
|
|
_logger = logger;
|
|
_fileStorageSettings = fileStorageSettings.Value;
|
|
_contentTypeProvider = contentTypeProvider;
|
|
_emailSender = emailSender;
|
|
}
|
|
|
|
/// <summary>
|
|
/// Downloads a file with support for resume (range requests) and chunked transfer.
|
|
/// Supports HTTP 206 Partial Content for efficient downloads and resume capability.
|
|
/// Sends email notification when download starts.
|
|
/// </summary>
|
|
/// <param name="fileName">The name of the file to download (optional - uses default from settings if not provided)</param>
|
|
/// <returns>File stream with appropriate headers for resumable downloads</returns>
|
|
/// <response code="200">Full file content</response>
|
|
/// <response code="206">Partial file content (range request)</response>
|
|
/// <response code="404">File not found</response>
|
|
/// <response code="416">Requested range not satisfiable</response>
|
|
[HttpGet("{fileName?}")]
|
|
[SwaggerOperation(Summary = "Download file", Description = "Downloads a file with support for full and ranged (resumable) transfers.")]
|
|
[SwaggerResponse(StatusCodes.Status200OK, "Full file content returned")]
|
|
[SwaggerResponse(StatusCodes.Status206PartialContent, "Partial file content returned for a range request")]
|
|
[SwaggerResponse(StatusCodes.Status400BadRequest, "No file name provided and no default configured")]
|
|
[SwaggerResponse(StatusCodes.Status404NotFound, "Requested file was not found")]
|
|
[SwaggerResponse(StatusCodes.Status416RangeNotSatisfiable, "Requested byte range is invalid")]
|
|
[SwaggerResponse(StatusCodes.Status500InternalServerError, "Unexpected server error while downloading")]
|
|
[ProducesResponseType(StatusCodes.Status200OK)]
|
|
[ProducesResponseType(StatusCodes.Status206PartialContent)]
|
|
[ProducesResponseType(typeof(ErrorResponse), StatusCodes.Status400BadRequest)]
|
|
[ProducesResponseType(typeof(ErrorResponse), StatusCodes.Status404NotFound)]
|
|
[ProducesResponseType(StatusCodes.Status416RangeNotSatisfiable)]
|
|
[ProducesResponseType(typeof(ErrorResponse), StatusCodes.Status500InternalServerError)]
|
|
public async Task<IActionResult> DownloadFile(string? fileName = null)
|
|
{
|
|
try
|
|
{
|
|
if (string.IsNullOrWhiteSpace(fileName))
|
|
{
|
|
fileName = _fileStorageSettings.DefaultFileName;
|
|
|
|
if (string.IsNullOrWhiteSpace(fileName))
|
|
{
|
|
_logger.LogWarning("No file name provided and no default file name configured");
|
|
return BadRequest(new ErrorResponse { Error = "File name is required", Code = "file_name_required" });
|
|
}
|
|
|
|
_logger.LogInformation("Using default file name from settings: {FileName}", fileName);
|
|
}
|
|
|
|
var fileStoragePath = _fileStorageSettings.Path;
|
|
|
|
if (!Path.IsPathRooted(fileStoragePath))
|
|
{
|
|
var solutionRoot = Directory.GetCurrentDirectory();
|
|
if (solutionRoot.EndsWith("api", StringComparison.OrdinalIgnoreCase))
|
|
solutionRoot = Directory.GetParent(solutionRoot)?.FullName ?? solutionRoot;
|
|
fileStoragePath = Path.Combine(solutionRoot, fileStoragePath);
|
|
}
|
|
|
|
var sanitizedFileName = Path.GetFileName(fileName);
|
|
var filePath = Path.Combine(fileStoragePath, sanitizedFileName);
|
|
|
|
if (!System.IO.File.Exists(filePath))
|
|
{
|
|
_logger.LogWarning("File not found: {FilePath}", filePath);
|
|
return NotFound(new ErrorResponse { Error = "File not found", Code = "file_not_found" });
|
|
}
|
|
|
|
var fileLength = new FileInfo(filePath).Length;
|
|
|
|
if (!_contentTypeProvider.TryGetContentType(filePath, out var contentType))
|
|
contentType = "application/octet-stream";
|
|
|
|
var userIp = HttpContext.Connection.RemoteIpAddress?.ToString();
|
|
_ = Task.Run(async () =>
|
|
{
|
|
try
|
|
{
|
|
await _emailSender.SendFileDownloadNotificationAsync(sanitizedFileName, userIp, CancellationToken.None);
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
_logger.LogError(ex, "Failed to send file download notification for {FileName}", sanitizedFileName);
|
|
}
|
|
});
|
|
|
|
var rangeHeader = Request.Headers[HeaderNames.Range].ToString();
|
|
if (!string.IsNullOrEmpty(rangeHeader))
|
|
return await HandleRangeRequest(filePath, fileLength, contentType, rangeHeader, sanitizedFileName);
|
|
|
|
_logger.LogInformation("Starting full file download: {FileName} ({FileSize} bytes)", sanitizedFileName, fileLength);
|
|
|
|
var stream = new FileStream(filePath, FileMode.Open, FileAccess.Read, FileShare.Read, BufferSize, useAsync: true);
|
|
Response.Headers.Append(HeaderNames.AcceptRanges, "bytes");
|
|
Response.Headers.Append(HeaderNames.ContentLength, fileLength.ToString());
|
|
|
|
return File(stream, contentType, sanitizedFileName, enableRangeProcessing: true);
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
_logger.LogError(ex, "Error downloading file: {FileName}", fileName);
|
|
return StatusCode(StatusCodes.Status500InternalServerError, new ErrorResponse { Error = "An error occurred while downloading the file", Code = "download_failed" });
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Handles HTTP range requests for partial content downloads and resume support.
|
|
/// </summary>
|
|
private async Task<IActionResult> HandleRangeRequest(
|
|
string filePath,
|
|
long fileLength,
|
|
string contentType,
|
|
string rangeHeader,
|
|
string fileName)
|
|
{
|
|
try
|
|
{
|
|
var range = rangeHeader.Replace("bytes=", "").Split('-');
|
|
|
|
long startByte = 0;
|
|
long endByte = fileLength - 1;
|
|
|
|
if (!string.IsNullOrEmpty(range[0]))
|
|
startByte = long.Parse(range[0]);
|
|
|
|
if (range.Length > 1 && !string.IsNullOrEmpty(range[1]))
|
|
endByte = long.Parse(range[1]);
|
|
|
|
if (startByte > endByte || startByte >= fileLength)
|
|
{
|
|
_logger.LogWarning("Invalid range request: {Range} for file size {FileLength}", rangeHeader, fileLength);
|
|
return StatusCode(StatusCodes.Status416RangeNotSatisfiable);
|
|
}
|
|
|
|
if (endByte >= fileLength)
|
|
endByte = fileLength - 1;
|
|
|
|
var contentLength = endByte - startByte + 1;
|
|
|
|
_logger.LogInformation(
|
|
"Range request for {FileName}: bytes {Start}-{End}/{Total} ({ContentLength} bytes)",
|
|
fileName, startByte, endByte, fileLength, contentLength);
|
|
|
|
var stream = new FileStream(filePath, FileMode.Open, FileAccess.Read, FileShare.Read, BufferSize, useAsync: true);
|
|
stream.Seek(startByte, SeekOrigin.Begin);
|
|
|
|
Response.StatusCode = StatusCodes.Status206PartialContent;
|
|
Response.Headers.Append(HeaderNames.AcceptRanges, "bytes");
|
|
Response.Headers.Append(HeaderNames.ContentRange, $"bytes {startByte}-{endByte}/{fileLength}");
|
|
Response.Headers.Append(HeaderNames.ContentLength, contentLength.ToString());
|
|
Response.ContentType = contentType;
|
|
|
|
await StreamRangeAsync(stream, Response.Body, contentLength);
|
|
await stream.DisposeAsync();
|
|
|
|
return new EmptyResult();
|
|
}
|
|
catch (Exception ex)
|
|
{
|
|
_logger.LogError(ex, "Error processing range request for file: {FileName}", fileName);
|
|
return StatusCode(StatusCodes.Status500InternalServerError);
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Efficiently streams a specific byte range from source to destination.
|
|
/// </summary>
|
|
private static async Task StreamRangeAsync(Stream source, Stream destination, long bytesToRead)
|
|
{
|
|
var buffer = new byte[BufferSize];
|
|
long totalBytesRead = 0;
|
|
|
|
while (totalBytesRead < bytesToRead)
|
|
{
|
|
var bytesToReadThisIteration = (int)Math.Min(BufferSize, bytesToRead - totalBytesRead);
|
|
var bytesRead = await source.ReadAsync(buffer.AsMemory(0, bytesToReadThisIteration));
|
|
|
|
if (bytesRead == 0)
|
|
break;
|
|
|
|
await destination.WriteAsync(buffer.AsMemory(0, bytesRead));
|
|
totalBytesRead += bytesRead;
|
|
}
|
|
|
|
await destination.FlushAsync();
|
|
}
|
|
}
|
|
}
|