views:

462

answers:

1

I have a web page that sends email to multiple users (online distribution list). After the submit button is clicked and the email is sent, a status page is shown listing how many emails were sent, errors, and other information. If the user clicks the back button, the email is resent. How can I prevent this?

NOTE: The browser DOES prompt the user to "resubmit" or "resend" data to the page before actually sending the email, but that does not stop my users from clicking it and then wondering why two copies of the email were sent out.

Environment:

  • Server: C#, ASP.NET 2.0, IIS6
  • Client: any browser (I don't want an IE-specific solution like SmartNavigation)
+9  A: 

In your code, right after the point where e-mails are sent, do a 302 redirect to a confirmation page:

protected void btnSend_Click(object sender, EventArgs e)
{
    SendManyEmails();
    Response.Redirect("confirmation.aspx");
}

With this kind of code, the POST to the original page will not end up in the browser history.

This common pattern is known as the Post/Redirect/Get pattern.

Bonus info about keeping state when doing Post/Redirect/Get

The main drawback of this pattern is that all state from the handling of the POST request is lost when redirecting the user - thus, commencing a new request context. In ASP.NET this includes members within the Page and all Control objects, as well as everything stored in the ViewState.

If you generate some kind of "status object" - maybe a log of sent mail messages - while handling the POST request, you will need some way to save this object for the following GET request. Some web frameworks has functionality specifically for this: RoR has flash, ASP.NET MVC has TempData. ASP.NET forms has no such concept built in, so you will have to figure something out yourself.

Saving the object to the Session on the POST, reading and deleting it on the following GET would be one way to solve this. You can build an abstraction around this if you use it in several places, or you could search the web for existing implementations of flash/TempData for ASP.NET forms.

Jørn Schou-Rode
+1 for good answer that taught me something new. :) I really ought to get more into web dev't.
Greg D
+1 Nice one, I never thought about the back button repost problem. Very Nice!
Chuck Conway
It may come to something like this, but doing a redirect, I lose all my status info that was generated in the send() method. That could be a lot of overhead. +1
Rick
Nice one. Every day is a school day.
Sonny Boy
@Rick: I have just added a bonus section to my answer, regarding the situation you describe in your last comment.
Jørn Schou-Rode