Ведение журнала ошибок в ASP.NET MVC

Question

В настоящее время я использую log4net в своем приложении ASP.NET MVC для регистрации исключений. Я делаю это, когда все мои контроллеры наследуются от класса BaseController. В событии OnActionExecuting BaseController я регистрирую любые исключения, которые могли произойти:

protected override void OnActionExecuted(ActionExecutedContext filterContext)
{
    // Log any exceptions
    ILog log = LogManager.GetLogger(filterContext.Controller.GetType());

    if (filterContext.Exception != null)
    {
        log.Error("Unhandled exception: " + filterContext.Exception.Message +
            ". Stack trace: " + filterContext.Exception.StackTrace, 
            filterContext.Exception);
    }
}

Это прекрасно работает, если во время действия контроллера произошло необработанное исключение.

Что касается 404 ошибок, в моем файле web.config настроена ошибка, например:

<customErrors mode="On">
    <error statusCode="404" redirect="~/page-not-found"/>
</customErrors>

И в действии контроллера, которое обрабатывает URL «страница не найдена», я записываю исходный запрашиваемый URL:

[AcceptVerbs(HttpVerbs.Get)]
public ActionResult PageNotFound()
{
    log.Warn("404 page not found - " + Utils.SafeString(Request.QueryString["aspxerrorpath"]));

    return View();
}

И это тоже работает.

Проблема, с которой я сталкиваюсь, заключается в том, как регистрировать ошибки, которые находятся на самих страницах .aspx. Допустим, у меня есть ошибка компиляции на одной из страниц или некоторый встроенный код, который выдает исключение:

<% ThisIsNotAValidFunction(); %>
<% throw new Exception("help!"); %>

Кажется, что атрибут HandleError правильно перенаправляет это на мою страницу Error.aspx в папке Shared, но он определенно не перехватывается методом OnActionExecuted моего BaseController. Я думал, что, возможно, смогу разместить код регистрации на самой странице Error.aspx, но я не уверен, как получить информацию об ошибке на этом уровне.

Answer 1

Я бы рассмотрел упрощение вашего веб-приложения, подключив Elmah .

Вы добавляете сборку Elmah в свой проект, а затем настраиваете свой web.config. Затем он будет регистрировать исключения, созданные на уровне контроллера или страницы. Он может быть настроен для входа в различные места (например, SQL Server, электронная почта и т. Д.). Он также предоставляет веб-интерфейс, так что вы можете просматривать журнал исключений.

Это первое, что я добавляю в любое создаваемое мною приложение asp.net.

Я все еще использую log4net, но я, как правило, использую его для регистрации отладки / информации и оставляю все исключения Elmah.

Более подробную информацию вы также можете найти в вопросе Как регистрировать ошибки (исключения) в ваших приложениях ASP.NET? .

Answer 2

Вы можете подключиться к событию OnError в Global.asax.

Примерно так:

/// <summary>
/// Handles the Error event of the Application control.
/// </summary>
/// <param name="sender">The source of the event.</param>
/// <param name="e">The <see cref="System.EventArgs"/> instance containing the event data.</param>
protected void Application_Error(object sender, EventArgs e)
{
    if (Server != null)
    {
        Exception ex = Server.GetLastError();

        if (Response.StatusCode != 404 )
        {
            Logging.Error("Caught in Global.asax", ex);
        }

    }


}

Answer 3

MVC3
Создайте атрибут, который наследуется от HandleErrorInfoAttribute и включает ваш выбор ведения журнала

public class ErrorLoggerAttribute : HandleErrorAttribute 
{
    public override void OnException(ExceptionContext filterContext)
    {
        LogError(filterContext);
        base.OnException(filterContext);
    }

    public void LogError(ExceptionContext filterContext)
    {
       // You could use any logging approach here

        StringBuilder builder = new StringBuilder();
        builder
            .AppendLine("----------")
            .AppendLine(DateTime.Now.ToString())
            .AppendFormat("Source:\t{0}", filterContext.Exception.Source)
            .AppendLine()
            .AppendFormat("Target:\t{0}", filterContext.Exception.TargetSite)
            .AppendLine()
            .AppendFormat("Type:\t{0}", filterContext.Exception.GetType().Name)
            .AppendLine()
            .AppendFormat("Message:\t{0}", filterContext.Exception.Message)
            .AppendLine()
            .AppendFormat("Stack:\t{0}", filterContext.Exception.StackTrace)
            .AppendLine();

        string filePath = filterContext.HttpContext.Server.MapPath("~/App_Data/Error.log");

        using(StreamWriter writer = File.AppendText(filePath))
        {
            writer.Write(builder.ToString());
            writer.Flush();
        }
    }

Поместить атрибут в Global.asax RegisterGlobalFilters

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

Answer 4

Представление Error.aspx определяется следующим образом:

namespace MvcApplication1.Views.Shared
{
    public partial class Error : ViewPage<HandleErrorInfo>
    {
    }
}

HandleErrorInfo имеет три свойства: строка ActionName строка ControllerName Исключение Исключение

Вы должны иметь доступ к HandleErrorInfo и, следовательно, к Исключению в представлении.

Answer 5

Задумывались ли вы о расширении атрибута HandleError? Кроме того, у Скотта есть хороший пост в блоге о фильтрах-перехватчиках для контроллеров / действий здесь .

Answer 6

Вы можете попробовать изучить HttpContext.Error, но я не уверен в этом.