Errori di registrazione in ASP.NET MVC

Question

Attualmente sto utilizzando log4net nella mia applicazione ASP.NET MVC per registrare le eccezioni. Il modo in cui sto facendo questo è avere tutti i miei controller ereditati da una classe BaseController. Nel caso in OnActionExecuting del BaseController, annoto tutte le eccezioni che possono essersi verificati:

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

Questa grande opera se un'eccezione non gestita durante un'azione di controllo.

Per quanto riguarda i 404 errori, ho un errore personalizzato istituito nel mio web.config in questo modo:

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

E nella azione di controllo che gestisce l'url "page-not-found", accedo l'URL originale richiesto:

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

    return View(); 
} 

E questo funziona anche.

Il problema che sto avendo è come registrare gli errori che si trovano nelle pagine .aspx. Diciamo che ho un errore di compilazione su una delle pagine o qualche codice inline che un'eccezione:

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

Sembra che l'attributo HandleError sia correttamente Ricalcolo Con questo alla mia pagina Error.aspx nella cartella condivisa, ma sicuramente non viene catturato dal metodo OnActionExecuted del mio BaseController. Stavo pensando che potrei inserire il codice di registrazione nella pagina Error.aspx, ma non sono sicuro di come recuperare le informazioni sull'errore a quel livello.

Answer 1

Vorrei prendere in considerazione la semplificazione della vostra applicazione web collegando Elmah.

Si aggiunge l'assembly Elmah al progetto e quindi si configura web.config. Registrerà quindi le eccezioni create a livello di controller o di pagina. Può essere configurato per accedere a diversi luoghi (come SQL Server, Email ecc.). Fornisce inoltre un front-end Web, in modo che sia possibile sfogliare il registro delle eccezioni.

È la prima cosa che aggiungo a qualsiasi app di asp.net mvc che creo.

Uso ancora log4net, ma tendo ad usarlo per registrare debug/informazioni e lasciare tutte le eccezioni a Elmah.

È inoltre possibile trovare ulteriori informazioni nella domanda How do you log errors (Exceptions) in your ASP.NET apps?.

Answer 2

Si può provare ad esaminare HttpContext.Error, ma non sono sicuro su questo.

Answer 3

Hai mai pensato di estendere l'attributo HandleError? Inoltre, Scott ha un buon post sui filtri intercettori su controller/azioni here.

Answer 4

La vista Error.aspx viene definito in questo modo:

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

Il HandleErrorInfo ha tre proprietà: stringa ActionName stringa NomeControllore Exception Eccezione

Si dovrebbe essere in grado di accedere a HandleErrorInfo e quindi la Eccezione nella vista.

Answer 5

È possibile connettersi all'evento OnError in Global.asax.

Qualcosa di simile a questo:

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

MVC3
Crea attributo che eredita da HandleErrorInfoAttribute e comprende la vostra scelta di registrazione

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

attributo posto in Global.asax RegisterGlobalFilters

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