Jak zmusić ELMAH do pracy z ASP.NET atrybut MVC [HandleError]?

Przepraszam, ale myślę, że przyjęta odpowiedź to przesada. Wszystko co musisz zrobić to:

public class ElmahHandledErrorLoggerFilter : IExceptionFilter
{
    public void OnException (ExceptionContext context)
    {
        // Log only handled exceptions, because all other will be caught by ELMAH anyway.
        if (context.ExceptionHandled)
            ErrorSignal.FromCurrentContext().Raise(context.Exception);
    }
}

A następnie zarejestrować go (kolejność jest ważna) w Global.asax.cs:

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

Jest teraz ELMAH.Pakiet MVC w NuGet, który zawiera ulepszone rozwiązanie Atif, a także kontroler obsługujący interfejs elmah w ramach routingu MVC (nie trzeba już używać axd)
Problem z tym rozwiązaniem (i ze wszystkimi tutaj) polega na tym, że w ten czy inny sposób obsługa błędu elmah faktycznie obsługuje błąd, ignorując to, co możesz chcieć skonfigurować jako tag customError lub poprzez ErrorHandler lub własny handler błędów
Najlepszym rozwiązaniem IMHO jest Utwórz filtr, który będzie działał na końcu wszystkich innych filtrów i rejestruje zdarzenia, które zostały już obsługiwane. Moduł elmah powinien zająć się logowaniem innych błędów, które nie są obsługiwane przez aplikację. Pozwoli to również na korzystanie z monitora zdrowia i wszystkich innych modułów, które można dodać do asp.net aby spojrzeć na zdarzenia błędów

Napisałem to patrząc z reflektorem na ErrorHandler wewnątrz elmah.mvc

public class ElmahMVCErrorFilter : IExceptionFilter
{
   private static ErrorFilterConfiguration _config;

   public void OnException(ExceptionContext context)
   {
       if (context.ExceptionHandled) //The unhandled ones will be picked by the elmah module
       {
           var e = context.Exception;
           var context2 = context.HttpContext.ApplicationInstance.Context;
           //TODO: Add additional variables to context.HttpContext.Request.ServerVariables for both handled and unhandled exceptions
           if ((context2 == null) || (!_RaiseErrorSignal(e, context2) && !_IsFiltered(e, context2)))
           {
            _LogException(e, context2);
           }
       }
   }

   private static bool _IsFiltered(System.Exception e, System.Web.HttpContext context)
   {
       if (_config == null)
       {
           _config = (context.GetSection("elmah/errorFilter") as ErrorFilterConfiguration) ?? new ErrorFilterConfiguration();
       }
       var context2 = new ErrorFilterModule.AssertionHelperContext((System.Exception)e, context);
       return _config.Assertion.Test(context2);
   }

   private static void _LogException(System.Exception e, System.Web.HttpContext context)
   {
       ErrorLog.GetDefault((System.Web.HttpContext)context).Log(new Elmah.Error((System.Exception)e, (System.Web.HttpContext)context));
   }


   private static bool _RaiseErrorSignal(System.Exception e, System.Web.HttpContext context)
   {
       var signal = ErrorSignal.FromContext((System.Web.HttpContext)context);
       if (signal == null)
       {
           return false;
       }
       signal.Raise((System.Exception)e, (System.Web.HttpContext)context);
       return true;
   }
}

Teraz, w konfiguracji filtra chcesz zrobić coś takiego:

    public static void RegisterGlobalFilters(GlobalFilterCollection filters)
    {
        //These filters should go at the end of the pipeline, add all error handlers before
        filters.Add(new ElmahMVCErrorFilter());
    }

Zauważ, że zostawiłem tam komentarz, aby przypomnieć ludziom, że jeśli chcą dodać globalny filtr, który faktycznie będzie obsługiwał wyjątek, powinien przejść przed tym ostatnim filtrem, w przeciwnym razie napotkasz przypadek, w którym nieobsługiwany wyjątek będzie ignorowany przez ElmahMVCErrorFilter, ponieważ nie został obsługiwany i powinien być zalogowany przez moduł Elmah, ale następnie następny filtr zaznacza wyjątek jako obsługiwany i moduł go ignoruje, co spowoduje, że wyjątek zostanie zignorowany przez elmahmvcerrorfilter. nigdy nie dostał się do elmah.

Teraz upewnij się, że appsettings dla elmah w webconfig wyglądają mniej więcej tak:

