メールアドレスに + が含まれる場合にパスワード変更ができない

[wkoichi]

会員がパスワードを忘れた際のパスワード再設定機能について、
会員のメールアドレスに + が含まれる場合 (例： aaa+b@example.com ) 、パスワード再設定の通知メール本文内の URL にアクセスすると必ず「Invalid request 2」のメッセージが表示され、パスワード変更フォームが表示されません。

原因は usc-e-shop/classes/usceshop.class.php の uscesmode_changepassword メソッド内で以下のようにHTTPリクエスト変数の値を urldecode 関数で URLデコードしていること、のように思われます。HTTPリクエスト変数の値はURLデコード済みであるはずですので、結果として二重にURLデコードしてしまっているのではないでしょうか。

$mem_mail = urldecode($_REQUEST[‘mem’]);
$lostkey = urldecode($_REQUEST[‘key’]);

私の手元の環境では上記箇所で urldecode を噛ませないようにしたところ、問題のメールアドレスでも正常にパスワード再設定を完了できました。

ご確認のほどよろしくお願い致します。

——————————————-
WordPress のバージョン： 4.3.1
Welcart のバージョン： 1.6.2
ご利用のテーマ： welcartデフォルト
症状を確認したブラウザ： chrome
サーバー（会社名、サービス名）：
SSLの利用：　
WordPress のパーマリンク設定：
——————————————–

[kitamuuu]

こんにちは。

仰られている箇所ではメールに届いたURLリンクのパラメータから取得した値をurldecodeしている箇所ですので二重decodeという事にはなりえません。

＋つきのメールアドレスアカウントを作成できませんでしたので、こちらでは通常のメールアドレスでしか確認しておりませんが、「Invalid request 2」というエラーは古いリンクURLからアクセスした場合にも表示されますので、一番新しくメールに届いたリンクからアクセスしているかご確認いただけますでしょうか。

[wkoichi]

kitamuuuさん

ご返信ありがとうございました。

一番新しくメールに届いたリンクからアクセスしても「Invalid request 2」となります。

二重decodeという事にはなりえません

とありますが、$_REQUESTの値はPHPの処理系がその値をセットした時点でデコード済みになっているはずです。$_REQUESTの値を urldecode 関数でデコードしてしまうと通常は二重デコードになってしまいます。以下のPHPマニュアルを参照してみてください。
http://php.net/manual/ja/function.urldecode.php

• この返信は8年、 11ヶ月前にwkoichiが編集しました。

[kitamuuu]

wkoichiさん

PHPのマニュアルも含め、改めて確認しましたところ仰る通り2重デコードされておりました。
ご指摘ありがとうございます。

この件はWelcartの次期バージョン1.6.4にて修正されますが、もし修正をお急ぎでしたら以下のURLより、リリース前の「Development Version」がzipでダウンロードできますので、そちらをダウンロードしてお使いください。
(ダウンロードリンクは一番下にあります)

https://wordpress.org/plugins/usc-e-shop/download/

[wkoichi]

ご確認ありがとうございました。
次期バージョンのリリースを待ちたいと思います。

お手数かけますが、ご対応のほどよろしくお願いします m(_ _)m

[投稿者]

投稿