Comment faire travailler ELMAH ASP.NET attribut MVC [HandleError]?

désolé, mais je pense que la réponse acceptée est exagérée. Tout ce que vous devez faire est ceci:

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);
    }
}

et ensuite l'enregistrer (l'ordre est important) dans Global.asax.cs:

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

il y a maintenant un ELMAH.Paquet MVC dans NuGet qui comprend une solution améliorée par Atif et aussi un contrôleur qui gère l'interface elmah dans le routage MVC (plus besoin d'utiliser cet axd)

Le problème avec cette solution (et avec toutes les autres ici) est que d'une manière ou d'une autre le gestionnaire d'erreurs elmah est en train de gérer l'erreur, ignorant ce que vous pourriez vouloir configurer comme une étiquette customError ou via ErrorHandler ou votre propre gestionnaire d'erreurs

La meilleure solution IMHO est de créer un filtre qui agira à la fin de tous les autres filtres et enregistrer les événements qui ont déjà été traités. Le module elmah doit prendre soin d'enregistrer les autres erreurs qui ne sont pas gênées par l'application. Cela vous permettra également d'utiliser le moniteur de la santé et tous les autres modules qui peuvent être ajoutés à asp.net à regarder les événements d'erreur

j'ai écrit ceci en regardant avec réflecteur L'ErrorHandler à l'intérieur d'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;
   }
}

maintenant, dans votre configuration de filtre vous voulez faire quelque chose comme ceci:

    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());
    }

notez que j'ai laissé un commentaire pour rappeler aux gens que s'ils veulent ajouter un filtre global qui gérera effectivement l'exception, il devrait passer avant ce dernier filtre, sinon vous courrez dans le cas où L'exception non gérée sera ignorée par le filtre ElmahMVCErrorFilter parce qu'elle n'a pas été traitée et qu'elle devrait être logée par le module Elmah mais le filtre suivant marque l'exception telle qu'elle est manipulée et le module l'ignore, ce qui fait que l'exception ne se transforme jamais en elmah.

maintenant, assurez-vous que les applications pour elmah dans votre webconfig ressemblent à quelque chose comme ceci:

<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 -->

L'important ici est "elmah.mvc.disableHandleErrorFilter", si ceci est faux, il utilisera le handler à l'intérieur d'elmah.mvc qui va réellement gérer l'exception en utilisant le HandleErrorHandler par défaut qui ignorera vos paramètres customError

cette configuration vous permet de définir vos propres balises ErrorHandler dans les classes et les vues, tout en enregistrant ces erreurs par le biais de L'ElmahMVCErrorFilter, en ajoutant une configuration customError à votre web.config via le module elmah, même en écrivant vos propres gestionnaires d'erreurs. La seule chose que vous devez faire est de ne pas ajouter de filtres qui seront réellement gérer l'erreur avant le filtre elmah que nous avons écrit. Et j'ai oublié de mentionner: aucun duplicata à elmah.

vous pouvez prendre le code ci-dessus et aller un peu plus loin en introduisant une usine de contrôleur personnalisé qui injecte l'attribut HandleErrorWithElmah dans chaque contrôleur.

pour plus d'informations, consultez ma série de blogs sur l'ouverture de session dans MVC. Le premier article traite de l'installation et de la mise en service d'Elmah pour MVC.

il y a un lien vers le code téléchargeable à la fin de l'article. Espérons que cela aide.

http://dotnetdarren.wordpress.com /

je suis nouveau dans ASP.NET MVC. J'ai connu le même problème, voici mon réalisable dans mon Erorr.vbhtml (ça marche si vous devez vous connecter à l'erreur à l'aide de Elmah journal)

@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> 

C'est tout simplement!

une solution complètement alternative est de ne pas utiliser le MVC HandleErrorAttribute , et à la place de compter sur ASP.Net la gestion des erreurs, avec laquelle Elmah est conçu pour travailler.

vous devez supprimer le global par défaut HandleErrorAttribute de App_Start\FilterConfig (ou Global.asax), et ensuite configurer une page d'erreur dans votre Web.config:

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

Note, Ceci peut être une URL MVC routée, donc ce qui précède redirigerait vers l'action ErrorController.Index quand une erreur se produit.

pour moi, il était très important de faire fonctionner la journalisation par e-mail. Après un certain temps, je découvre que cela n'a besoin que de 2 lignes de code de plus dans l'exemple 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);
        [...]
    }
    [...]
}

j'espère que cela va aider quelqu'un :)

c'est exactement ce dont j'avais besoin pour ma configuration de site MVC!

j'ai ajouté une petite modification à la méthode OnException pour gérer les instances multiples HandleErrorAttribute , comme suggéré par Atif Aziz:

gardez à l'esprit que vous pouvez prendre soin que si plusieurs HandleErrorAttribute instances sont en vigueur, alors la journalisation en double ne se produit pas.

je coche simplement context.ExceptionHandled avant invoquer la classe de base, juste pour savoir si quelqu'un d'autre a manipulé l'exception avant le gestionnaire actuel.

Cela fonctionne pour moi et je poste le code au cas où quelqu'un d'autre en a besoin et de demander si quelqu'un sait si j'ai oublié quelque chose.

j'Espère que c'est utile:

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);
}