Web API 2 - 不使用TaskCancellation的全局异常处理程序

时间:2015-09-14 13:26:55

标签: c# asp.net-web-api async-await

用于演示问题的代码:

  • 假设Test Thing是一个真正的实现,例如DB调用。
  • 无法处理异常的documentation for Web API states将在全局处理程序中捕获,允许您处理它们。
  • 如果我用ExceptionFilter替换MyErrorHandler,这确实有效,除了我正在使用的代码库使用处理程序,因为错误逻辑是一个交叉问题,无论错误来自何处都是相同的。
  • 如果抛出的异常类型不是TaskCancelledException,则会按预期调用处理程序。
  • 我也尝试过最新版本的Web API(5.2.3)。
  • 唯一的解决方法是在任何可以抛出此类异常的地方添加一个try / catch块,不用说这很痛苦,我希望避免使用处理程序。

我讨厌将这称为错误,因为这不是我的代码,但经过数小时的尝试,它开始有这样的感觉。

using System;
using System.Net;
using System.Threading;
using System.Threading.Tasks;
using System.Web.Http;
using System.Web.Http.ExceptionHandling;

namespace WebApplication3.Controllers
{
    public class TestController : ApiController
    {
        public async Task<string> Get()
        {
            var cancellationTokenSource = new CancellationTokenSource(TimeSpan.FromSeconds(1));
            return await new TestThing().ExecuteAsync(cancellationTokenSource.Token);
        }
    }

    public class MyErrorHandler : ExceptionHandler
    {
        public override Task HandleAsync(ExceptionHandlerContext context, CancellationToken cancellationToken)
        {
            throw new HttpResponseException(HttpStatusCode.InternalServerError);
        }
    }


    public class TestThing
    {
        public async Task<string> ExecuteAsync(CancellationToken cancellationToken)
        {
            // Remove this to see the problem, I don't want to add these
            // try/catch statements all over the codebase.
            try
            {
                await Task.Delay(TimeSpan.FromMinutes(1), cancellationToken);
            }
            catch (Exception ex)
            {
                throw new Exception("Failure...");
            }

            return await Task.FromResult("Testing...");
        }
    }
}

2 个答案:

答案 0 :(得分:1)

由于缺乏建议或答案,我选择了自定义消息处理程序。

    public class AsyncFixHandler : DelegatingHandler
    {
        protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
        {
            try
            {
                return await base.SendAsync(request, cancellationToken);
            }
            catch (TaskCanceledException)
            {
                // TODO: Log the issue here
                return new HttpResponseMessage(HttpStatusCode.InternalServerError);
            }
        }
    }

这不太理想,但是try / catch在一个地方。我成功地使用它作为解决方案,直到出现更好的东西。

答案 1 :(得分:1)

这似乎是旧的错误,还有一些人仍在报告问题。我建议您在ASP.NET Github repo中创建一个问题,暂时使用您的解决方法或another one