<add key="elmah.mvc.disableHandler" value="false" /> <!-- This handles elmah controller pages, if disabled elmah pages will not work -->
<add key="elmah.mvc.disableHandleErrorFilter" value="true" /> <!-- This uses the default filter for elmah, set to disabled to use our own -->
<add key="elmah.mvc.requiresAuthentication" value="false" /> <!-- Manages authentication for elmah pages -->
<add key="elmah.mvc.allowedRoles" value="*" /> <!-- Manages authentication for elmah pages -->
<add key="elmah.mvc.route" value="errortracking" /> <!-- Base route for elmah pages -->

Ta konfiguracja pozwala ustawić własne znaczniki ErrorHandler w klasach i widokach, podczas logowania te błędy przez ElmahMVCErrorFilter, dodając konfigurację customError do twojej sieci.config poprzez moduł elmah, nawet pisząc własne narzędzia do obsługi błędów. Jedyne co musisz zrobić, to pamiętać, aby nie dodawać żadnych filtrów, które faktycznie obsłuży błąd przed filtrem elmah, który napisaliśmy. I zapomniałem wspomnieć: żadnych duplikatów w elmah.

Możesz wziąć powyższy kod i pójść o krok dalej, wprowadzając niestandardową fabrykę kontrolerów, która wstrzykuje atrybut HandleErrorWithElmah do każdego kontrolera.

Aby uzyskać więcej informacji, zajrzyj na mój blog o logowaniu do MVC. Pierwszy artykuł dotyczy konfiguracji Elmah i uruchomienia MVC.

Na końcu artykułu znajduje się link do kodu do pobrania. Mam nadzieję, że to pomoże.

Http://dotnetdarren.wordpress.com/

Jestem nowy w ASP.NET MVC. Stanąłem w obliczu tego samego problemu, poniżej znajduje się mój wykonalny w moim eror.vbhtml (działa, jeśli musisz tylko zalogować błąd używając elmah log)

@ModelType System.Web.Mvc.HandleErrorInfo

    @Code
        ViewData("Title") = "Error"
        Dim item As HandleErrorInfo = CType(Model, HandleErrorInfo)
        //To log error with Elmah
        Elmah.ErrorLog.GetDefault(HttpContext.Current).Log(New Elmah.Error(Model.Exception, HttpContext.Current))
    End Code

<h2>
    Sorry, an error occurred while processing your request.<br />

    @item.ActionName<br />
    @item.ControllerName<br />
    @item.Exception.Message
</h2> 

Całkowicie alternatywnym rozwiązaniem jest nie używać MVC HandleErrorAttribute, a zamiast tego polegać na ASP.Net obsługa błędów, z którymi elmah jest przeznaczony do pracy.

Musisz usunąć domyślną globalną HandleErrorAttribute z App_Start\FilterConfig (lub Global.asax), a następnie skonfiguruj stronę błędu w swojej sieci.config:

<customErrors mode="RemoteOnly" defaultRedirect="~/error/" />

Uwaga, Może to być adres URL przekierowywany przez MVC, więc powyższy adres przekierowywałby do akcji ErrorController.Index, gdy wystąpi błąd.

Dla mnie bardzo ważne było, aby logowanie e-mail działało. Po jakimś czasie odkrywam, że to wymaga tylko 2 linijek kodu więcej w przykładzie Atif.

public class HandleErrorWithElmahAttribute : HandleErrorAttribute
{
    static ElmahMVCMailModule error_mail_log = new ElmahMVCMailModule();

    public override void OnException(ExceptionContext context)
    {
        error_mail_log.Init(HttpContext.Current.ApplicationInstance);
        [...]
    }
    [...]
}

Mam nadzieję, że to komuś pomoże:)

To jest dokładnie to, czego potrzebowałem do konfiguracji mojej strony MVC!

Dodałem małą modyfikację metody OnException do obsługi wielu instancji HandleErrorAttribute, zgodnie z sugestią Atif Aziz:

Pamiętaj, że być może będziesz musiał zadbać o to, aby w przypadku wystąpienia wielu instancji HandleErrorAttribute nie wystąpi duplikat logowania.

Po prostu sprawdzam context.ExceptionHandled przed wywołaniem klasy bazowej, aby wiedzieć, czy ktoś inny obsługiwał wyjątek przed bieżącym handlerem.
Informatyka działa dla mnie i zamieszczam kod na wypadek, gdyby ktoś inny go potrzebował i pytam, czy ktoś wie, czy coś przeoczyłem.

Mam nadzieję, że się przyda:

public override void OnException(ExceptionContext context)
{
    bool exceptionHandledByPreviousHandler = context.ExceptionHandled;

    base.OnException(context);

    Exception e = context.Exception;
    if (exceptionHandledByPreviousHandler
        || !context.ExceptionHandled  // if unhandled, will be logged anyhow
        || RaiseErrorSignal(e)        // prefer signaling, if possible
        || IsFiltered(context))       // filtered?
        return;

    LogException(e);
}