asp.net – MVC [HandleError] HandleErrorAttribute在使用全局日志记录时调用两次

在我正在使用的MVC3 Web应用程序中

public static void RegisterGlobalFilters(GlobalFilterCollection filters)
{
    filters.Add(new HandleErrorAttribute());
}

应用全局错误处理,如果发生未处理的异常,用户将显示“错误”视图.

对于一个特定的视图,如果通过使用[HandleError(View =“SpecialError”)]装饰方法发生未处理的异常,我还想要显示一个不同的错误视图.这工作正常

然后我想添加全局日志记录未处理的异常.我创建了一个具有日志记录代码的自定义HandleError属性：

public class MyHandleErrorAttribute : HandleErrorAttribute
    {
        public override void OnException(ExceptionContext context)
        {
            // Write to log code
            base.OnException(context);
        }
    }

并更新了RegisterGlobalFilters和方法装饰,以改用此属性名称.这通常工作,但是当使用MyHandleError(View =“SpecialError”)装饰的方法中发生异常时,OnException方法被调用两次.我原来假设用这个属性来装饰方法取代了全局处理程序,但是它似乎只是被添加到(这更有意义,但不是我想要的).通过调用OnException两次,同样的异常记录两次,这不会发生.我不认为OnException被调用两次,因为它是一个自定义属性 – 我相信这也发生在标准的HandleError属性,它现在可以看到,因为我正在创建一个记录.

最终,我想记录所有未处理的异常(一次),同时保留[HandleError]提供的功能,特别是为特定的方法异常设置不同的视图.有没有干净的方法呢？

解决方法

我相信我自己找到了一个干净的解决方案.扩展HandleError似乎是一个好主意,但现在我认为这是错误方向的一步.我不想处理任何错误,只是在HandleError接收到之前写入异常记录一次.因此,默认的HandleError可以原样保留.虽然OnException可以被多次调用,但它似乎在HandleErrorAttribute的标准实现中是完全良性的.

而是创建了一个异常记录过滤器：

public class LoggedExceptionFilter : IExceptionFilter
    {
        public void OnException(ExceptionContext filterContext)
        {
            // logging code
        }
    }

它不需要太过继承FilterAttribute,因为它刚刚在RegisterGlobalFilters中与HandleErrorAttribute一起注册.

public static void RegisterGlobalFilters(GlobalFilterCollection filters)
    {
        filters.Add(new LoggedExceptionFilter());
        filters.Add(new HandleErrorAttribute());
    }

这允许在不改变标准[HandleError]功能的情况下整理记录异常