ブラウザ更新でメールが二重送信される

Question

予約フォームでリロードを実施すると、メールが二重送信されてしまいます。

入力画面⇒確認画面⇒確認画面で送信ボタンを押す⇒完了画面＋完了メール送信
という遷移です。
現在、完了画面でブラウザ更新(F5)を行うとメールが何度も送信されてしまいます。

完了画面でセッションを作成し、リロードしたらセッションで判定を行い、トップページへリダイレクトさせたいと思ったのですが、判定が動作しません。

	<?php
	if ($error_flg !== false) {
	?>
			<div align="center">
				<h1>入力にエラーがあります。下記をご確認の上「戻る」ボタンにて修正をお願い致します。</h1>
				<br/>
				<br/>
				<div style="color:red"><?php echo $error_msg; ?></div>
				<br/>
				<br/>
				<input type="button" value=" 前画面に戻る " onClick="history.back()">
			</div>
	<?php
	} else {
		if(isset($_SESSION['reloadno'])){
			header('location: /');
			exit();
		}else{
		  $_SESSION['reloadno'] = "リロード判定";
		  print_r($_SESSION);
		  echo 'リロードしてない';
		}
	?>
			<h1>ご予約を承りました</h1>

実際に表示される画面

Array ( [reloadno] => リロード判定 ) リロードしてない
ご予約を承りました

セッション判定の書き方に不備があると思うのですが、分からない状況です。
お力を貸していただきたいです。

Comments

[m.ts10806]

そもそも二重送信を防止しては

[meli]

二重送信防止で調べたところ、この方法が出来そうかなと思っただけなので、
他の方法があればご教授いただきたいです。

Answer 1

完了画面で完了メールの送信をするのやめたらいいんじゃないですか？

確認画面で送信ボタンを押す、完了メール送信 ⇒ 完了画面へリダイレクト

ではダメなんですかね？
これで良ければ、完了画面をリロードしても、<h1>ご予約を承りました</h1> が表示されるだけで
終わります。

ただ、そのままでは直接完了画面のURLを打たれた時にも完了画面が表示されてしまいますから、
何らかの制御はした方がいいかもしれません。
それは別件なので深くは触れませんが、それこそおっしゃる通り、トップ画面に戻すなどですね。

Comments

[meli]

ご回答ありがとうございます。
やはり完了ページを分けた方がいいですよね。
その場合、ブラウザバックをしたときに確認画面がリセットされないため、
入力者が送信ボタンを押すとメールが送信されてしまうので、
そこをリセットをかけるか、別ページに遷移したいと言われてしまいました…。
ブラウザバックでトップページにリダイレクトなどは可能なのでしょうか？

[root_jp]

そもそもボタンを複数回押されて同じ処理が走ってしまうのがまずいです。
ブラウザバックで戻って再度ボタンを押すという事をしなくても、
ボタンをカチカチと連打すれば２回処理がいってしまうからです。
一般的にはトランザクショントークンというものを使って防ぎますので、根本的に解決したいなら調べてみてください。

[meli]

そうなんですよね…。
二重submit対策について調べた方がよさそうですね。
ありがとうございます。

Answer 2

PHP Session config session.auto_start より：

session.auto_start specifies whether the session module starts a session automatically on request startup. Defaults to 0 (disabled).

翻訳：session.auto_start はリクエストを起動する時sessionを自動的にスタートするかを指定します。デフォルトは0（しない）。

この設定を確認してください。問題文を見ると、session_start()を呼び出していませんので、この設定もデフォルトのままだとしたら、$_SESSIONはただの普通の変数となっています。リロード判定も保存してないのです。

Comments

[meli]

ご回答ありがとうございます。
省略してしまいましたが、完了処理の上部にsession_start();の記述は行いました。

[YufanLou]

$_SESSION['reloadno'] = "リロード判定";
print_r($_SESSION);
この2行の上もprint_r($_SESSION);したら出力はなんですか？

[meli]

if文内でのprint_rでしょうか？
その場合、if文に入らないので出力できないです。