How to prevent multiple inserts when submitting a form in PHP?
Asked Answered
W

14

31

Sometimes the user may press Enter twice, and the post is inserted twice.

Is there a solution to prevent this other than check if there is already a post with the same title and content?

Waly answered 25/1, 2010 at 16:56 Comment(2)
Can you disable the button once it is pressed?Demimondaine
I would recomend using Form Keys. Have a look at that here: net.tutsplus.com/tutorials/php/secure-your-forms-with-form-keysTrackandfield
C
55

There are several solutions to this problem:

  1. Use Javascript to disable the form's submit button when it is posted. Downside to this is that this is ABSOLUTELY not a foolproof way. It's very easy to submit forms without actually clicking the button, and this would also not work for users with JavaScript disabled. I would definitely not recommend this method.

    Example:

    <script language="javascript">
    <!--
        function disableSubmitButton() {
            // you may fill in the blanks :)
        }
    -->
    </script>
    <form action="foo.php" method="post">
        <input type="text" name="bar" />
        <input type="submit" value="Save" onclick="disableSubmitButton();">
    </form>
    
  2. Use PHP sessions to set a session variable (for example $_SESSION['posttimer']) to the current timestamp on post. Before actually processing the form in PHP, check if the $_SESSION['posttimer'] variable exists and check for a certain timestamp difference (IE: 2 seconds). This way, you can easily filter out double submits.

    Example:

    // form.html
    <form action="foo.php" method="post">
        <input type="text" name="bar" />
        <input type="submit" value="Save">
    </form>
    
    // foo.php
    if (isset($_POST) && !empty($_POST)) 
    {
        if (isset($_SESSION['posttimer']))
        {
            if ( (time() - $_SESSION['posttimer']) <= 2)
            {
                // less then 2 seconds since last post
            }
            else
            {
                // more than 2 seconds since last post
            }
        }
        $_SESSION['posttimer'] = time();
    }
    
  3. Include a unique token on each POST. In this case, you would also set a session variable to the token you want to include and then render the token in the form. Once the form is submitted, you re-generate the token. When the submitted token does not match the token in your session, the form has been re-submitted and should be declared invalid.

    Example:

    // form.php
    <?php
        // obviously this can be anything you want, as long as it is unique
        $_SESSION['token'] = md5(session_id() . time());
    ?>
    <form action="foo.php" method="post">
        <input type="hidden" name="token" value="<?php echo $_SESSION['token'] ?>" />
        <input type="text" name="bar" />
        <input type="submit" value="Save" />
    </form>
    
    // foo.php
    if (isset($_SESSION['token']))
    {
        if (isset($_POST['token']))
        {
            if ($_POST['token'] != $_SESSION['token'])
            {
                // double submit
            }
        }
    }
    
Calender answered 25/1, 2010 at 17:2 Comment(6)
+1 Great answer, but I'd edit so your recommendation comes first, and then you follow with the methods you don't recommend - would make for easier reading. That said, it should be the accepted answer.Centigram
For the javascript solution, you could create that function for form submit event, not submit button click. That function would be called and would disable the submit button.Coblenz
It seen that method Three is perfectRoundup
I am not sure if this is correct. if you click submit twice on the same form, the same token will be submitted twice. hence, the token in the session needs to be reset once it is used.Itinerancy
also confused as to why submitting the form twice will generate a different token the second time around. Can someone elaborate?Bursa
If anyone interested to know why the token changes when we click submit a second time pls refer to this question #51599985Bursa
P
31

Use a unique token with the post, so that each post/postback is only handled once.

Disabling the submit button is not a very good solution since people could have javascript turned off, or doing other weird things. Client side validation is a good start, but should always be backed up with server side handling as well.

Pickar answered 25/1, 2010 at 17:1 Comment(3)
+1 - this is the most common method, and has no client-side dependencies. Why is this not accepted as the answer?Graphemics
@Steve Not true. POSTs are not expected to be idempotent.Anamorphosis
Refer to this question if you want to know why the token changes when we click "submit" a second time. #51599985Bursa
B
5

Generate a unique, one-time use key to submit with the form.

Relying on Javascript is a bad idea because it may have been disabled in the client by the user. With the key scheme, when the submit is received by the server, submission for that key can be locked. You can respond to duplicate submits however you like.

For this approach to work, keys must be unique and very difficult to predict. Otherwise some forms could be locked due to key collisions. So you don't have to track keys for every form submission and avoid collisions, keys should expire with that user's session. The other thing to watch out for is if malicious users are able to predict the key, your code may be vulnerable to some kind of hijacking or DOS exploit.

Brande answered 25/1, 2010 at 17:6 Comment(2)
This is the approach I use. Generate a GUID whenever the page is rendered, and include this in your database whenever you save a form. In your submit processing code, make sure that GUID doesn't already exist in the db before inserting.Erotic
What type of GUID generation algorithm do you normally use?Brande
S
1

