OpenPNE 3

当チケットの修正のための Pull Request を送信しました。
https://github.com/openpne/OpenPNE3/pull/282

修正内容の詳細は下記の通りです。

(参考) 通常の登録フロー¶

[30/Oct/2015:21:01:47 +0900] "GET /member/register/token/230144fcee6e171ed5ff531f7d4007efa HTTP/1.1" 200 2206
[30/Oct/2015:21:01:57 +0900] "GET /member/registerInput/token/230144fcee6e171ed5ff531f7d4007efa HTTP/1.1" 200 3868
[30/Oct/2015:21:03:49 +0900] "POST /member/registerInput/token/230144fcee6e171ed5ff531f7d4007efa HTTP/1.1" 302 929
[30/Oct/2015:21:03:49 +0900] "GET /opAuthMailAddress/registerEnd/token/230144fcee6e171ed5ff531f7d4007efa HTTP/1.1" 302 685
[30/Oct/2015:21:03:50 +0900] "GET / HTTP/1.1" 200 3696

プロフィール入力画面 (member/registerInput) の「送信」ボタンをクリックすると、
"POST /member/registerInput" → "GET /opAuthMailAddress/registerEnd" → "GET /" とリダイレクトされホーム画面が表示されます。

IE7 で member/registerInput の「送信」ボタンを連打した場合¶

[30/Oct/2015:21:29:07 +0900] "GET /member/register/token/33668edc9e77cff2cadcaabcdf5392e59 HTTP/1.1" 200 2213
[30/Oct/2015:21:29:10 +0900] "GET /member/registerInput/token/33668edc9e77cff2cadcaabcdf5392e59 HTTP/1.1" 200 3699
[30/Oct/2015:21:30:04 +0900] "POST /member/registerInput/token/33668edc9e77cff2cadcaabcdf5392e59 HTTP/1.1" 302 929
[30/Oct/2015:21:30:04 +0900] "POST /member/registerInput/token/33668edc9e77cff2cadcaabcdf5392e59 HTTP/1.1" 404 3782
[30/Oct/2015:21:30:05 +0900] "POST /member/registerInput/token/33668edc9e77cff2cadcaabcdf5392e59 HTTP/1.1" 404 3782

プロフィール入力画面 (member/registerInput) で「送信」ボタンを連打すると "POST /member/registerInput" が連続して発行されます。
"POST /member/registerInput" は 2 回目以降のリクエストでは、$this->getUser()->isRegisterBegin() が false になるため 404 エラーが返されます。
この場合、"GET /opAuthMailAddress/registerEnd" に遷移しないままエラー画面で止まるためメンバー登録は完了しません（is_active = 0 のまま）

このユーザーはリダイレクト等で "GET /opAuthMailAddress/registerEnd" に誘導することができればメンバー登録を完了できます（アドレスバーに直接入力しても正常に完了します）

Firefox で member/registerInput の「送信」ボタンを連打した場合¶

[02/Nov/2015:20:36:22 +0900] "GET /member/register/token/48c745a912ad5add51a0d9d197d234908 HTTP/1.1" 200 2321
[02/Nov/2015:20:36:26 +0900] "GET /member/registerInput/token/48c745a912ad5add51a0d9d197d234908 HTTP/1.1" 200 3706
[02/Nov/2015:20:36:53 +0900] "POST /member/registerInput/token/48c745a912ad5add51a0d9d197d234908 HTTP/1.1" 302 961
[02/Nov/2015:20:36:54 +0900] "GET /opAuthMailAddress/registerEnd/token/48c745a912ad5add51a0d9d197d234908 HTTP/1.1" 302 685
[02/Nov/2015:20:36:54 +0900] "POST /member/registerInput/token/48c745a912ad5add51a0d9d197d234908 HTTP/1.1" 404 3948

Firefox からプロフィール入力画面 (member/registerInput) で「送信」ボタンを連打すると、"POST /member/registerInput" へのリクエストが複数回行われる点は同じですが、その間に裏で "GET /opAuthMailAddress/registerEnd" へのリダイレクトも行われています。
見かけ上は最後の "POST /member/registerInput" の 404 エラーに遷移して止まる点で他のブラウザと同じですが、"GET /opAuthMailAddress/registerEnd" へのリクエストが行われているためメンバー登録が完了している状態となっています（is_active = 1 の状態）

このユーザーはメンバー登録が完了しログイン済みの状態となっているため、リダイレクト等で "GET /" に誘導できれば正常にホーム画面が表示されます。

修正内容¶

--- a/lib/action/opMemberAction.class.php
+++ b/lib/action/opMemberAction.class.php
@@ -170,6 +170,9 @@ public function executeRegisterInput(opWebRequest $request)
   {
     $this->forward404Unless(opToolkit::isEnabledRegistration((sfConfig::get('app_is_mobile') ? 'mobile' : 'pc')));

+    $this->redirectIf($this->getUser()->isSNSMember(), array('sf_route' => 'homepage'));
+    $this->redirectIf($this->getUser()->isRegisterFinish(), $this->getUser()->getRegisterEndAction());
+
     $this->token = $request['token'];
     $member = $this->getUser()->setRegisterToken($this->token);

それぞれ、$this->getUser()->isMember() は Firefox 向け、$this->getUser()->isRegisterFinish() はその他のブラウザ向けのリダイレクトとなっています。
setRegisterToken() より前に追加しているため、この判定は URL に含まれる register_token ではなく Cookie に含まれる現在のセッションに紐付くユーザーに対して判定されます。

みました

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#3150-8 のリダイレクトによる対処では IE10 での検証でエラー表示を防ぐことができなかったため、クライアント側で JavaScript によってフォームの二重送信を抑制する方法に変更して改めて修正を行いました。

下記 Pull Request にて修正しました。
https://github.com/openpne/OpenPNE3/pull/468

• 古い JS ファイルがブラウザキャッシュに残っているとキャッシュが削除されるまで動作しないと思いますので、 apps/pc_frontend/config/view.yml の記述を例えば、 util.js?20170605 とさせるなど対策をしておいたほうがよりよいと思います
• 再現するか確認が難しいかもしれませんが、フィーチャーフォン向けの対策は不要でしょうか。

モバイルシミュレータ (Firefox の アドオン) ですが 、メンバー登録時の携帯個体識別番号取得設定 を 「登録時に取得しない」しておくと、再現はしました [取得する設定の場合、アラートがでるので、連投の抑止になるため]

member/registerInput のフォームについて F001 (au), KC3Q (au) の各端末で確認した所、ボタンの連打によって複数のリクエストが送信されることはなく、登録も正常に完了しました。

確認ありがとうございました
レビューOKです

マージしました