Browse Source

Better logging

pull/65/head
Sebastian Stehle 9 years ago
parent
commit
0cc16d6ace
  1. 2
      src/Squidex/Controllers/Api/Assets/AssetContentController.cs
  2. 5
      src/Squidex/Pipeline/FileCallbackResultExecutor.cs

2
src/Squidex/Controllers/Api/Assets/AssetContentController.cs

@ -47,7 +47,7 @@ namespace Squidex.Controllers.Api.Assets
{ {
var asset = await assetRepository.FindAssetAsync(id); var asset = await assetRepository.FindAssetAsync(id);
if (asset == null || asset.FileVersion < version) if (asset == null || asset.FileVersion < version || width == 0 || height == 0)
{ {
return NotFound(); return NotFound();
} }

5
src/Squidex/Pipeline/FileCallbackResultExecutor.cs

@ -6,6 +6,7 @@
// All rights reserved. // All rights reserved.
// ========================================================================== // ==========================================================================
using System;
using System.Threading.Tasks; using System.Threading.Tasks;
using Microsoft.AspNetCore.Mvc; using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.Internal; using Microsoft.AspNetCore.Mvc.Internal;
@ -28,8 +29,10 @@ namespace Squidex.Pipeline
await result.Callback(context.HttpContext.Response.Body); await result.Callback(context.HttpContext.Response.Body);
} }
catch catch (Exception e)
{ {
Logger.LogCritical(new EventId(99), e, "Failed to send result.");
context.HttpContext.Response.Headers.Clear(); context.HttpContext.Response.Headers.Clear();
context.HttpContext.Response.StatusCode = 404; context.HttpContext.Response.StatusCode = 404;
} }

Loading…
Cancel
Save