Asp.net MVC переопределить OnException в базовом контроллере продолжает распространяться на Application_Error - PullRequest
50 голосов
/ 13 июня 2011

Я пытаюсь вернуть представление, а не выдавать пользователю перенаправление на основании определенных ошибок, которые могут возникнуть в моем приложении, я хочу обработать ошибки + записать их в свой базовый контроллер, я не хочу, чтобы ошибка распространяласьвплоть до моего метода Global.asax - Application_Error (), так как я хочу, чтобы этот метод обрабатывал любые другие ошибки в моем приложении, например, пользователь вводит фиктивный URL, кто-нибудь нашел способ обойти это?

ПРИМЕЧАНИЕ. Я оставилмой закомментированный код, поскольку у меня был обходной путь для некоторых проблем, это также показывает, что у меня есть несколько исключений для возможной обработки ...

РЕДАКТИРОВАТЬ: Если я запускаю RedirectToAction в этом переопределении OnException, все работает, как ожидалось, нохочу вернуть представление без перенаправления ...

Мой метод базового контроллера:

    protected override void OnException(ExceptionContext filterContext)
    {
        //dont interfere if the exception is already handled
        if (filterContext.ExceptionHandled)
            return;

        //let the next request know what went wrong
        filterContext.Controller.TempData["exception"] = filterContext.Exception;

        //log exception
        _logging.Error(User.Identity.Name, ExceptionHelper.BuildWebExceptionMessage(filterContext.Exception));


        //set up redirect to my global error handler
        //if (filterContext.Exception.GetType() == typeof(NoAccessException))
        //    filterContext.Result = View(new RouteValueDictionary
        //    (new { area = "", controller = "Error", action = "PublicError" }));

        //else {
        //Only return view, no need for redirection
        filterContext.Result = View(new RouteValueDictionary
        (new { area = "", controller = "Error", action = "NoAccess" }));
        //}
        //advise subsequent exception filters not to interfere and stop
        // asp.net from showing yellow screen of death
        filterContext.ExceptionHandled = true;

        //erase any output already generated
        filterContext.HttpContext.Response.Clear();

        //base.OnException(filterContext);
    }

Этот метод должен обрабатывать любые другие ошибки, которые могут появиться в моем приложении, я не хочуошибки выше обрабатываются внутри моего Application_Error ()

protected void Application_Error()
        {

            Exception exception = Server.GetLastError();
            // Log the exception.

            var logger = Container.Get<ILoggingService>();
            logger.Error(User.Identity.Name, ExceptionHelper.BuildWebExceptionMessage(exception));

            Response.Clear();

            HttpException httpException = exception as HttpException;

            RouteData routeData = new RouteData();
            routeData.Values.Add("controller", "Error");

            //if (httpException == null)
            //{
            routeData.Values.Add("action", "PublicError");
            //}
            //else //It's an Http Exception, Let's handle it.
            //{
            //    switch (httpException.GetHttpCode())
            //    {
            //        case 404:
            //            // Page not found.
            //            routeData.Values.Add("action", "HttpError404");
            //            break;
            //        case 500:
            //            // Server error.
            //            routeData.Values.Add("action", "HttpError500");
            //            break;

            //        // Here you can handle Views to other error codes.
            //        // I choose a General error template  
            //        default:
            //            routeData.Values.Add("action", "General");
            //            break;
            //    }
            //}

            // Pass exception details to the target error View.
            routeData.Values.Add("error", exception);

            // Clear the error on server.
            Server.ClearError();

            // Avoid IIS7 getting in the middle
            Response.TrySkipIisCustomErrors = true;

            // Call target Controller and pass the routeData.
            IController errorController = new ErrorController();
            errorController.Execute(new RequestContext(
                 new HttpContextWrapper(Context), routeData));
        }

1 Ответ

71 голосов
/ 14 июня 2011

Должно работать следующее:

protected override void OnException(ExceptionContext filterContext)
{
    if (filterContext.ExceptionHandled)
    {
        return;
    }
    filterContext.Result = new ViewResult
    {
        ViewName = "~/Views/Shared/Error.aspx"
    };
    filterContext.ExceptionHandled = true;
}

Также убедитесь, что в этом методе не создается исключение, иначе оно будет распространяться на Application_Error.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...