255

When I use Response.Redirect(...) to redirect my form to a new page I get the error:

A first chance exception of type 'System.Threading.ThreadAbortException' occurred in mscorlib.dll
An exception of type 'System.Threading.ThreadAbortException' occurred in mscorlib.dll but was not handled in user code

My understanding of this is that the error is being caused by the webserver aborting the remainder of the page the response.redirect was called on.

I know I can add a second parameter to Response.Redirect that is called endResponse. If I set endResponse to True I still get the error but if I set it to False then I do not. I am pretty sure though that that means the webserver is running the rest of the page I redirected away from. Which would seem to be inefficient to say the least. Is there a better way to do this? Something other than Response.Redirect or is there a way to force the old page to stop loading where I will not get a ThreadAbortException?

10 Answers 10

366

The correct pattern is to call the Redirect overload with endResponse=false and make a call to tell the IIS pipeline that it should advance directly to the EndRequest stage once you return control:

Response.Redirect(url, false);
Context.ApplicationInstance.CompleteRequest();

This blog post from Thomas Marquardt provides additional details, including how to handle the special case of redirecting inside an Application_Error handler.

7
  • 6
    It executes code after Context.ApplicationInstance.CompleteRequest();. Why? Will I have to return from the event handler conditionally?
    – IsmailS
    Nov 26, 2010 at 7:46
  • 4
    @Ismail: The old version of Redirect throws a ThreadAbortException to prevent execution of any subsequent code. The newer, preferred version doesn't throw, but you are responsible for returning control early if you have additional code in the handler. Nov 29, 2010 at 18:56
  • 14
    I think it's more accurate to say "the second overload" rather than The old version of Redirect phrase you use in your comment, it's not like MS changed the implementation, it's just another overload.
    – BornToCode
    Jul 14, 2015 at 1:39
  • 2
    I do not think this is an ideal pattern. You are asking the page not to end the response and continue the execution and then completing request programmatically. But what about the rendering of aspx page and event handlers? not ending the response means, it will finish rendering the aspx page before hitting "completeRequest()". Now if I am using a server side property in my page say a session variable to determine valid login, which if expire will throw a null exception before even redirecting. And the only way to fix that is to make the endResponse back to true.
    – Abs
    Jan 5, 2017 at 1:13
  • 1
    Was going to up vote this answer, but that page code continued to execute. This is not ideal in my case. Much cleaner to handle or ignore the "ThreadAbortException"
    – DaniDev
    Jan 31, 2018 at 19:23
171

There is no simple and elegant solution to the Redirect problem in ASP.Net WebForms. You can choose between the Dirty solution and the Tedious solution

Dirty: Response.Redirect(url) sends a redirect to the browser, and then throws a ThreadAbortedException to terminate the current thread. So no code is executed past the Redirect()-call. Downsides: It is bad practice and have performance implications to kill threads like this. Also, ThreadAbortedExceptions will show up in exception logging.

