ASP.NET MVC全局错误处理

时间:2012-02-27 07:05:20

标签: c# asp.net-mvc-3 error-handling

我有一个自定义HandleError属性来处理MVC管道上的错误;我的protected void Application_Error(object sender, EventArgs e)上有Global.asax方法处理来自管道外部的错误。

我遇到过一个我不知道可能的情景;在实现DI时,connectionString存在依赖关系,它取自应用程序配置文件。

由于连接字符串尚不存在,因此在创建控制器时会出错,这通常会使Application_Error处理程序触发,并呈现正确的错误页面(通过将部分视图呈现为字符串并发送它作为响应,如果失败,它只会将“致命异常”写入响应。

除非在这种情况下,我得到了虚假的默认ASP.NET“运行时错误”黄色死亡屏幕。告诉我:

  

运行时错误

     

描述:发生了应用程序错误   服务器。此应用程序的当前自定义错误设置阻止   查看应用程序错误的详细信息。

     

详细信息:要启用此特定错误消息的详细信息   在本地服务器机器上可以查看,请创建一个   位于根目录中的“web.config”配置文件中的标记   当前Web应用程序的目录。这个标签   然后应将其“mode”属性设置为“RemoteOnly”。启用   要在远程机器上查看的详细信息,请将“模式”设置为   “关”。

我的defaultRedirect中没有设置customErrors,也没有Off,因为我不想重定向但是在同一页面上呈现错误用户在,避免不必要的重定向。

我该如何处理这样的场景?甚至是什么原因导致它以这种方式运行而不像控制器之外的任何其他错误?

我意识到它不太可能经常发生,但我希望能够阻止YSOD(部分原因是因为我想隐藏我正在使用的技术,但主要是因为它不是很漂亮,也不是用户友好的)

我甚至尝试为UnhandledExceptions注册一个处理程序,但它也没有触发。

AppDomain.CurrentDomain.UnhandledException += CurrentDomain_UnhandledException;

最终产生此代码的代码是:

return ConfigurationManager.ConnectionStrings[key].ConnectionString;,其中ConnectionStrings[key]null

更新

这就是处理应用程序错误的方法:

    protected void Application_Error(object sender, EventArgs e)
    {
        this.HandleApplicationError(new ResourceController());
    }

    public static void HandleApplicationError(this HttpApplication application, BaseController controller)
    {
        if (application == null)
        {
            throw new ArgumentNullException("application");
        }
        if (controller == null)
        {
            throw new ArgumentNullException("controller");
        }
        application.Response.Clear();
        Exception exception = application.Server.GetLastError();
        LogApplicationException(application.Response, exception);
        try
        {
            RenderExceptionViewResponse(application, exception, controller);
        }
        catch (Exception exceptionRenderingView) // now we're in trouble. let's be as graceful as possible.
        {
            RenderExceptionTextResponse(application, exceptionRenderingView);
        }
        finally
        {
            application.Server.ClearError();
        }
    }

    private static void LogApplicationException(HttpResponse response, Exception exception)
    {
        if (exception is HttpException)
        {
            HttpException httpException = (HttpException)exception;
            if (httpException.GetHttpCode() == (int)HttpStatusCode.NotFound)
            {
                _log.Debug(Resources.Error.WebResourceNotFound, httpException);
                response.Status = Resources.Constants.NotFound;
                return;
            }
        }
        _log.Error(Resources.Error.UnhandledException, exception);
    }

    private static void RenderExceptionViewResponse(HttpApplication application, Exception exception, BaseController controller)
    {
        if (!RenderAsJsonResponse(application, Resources.User.UnhandledExceptionJson))
        {
            ErrorViewModel model = WebUtility.GetErrorViewModel(exception);
            string result = controller.RenderViewToString(Resources.Constants.ErrorViewName, model);
            application.Response.Write(result);
        }
    }

    private static void RenderExceptionTextResponse(HttpApplication application, Exception exceptionRenderingView)
    {
        application.Response.Clear();

        if (!RenderAsJsonResponse(application, Resources.User.FatalExceptionJson))
        {
            application.Response.Write(Resources.User.FatalException);
        }
        _log.Fatal(Resources.Error.FatalException, exceptionRenderingView);
    }

    private static bool RenderAsJsonResponse(HttpApplication application, string message)
    {
        if (application.Request.IsAjaxRequest())
        {
            application.Response.Status = Resources.Constants.HttpSuccess;
            application.Response.ContentType = Resources.Constants.JsonContentType;
            application.Response.Write(message);
            return true;
        }
        return false;
    }

这是我用来装饰我的基本控制器的属性:

public class ErrorHandlingAttribute : HandleErrorAttribute
{
    public Type LoggerType { get; set; }

    public ErrorHandlingAttribute()
        : this(typeof(ErrorHandlingAttribute))
    {
    }

    public ErrorHandlingAttribute(Type loggerType)
    {
        LoggerType = loggerType;
    }

    public override void OnException(ExceptionContext filterContext)
    {
        if (filterContext.ExceptionHandled)
        {
            return;
        }
        if (filterContext.HttpContext.Request.IsAjaxRequest())
        {
            OnAjaxException(filterContext);
        }
        else
        {
            OnRegularException(filterContext);
        }
    }

    internal protected void OnRegularException(ExceptionContext filterContext)
    {
        Exception exception = filterContext.Exception;

        ILog logger = LogManager.GetLogger(LoggerType);
        logger.Error(Resources.Error.UnhandledException, exception);

        filterContext.HttpContext.Response.Clear();

        ErrorViewModel model = WebUtility.GetErrorViewModel(exception);
        filterContext.Result = new ViewResult
        {
            ViewName = Resources.Constants.ErrorViewName,
            ViewData = new ViewDataDictionary(model)
        };
        filterContext.ExceptionHandled = true;
    }

    internal protected void OnAjaxException(ExceptionContext filterContext)
    {
        Exception exception = filterContext.Exception;

        ILog logger = LogManager.GetLogger(LoggerType);
        logger.Error(Resources.Error.UnhandledAjaxException, exception);

        filterContext.HttpContext.Response.Clear();
        filterContext.HttpContext.Response.Status = Resources.Constants.HttpSuccess;

        string errorMessage = WebUtility.GetUserExceptionMessage(exception, true);

        filterContext.Result = new ExceptionJsonResult(new[] { errorMessage });
        filterContext.ExceptionHandled = true;
    }
}

这是我的customErrors

<customErrors mode="On" />

正如您所看到的那些非常广泛,但是在访问ConnectionStrings不存在ConnectionString的情况下它们甚至不会触发;这有点令人费解。

在任何控制器中包含异常或不在控制器内的异常时触发,所以我不明白为什么这种情况有所不同。

3 个答案:

答案 0 :(得分:7)

Application_Error可能是处理ASP.NET WebForms中错误的推荐方法。但不是在MVC中。

我们有错误过滤器来处理我们的错误。过滤器的问题是它只在调用控制器时才有效。 404和401错误(未找到和授权)以及您的数据库连接问题是哪个问题。

customErrors是去这里的方式。我不明白为什么重定向应该是一个问题?

我正在博客文章中进行正确的错误处理:http://blog.gauffin.org/2011/11/how-to-handle-errors-in-asp-net-mvc/

答案 1 :(得分:0)

您尚未显示您在Application_Error事件中处理错误的确切方式,也未显示您的自定义HandleAttribute是如何实现的,因此很难猜出问题可能是什么。实现Application_Error时需要注意的一个常见问题是,您呈现的错误视图本身会引发错误。例如,假设一个依赖于布局的错误视图,在这个布局中,你调用一个Html.Action帮助器来呈现另一个本身进行数据库访问和操作的动作的内容。您的错误视图应尽可能保持静态。理想情况下,您应该为它们设置不同的布局以避免这种情况。

我可能还建议您查看following approach处理错误。

答案 2 :(得分:0)

为了纠正我们的访问者和搜索引擎,我们应该在发生错误时返回有意义的HTTP状态代码,面向我们的网站。即使在我们返回视图的同时,解释发生错误,在错误时返回HTTP状态代码200也是不公平的。如果用户键入了错误的地址(最常见的用户错误),我们应该返回HTTP状态代码404而不返回或重定向到View,其中将返回状态代码200。
有一个简短而简洁的建议摘要HERE