Disable the submit button using Javascript once the form gets submitted (onsubmit).

Note that if the user has disabled Javascript, they can still submit twice. Whether or not this happens often enough to justify also checking in your code (as you suggested in your question) is up to you.

Shirleenshirlene answered 25/1, 2010 at 16:58 Comment(0)
S
1

Alternatively, use one-time form keys.

Smallish answered 25/1, 2010 at 17:2 Comment(0)
I
1

I use this:

$form_token = $_SESSION['form_token'] = md5(uniqid());

send $form_token with the form, then...

I check the form_token:

if($_SESSION['form_token'] != $_POST['form_token']) {
   echo 'Error';
}
Isle answered 25/1, 2010 at 23:35 Comment(0)
M
0

disable the submit button once the user has submitted the form, using JavaScript. That is what, for example, StackOverflow uses when you answer a question.

For example

<input type="submit" onclick="this.disabled=true" />
Markitamarkka answered 25/1, 2010 at 16:58 Comment(0)
D
0

Use a JavaScript to disable the button as soon as it is clicked.

onclick="this.disabled=true;forms[0].submit();"
Daguerre answered 25/1, 2010 at 16:58 Comment(2)
The onclick handler won't run if the form is submitted via a keyboard stroke eg. user hits the enter key while focused on the submit button. Better use onsubmit like one of the other posters suggested.Doxia
@Asaph: I actually like the token idea @Mats suggested. But if were staying client side, it would actually probably be better to have a method that incorporates both click and submit. 2 mouse clicks can register in IE before the onsubmit actually fires. Stupid, but I've seen it.Daguerre
C
0

Pressing submit twice quite rarely results in multiple inserts, but if you want a dead-simple sollution, use

onsubmit="document.getElementById('submit').disabled = true"

in the form tag, provided you give your submit button id="submit"

Caboodle answered 25/1, 2010 at 17:0 Comment(0)
C
0

Another way to create a confirmation page where user can finally press the submit button. It might reduce the possibilities of this kind.

Cayla answered 20/7, 2017 at 7:52 Comment(0)
B
0

I'm doing the following on the action.php

if($_SESSION && isset($_SESSION['already_submitted'])) {

  # If already submitted just redirect to thank you page

} else {

    # If first submit then assign session value and process the form
    $_SESSION['already_submitted'] = true;

    # Process form

}

Note: Always initiate your sessions in the header using session_start();

Bursa answered 30/7, 2018 at 16:28 Comment(0)
S
-1

Aron Rotteveel's Include a unique token on each POST worked for me. However, my form still submits when a user refreshes the page (F5). I managed to solve this by adding a reset:

            // reset session token
            $_SESSION['token'] = md5( $_POST['token'] . time() );

my final scripts goes like:

if (isset($_SESSION['token'])) {
    if (isset($_POST['token'])) {
        if ($_POST['token'] != $_SESSION['token']) {
            echo '<h3>Oops! You already submitted this request.</h3>';
        } else {
            // process form
            // reset session token
            $_SESSION['token'] = md5( $_POST['token'] . time() );
        }
    } else {
        echo 'post token not set';
        $_SESSION['token'] = md5( $somevariable . time() );
    }
}

oh! don't forget to add session_start(); before <!DOCTYPE>

I am a PHP newbie so please correct me if you find irregularities.

Strychnic answered 20/11, 2011 at 7:44 Comment(4)
this is useless code. Just do an HTTP redirect after form submit.Cort
i wish to do that but my form works in a single page. i initially wish to use jquery unload() but i find this useful on occasions user accidentally pressed f5 (page reload). do you have a link in that redirect? i would love to reconsiderStrychnic
your reason makes no sense. you can do an HTTP redirect on the single page as well.Cort
see this my answer for exampe #7215360Cort
P
-1

Use JavaScript to stop it from sending

Pasture answered 27/1, 2012 at 13:0 Comment(1)
Example/demo code would make this a potentially valuable answer.Sexism
S
-2

$(document).ready(function() {
  $('#postHideBtn').show();
  $('#postHideBtn').click(function() {
    $('#postHideBtn').hide();
  });
});
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>

<form action="foo.php" method="post">
  <input type="text" name="bar" />
  <input type="submit" value="Save" id="postHideBtn" />
</form>
Sachsse answered 28/5, 2021 at 6:24 Comment(2)
Please share the context of your code in details too. As the question asked has no context, at least provide yours :)Leontineleontyne
Just spitting out some code is not enough! You should explain it too. See stackoverflow.com/help/how-to-answerLandloper

© 2022 - 2024 — McMap. All rights reserved.