Change ExceptionHandled after reviewing MVC5

aspnet · Jun 8, 2016 · 90fa7c6 · 90fa7c6
1 parent 13667d7
commit 90fa7c6
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Internal/ControllerActionInvoker.cs b/src/Microsoft.AspNetCore.Mvc.Core/Internal/ControllerActionInvoker.cs
@@ -345,7 +345,6 @@ private async Task InvokeResourceFilterAsync()
                     Debug.Assert(_exceptionContext != null);
                     if (_exceptionContext.Result != null)
                     {
-                        _exceptionContext.ExceptionHandled = true;
                         // This means that exception filters returned a result to 'handle' an error.
                         // We're not interested in seeing the exception details since it was handled.
                         await InvokeResultAsync(_exceptionContext.Result);
@@ -355,7 +354,7 @@ private async Task InvokeResourceFilterAsync()
                             Result = _exceptionContext.Result,
                         };
                     }
-                    else if (_exceptionContext.Exception != null)
+                    else if (_exceptionContext.Exception != null && !_exceptionContext.ExceptionHandled)
                     {
                         // If we get here, this means that we have an unhandled exception.
                         // Exception filted didn't handle this, so send it on to resource filters.