Tedious: The recommended way is to call Response.Redirect(url, false) and then Context.ApplicationInstance.CompleteRequest() However, code execution will continue and the rest of the event handlers in the page lifecycle will still be executed. (E.g. if you perform the redirect in Page_Load, not only will the rest of the handler be executed, Page_PreRender and so on will also still be called - the rendered page will just not be sent to the browser. You can avoid the extra processing by e.g. setting a flag on the page, and then let subsequent event handlers check this flag before before doing any processing.

(The documentation to CompleteRequest states that it "Causes ASP.NET to bypass all events and filtering in the HTTP pipeline chain of execution". This can easily be misunderstood. It does bypass further HTTP filters and modules, but it doesn't bypass further events in the current page lifecycle.)

The deeper problem is that WebForms lacks a level of abstraction. When you are in a event handler, you are already in the process of building a page to output. Redirecting in an event handler is ugly because you are terminating a partially generated page in order to generate a different page. MVC does not have this problem since the control flow is separate from rendering views, so you can do a clean redirect by simply returning a RedirectAction in the controller, without generating a view.

5
  • 8
    I believe the best description of webforms I ever heard was "lie sauce."
    – mcfea
    Jan 2, 2015 at 22:58
  • If you use the dirty option, You can turn off break on ThreadAbortException in Visual Studio. DEBUG > Exceptions.... Expand CLR > System.Threading > Uncheck System.Threading.ThreadAbortException.
    – Jess
    Sep 30, 2015 at 14:48
  • 1
    In my case this exception is not coming for each and every time, only for few times in between it is occuring. Means If and clicking on same button of Live application it is working but when same link and same button is clicked from other machine it is giving System.Threading.ThreadAbortException. Any idea why it not happening everytime?? Jan 5, 2017 at 13:07
  • 1
    If you pass false, and don't bother checking the flag, then the browser will receive the entire page with a 302 status code in the header - found. The malicious user can simply change the status to 200, and gain access to the entire page, which is very insecure.
    – jo0ls
    Dec 16, 2019 at 12:58
  • If Context doesn't resolve for you, try HttpContext. Aug 18, 2020 at 19:14
42

I know I'm late, but I've only ever had this error if my Response.Redirect is in a Try...Catch block.

Never put a Response.Redirect into a Try...Catch block. It's bad practice

As an alternative to putting the Response.Redirect into the Try...Catch block, I'd break up the method/function into two steps.

  1. inside the Try...Catch block performs the requested actions and sets a "result" value to indicate success or failure of the actions.

  2. outside of the Try...Catch block does the redirect (or doesn't) depending on what the "result" value is.

This code is far from perfect and probably should not be copied since I haven't tested it.

public void btnLogin_Click(UserLoginViewModel model)
{
    bool ValidLogin = false; // this is our "result value"
    try
    {
        using (Context Db = new Context)
        {
            User User = new User();

            if (String.IsNullOrEmpty(model.EmailAddress))
                ValidLogin = false; // no email address was entered
            else
                User = Db.FirstOrDefault(x => x.EmailAddress == model.EmailAddress);

            if (User != null && User.PasswordHash == Hashing.CreateHash(model.Password))
                ValidLogin = true; // login succeeded
        }
    }
    catch (Exception ex)
    {
        throw ex; // something went wrong so throw an error
    }

    if (ValidLogin)
    {
        GenerateCookie(User);
        Response.Redirect("~/Members/Default.aspx");
    }
    else
    {
        // do something to indicate that the login failed.
    }
}
2
  • 1
    Did not have the issue until I wrapped my code in a try, catch... I wonder what other code calls cause this behavior in .NET Dec 29, 2016 at 22:07
  • This also will happen inside an if-else statement, not just a try-catch. I believe it's any conditional statement, perhaps even in any event handler. At any rate, performing the change you suggest doesn't do anything to resolve the issue since it's still inside a conditional and an event handler.
    – TylerH
    Sep 4, 2020 at 19:41
9

Response.Redirect() throws an exception to abort the current request.

This KB article describes this behavior (also for the Request.End() and Server.Transfer() methods).

For Response.Redirect() there exists an overload:

Response.Redirect(String url, bool endResponse)

If you pass endResponse=false, then the exception is not thrown (but the runtime will continue processing the current request).

If endResponse=true (or if the other overload is used), the exception is thrown and the current request will immediately be terminated.

8

Here's the official line on the problem (I couldn't find the latest, but I don't think the situation has changed for later versions of .net)

1
  • 8
    @svick Regardless of link rot, link only answers aren't really great answers. meta.stackexchange.com/q/8231 I think that links are fantastic, but they should never be the only piece of information in your answer.
    – Ryan Gates
    Feb 4, 2015 at 14:59
8

This is just how Response.Redirect(url, true) works. It throws the ThreadAbortException to abort the thread. Just ignore that exception. (I presume it is some global error handler/logger where you see it?)

An interesting related discussion Is Response.End() Considered Harmful?.

1
  • 4
    Aborting a thread seems like a really heavy handed way to deal with premature end of response. I find it strange that the framework wouldn't prefer to re-use the thread instead of spinning up a new one to take its place.
    – spender
    May 5, 2010 at 22:13
3

Also I tried other solution, but some of the code executed after redirect.

public static void ResponseRedirect(HttpResponse iResponse, string iUrl)
    {
        ResponseRedirect(iResponse, iUrl, HttpContext.Current);
    }

    public static void ResponseRedirect(HttpResponse iResponse, string iUrl, HttpContext iContext)
    {
        iResponse.Redirect(iUrl, false);

        iContext.ApplicationInstance.CompleteRequest();

        iResponse.BufferOutput = true;
        iResponse.Flush();
        iResponse.Close();
    }

So if need to prevent code execution after redirect

try
{
   //other code
   Response.Redirect("")
  // code not to be executed
}
catch(ThreadAbortException){}//do there id nothing here
catch(Exception ex)
{
  //Logging
}
4
  • 2
    just follow up by the Jorge answer. This will ectually remove the logging of the Thread abort exception. Jan 17, 2014 at 16:20
  • When someone ask why he gets an Exception, telling him to just play with try..catch is not an answer. See the accepted answer. I commented on your answer while reviewing "late answer"
    – manuell
    Jan 17, 2014 at 16:24
  • 1
    That has the same effect as putting false for the 2nd argument of Response.Redirect, but the "false" is a nicer solution than capturing the ThreadAbortException. I don't see that there is ever a good reason to do it this way.
    – NickG
    Jun 13, 2016 at 13:08
  • This fixed my issue where I had an async method that although was catching ThreadAbortException still remained uncaught. I ended up adding an extension method to HttpResponse.
    – TechyGypo
    Jan 4 at 11:58
3

i even tryed to avoid this, just in case doing the Abort on the thread manually, but i rather leave it with the "CompleteRequest" and move on - my code has return commands after redirects anyway. So this can be done

public static void Redirect(string VPathRedirect, global::System.Web.UI.Page Sender)
{
    Sender.Response.Redirect(VPathRedirect, false);
    global::System.Web.UI.HttpContext.Current.ApplicationInstance.CompleteRequest();
}
2
  • Maybe protected method in public class MyBasePage : Page
    – Kiquenet
    Nov 5, 2020 at 12:31
  • Not System.Web.UI.HttpContext. Really System.Web.HttpContext
    – Kiquenet
    Nov 5, 2020 at 12:37
1

What I do is catch this exception, together with another possible exceptions. Hope this help someone.

 catch (ThreadAbortException ex1)
 {
    writeToLog(ex1.Message);
 }
 catch(Exception ex)
 {
     writeToLog(ex.Message);
 }
1
  • 2
    Better avoid ThreadAbortException exception than catch and do nothing ?
    – Kiquenet
    Oct 5, 2015 at 9:05
-1

I had that problem too.

Try using Server.Transfer instead of Response.Redirect

Worked for me.

3
  • 2
    Server.Transfer should still be throwing a ThreadAbortException: support.microsoft.com/kb/312629, so it is not a recommended solution. Oct 11, 2011 at 15:19
  • 9
    Server.Transfer will not send a redirect to the user. It has a different purpose altogether!
    – Marcel
    Sep 18, 2013 at 13:05
  • 1
    Server.transfer and responce.redirect are different
    – mzonerz
    Aug 17, 2016 at 14:10

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.