Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Unhandled Exception Global Handler for OWIN / Katana?

What is the proper way to implement a global Exception catcher-handler in a Katana (OWIN) implementation?

In a self-hosted OWIN/Katana implementation running as an Azure Cloud Service (worker role), I placed this code in a Middleware:

throw new Exception("pooo"); 

Then I placed this code in the Startup class Configuration method, setting a breakpoint in the event handler:

 AppDomain.CurrentDomain.UnhandledException +=      CurrentDomain_UnhandledExceptionEventHandler; 

and the event handler in the same class (with a breakpoint set on the first line):

private static void CurrentDomain_UnhandledExceptionEventHandler(object sender, UnhandledExceptionEventArgs e) {     var exception = (Exception)e.ExceptionObject;     Trace.WriteLine(exception.Message);     Trace.WriteLine(exception.StackTrace);     Trace.WriteLine(exception.InnerException.Message); } 

When the code runs the breakpoint is not hit. The Visual Studio Output window does include this however:

A first chance exception of type 'System.Exception' occurred in redacted.dll A first chance exception of type 'System.Exception' occurred in mscorlib.dll 

I also tried moving the wireup and handler to the Worker Role OnStart method but still the breakpoint is not hit.

I am not using WebAPI at all, but did look at posts on what is done there, but I found nothing clear, so here I am.

Running on .NET Framework 4.5.2, VS 2013.

All ideas appreciated. Thanks.

like image 654
Snowy Avatar asked Jun 18 '15 14:06

Snowy


2 Answers

Try writing a custom middleware and placing it as the first middleware:

public class GlobalExceptionMiddleware : OwinMiddleware {    public GlobalExceptionMiddleware(OwinMiddleware next) : base(next)    {}     public override async Task Invoke(IOwinContext context)    {       try       {           await Next.Invoke(context);       }       catch(Exception ex)       {           // your handling logic       }    }  } 

Place it as the first middleware:

public class Startup {     public void Configuration(IAppBuilder builder)     {         var config = new HttpConfiguration();          builder.Use<GlobalExceptionMiddleware>();         //register other middlewares     } } 

When we register this middleware as the first middle, any exceptions happening in other middlewares (down the stacktrace) will propagate up and be caught by the try/catch block of this middleware.

It's not mandatory to always register it as the first middleware, in case you don't need global exception handling for some middlewares, just register these middlewares before this one.

public class Startup {     public void Configuration(IAppBuilder builder)     {         var config = new HttpConfiguration();          //register middlewares that don't need global exception handling.          builder.Use<GlobalExceptionMiddleware>();         //register other middlewares     } } 
like image 97
Khanh TO Avatar answered Sep 24 '22 01:09

Khanh TO


Try this:

public class CustomExceptionHandler : IExceptionHandler {         public Task HandleAsync(ExceptionHandlerContext context, CancellationToken cancellationToken)     {       // Perform some form of logging        context.Result = new ResponseMessageResult(new HttpResponseMessage       {         Content = new StringContent("An unexpected error occurred"),         StatusCode = HttpStatusCode.InternalServerError       });        return Task.FromResult(0);     } } 

And at startup:

public void Configuration(IAppBuilder app) {   var config = new HttpConfiguration();    config.Services.Replace(typeof(IExceptionHandler), new CustomExceptionHandler()); } 
like image 26
Greg R Taylor Avatar answered Sep 22 '22 01:09

Greg R Taylor