Question

I’m redirecting to an Error page with a prettified error message in my Application_Error, in Global.asax.

At the moment it says:

Response.Redirect("Error.aspx", true);

Should that be:

Response.Redirect("Error.aspx", false); 

I’m not sure under which circumstances I should use true and which I should use false? The MSDN page says to prefer using false to avoid ThreadAbortExceptions, so when should I use true?

Answer 1

You use false when you don’t want to abort the thread. What that means is that false will cause the code to continue to execute. So lines of code which appear after the Response.Redirect will be executed. A true will just kill the thread so nothing further will execute, which in turn throws a ThreadAbortException.

So it’s really a judgment call based on how the rest of the code in that situation looks. Generally you want to put calls to Response.Redirect at the end of an execution path so that nothing further needs to be executed. But many times that’s not the case. It’s just a matter of how you control the logic flow in the code.

For example, if the next line after Response.Redirect is a return and the execution path simply ends, then you’re probably fine. But if there’s all kinds of logic and executing it in this case would leave the system in an unknown state, then you may want to abort the thread.

Personally I consider aborting the thread to be indicative of poor logic control. It’s similar to a well known code smell where exceptions are used to control logic flow, which is universally frowned upon. If you can control the logic flow without the need for aborting a thread and throwing an exception, that would probably be preferred.