views:

1909

answers:

6

I am having problems using the [HandleError] attribute on my Controller Actions - it doesn't seem to work at all (i.e. it doesn't matter if the filter is there or not - I get the same results...). When an Exception is thrown, I get the standard red-toned Server Error in '/' Application error page instead of my custom view.

I have found a couple of other threads on the subject here on SO, and in most cases it seems that setting the customErrors option to On in web.config solved the problem. It has not for me, so I need to find a different solution.

My controller action:

[HandleError]
public ActionResult Index()
{
    throw new Exception("oops...");
    return View();
}

In my web.config file

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

I have made sure that the Error.aspx file is in the Shared directory, too. What am I missing?

I am running ASP.NET MVC RC Refresh.

+3  A: 

You need to specify what page to redirect to as well.

<customErrors mode="On" defaultRedirect="Error.aspx" />

EDIT: Sorry the /Shared/ part should not bet there but you need to tell MVC which page to send the user to with Error.aspx. Then the default route looks for something called Error.aspx in shared.

It was very late! :) I guess that's why someone gave me a minus for the answer! :) At least it works here mate!

mhenrixon
I'm sorry, but it doesn't work with Error.aspx either...
Tomas Lycken
Also, it shouldn't be necessary according to this article http://weblogs.asp.net/scottgu/archive/2008/07/14/asp-net-mvc-preview-4-release-part-1.aspx (which is the most recent i have been able to find, although it's only on PV4...)
Tomas Lycken
I read that article but for me the error isnt handled unless I say where to redirect to.
mhenrixon
In my web.config i write <customErrors mode="On" defaultRedirect="Error.aspx"></customErrors>, and when an exception is thrown I get a 404 on this url: http://localhost:52812/Error.aspx?aspxerrorpath=/Program/Edit
Tomas Lycken
+2  A: 

Two useful things to know:

By default, HandleError does nothing when running under the development server. The intention is to show developers more useful information:

    public virtual void OnException(ExceptionContext filterContext) {
        if (filterContext == null) {
            throw new ArgumentNullException("filterContext");
        }

        // If custom errors are disabled, we need to let the normal ASP.NET exception handler
        // execute so that the user can see useful debugging information.
        if (filterContext.ExceptionHandled || !filterContext.HttpContext.IsCustomErrorEnabled) {
            return;
        }

Note that this case is precisely what customError is supposed to control. If setting customError="On" does not change this behavior:

  1. Check your syntax.
  2. Make sure you're editing the Web.config in the project root, not the one in Views.
  3. Make sure no code sets HttpContext.IsCustomErrorEnabled.
  4. If all else fails, try turning debug off in Web.config

Second, there certain types of errors which HandleError will never handle, notably ASP.NET compilation errors. You don't say which error you're encountering.

Craig Stuntz
thanks for your reply! your first remark does indeed explain why i see no effect... is there any way i can test the error handling locally before uploading to the server? how? my project compiles, but i am deliberately throwing runtime exceptions to test error handling.
Tomas Lycken
See updated reply.
Craig Stuntz
Hi again. I have now tried customErrors="On" with and without a default redirect, customErrors="Off", removing the node completely, and setting debug="false" - and the only difference is what info is shown on the red-toned error page... Is it debugging or using Cassini that causes the problem?
Tomas Lycken
When you say Cassini do you mean the ASP.NET development server? They're not the same, and I don't know what Cassini does.
Craig Stuntz
Yes, that's what I mean. The thing that runs by default when I click "play" in VS =) I have learned that they in fact are the same (in WROX's "ASP.NET 2.0 for Beginners", if i'm not mistaken...). Will debugging in IIS solve the problem?
Tomas Lycken
I have now configured the site to run in IIS instead of on the ASP.NET Development Server, but with no success. Any ideas here?
Tomas Lycken
Given what you have said, it's difficult to imagine that Web.config is being configured correctly. It is really the only explanation for the behavior you are seeing. My suggestion would be to find an example of a site that works, and compare it with what you are doing.
Craig Stuntz
A: 

Is there any resolution to this problem? Seems like none of the configuration settings really solve the problem. I have the same problem as the last comment in the last answer. 404 response on Error.aspx?aspxerrorpath=/path/to/page/causing/error

Any help?

Cole B
Okay, I believe I solved the problem. There were errors in my Error.aspx view. So, once I resolved those errors, the Error.aspx view was rendered as expected.
Cole B
Hi Cole! No, there never was a resolution to this problem - I started working on other parts of the project, and I haven't gotten back to this yet. I will need to dive into it again though - thanks for posting back when you found something! =)
Tomas Lycken
+2  A: 

I got the same issue and it took me two full days to figure it out finally. It turned out to be that I got an error in Site.Master page, and the Error.aspx used this same master page as all other pages. Obviously the Error.aspx couldn't deal with such situation.

My solution is to create a specific Error.master page that is lightweight and does not include any model data. Additionaly I created a static Error.htm in case an error occurs from Error.aspx. The Web.config setting is as follows:

<customErrors mode="On">
    <error statusCode="500" redirect="Error.htm" />
</customErrors>

Hope it helps.

Justin Chen
My Error.aspx page was also throwing its own exception. That's a pain to debug!
Zack Peterson
The Error.htm seems to return an HTTP response status code of 200 rather than 500 as it should!
Zack Peterson
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
A: 

To get around the 404 problem when the Error.aspx was supposed to be shown, I had to exclude the Error.aspx from the httpHandler section which prevented any views from being accessed directly (around the mvc 2 framework). I did this by putting Error.aspx in an 'Error' subfolder and putting a web.config in this subfolder with a <remove path="*" verb="*" /> in the httpHandlers section. My version of this problem (and its solution) may be specific to MVC 2.

Remember to update the defaultRedirect reference, when you move Error.aspx :)

LOAS