views:

1203

answers:

4

I believe I have set up our MVC app to use [HandleError] properly. This is my controller:

[HandleError]
public class SupportController : BaseController {

    public ActionResult Toss() {
        throw new Exception("uh oh");
    }
    // snip
}

And I have set my customErrors tag in web.config to "On":

<customErrors mode="On"></customErrors>

However, I am still getting the Yellow Screen of Death on Exceptions. Setting a breakpoint in my Toss() action shows that HttpContext.IsCustomErrorEnabled is set to true.

We are not doing anything with the View Engine, and the BaseController doesn't mess with anything either (and other Controllers that do not extend it have the same issue).

I am developing on Windows XP, and have the same issue when the app is deployed to a server 2003 box (IIS 6).

I do not think there is an Exception on the error.aspx page:

<%@ Page Language="C#" MasterPageFile="~/Views/Shared/Site.Master" Inherits="System.Web.Mvc.ViewPage<System.Web.Mvc.HandleErrorInfo>" %>

<asp:Content ID="Content1" ContentPlaceHolderID="head" runat="server">
</asp:Content>

<asp:Content ID="errorContent" ContentPlaceHolderID="MainContent" runat="server">
    <h2>
        Sorry, an error occurred while processing your request.
    </h2>
</asp:Content>

One thing that may be different is that this app was created back when MVC Beta was the latest version, and was upgraded to the RC and then RTM as they were released. Might there have been some kooky setting that is left over from there?

I can get this working on other apps, so I'm a little flummoxed.

+3  A: 

This tends to happen if there is a problem processing the error page.. if you debug the app, right after the initial exception you'll problem hit another (exception from the error page).. I had this happening and the reason for me was because I had a strongly typed master page, the error page was using this master page, and because the masterpage shares the same model as the actual page, the master page was getting a HandlerErrorInfo model, instead of the typed model I expected..

Personally I think this is a poor design in the asp.net mvc (along with the rest of it), but you can get around this easy enough by not using the same master page (you could even do masterpage inheritance where you have an inherited strongly typed master page that purely inherits the layer from the untyped one..

Otherwise this is some sort of exception happening in the error view.. (most likely).

meandmycode
You weren't 100% right, but you put me on the right track. The Master Page does not have a model, but it does do some stuff that was presumably throwing a null reference exception. Using a non master paged view works. Thanks!
swilliams
Happy to help, plenty of headache points in asp.net mvc, I seem to keep finding them and figured I should save somebody some 'head to keyboard impact' actions. :)
meandmycode
+2  A: 

Another reason for this problem may be ,

In Template MVC Application (generated by VS2008 / VS2008 Express) , Error.aspx (generated by VS) uses Master Page.

If Master Page access any ViewData it will throw null reference Exception , then the error.aspx won't be shown.

Use this Simple code as your Error.aspx , it will solve the problem, (along with CustomErrors=On )

<%@ Page Language="C#"  Inherits="System.Web.Mvc.ViewPage<System.Web.Mvc.HandleErrorInfo>" %>
<%= Model.Exception.Message %>
Palani
Yep, that was the problem for me, thanks.
Rob West
A: 

After much hair-pulling, I found that my problem falls into the general category of "throwing exception from code only to throw another error when rendering the error view." I had written a strongly-typed error page earlier expecting a System.Exception, then when I switched to using the [HandleError] attribute, the filter was passing the error page a System.HandleErrorInfo. The second exception happened at this line (in Default.aspx.cs):

httpHandler.ProcessRequest(HttpContext.Current);

Confusing.

Daniel
A: 

I had a same problem and after investigating a lot, I found that I was updating the web.Debug.config file under web.config. Then I created the the CustomErrors tag in the web.config file and it was working for me. :)