Backport（バックポート） #2076: パスワード再発行処理を行なおうとすると http://opauthmailaddress/ に飛ばされてしまう - OpenPNE 3 - OpenPNE Issue Tracking System

Backport（バックポート） #2076

パスワード再発行処理を行なおうとすると http://opauthmailaddress/ に飛ばされてしまう

Masato Nagasawa がほぼ13年前に追加. ほぼ13年前に更新.

ステータス:

Fixed（完了）

優先度:

High（高め）

担当者:

Kousuke Ebihara

対象バージョン:

OpenPNE3.6beta11

開始日:

2010-10-15

期日:

2011-06-24

進捗率:

100%

説明

Overview (現象)¶

1. 「パスワードを忘れた方へ」リンクよりパスワード再発行の申請を行う
2. パスワード再発行メールに
http://sns.example.com//opAuthMailAddress/passwordRecoveryComplete/token/hogehoge/id/XX
というURLが記載されているので、このURLにアクセスする
3. 新しいパスワード入力して、決定ボタンを押下すると、
http://opauthmailaddress/passwordRecoveryComplete/token/hogehoge/id/XX
というURLに飛ばされてしまう

元の報告:
http://twitter.com/77web/statuses/27406372500

確認バージョン¶

3.6beta6 再現した
3.4 未確認

Causes (原因)¶

この問題は base_url に指定する URL の末尾に"/"を含めた場合に発生します。
2系では末尾に"/"が必須でしたが、3系はこれとは逆の仕様であると思われます。
したがって、この仕様について設定ファイル中に明記されていない事が問題であると思われます。
（2系では設定ファイル中にコメントアウトとして記述されています）

Way to fix (修正内容)¶

適切な対応としては、コメントに仕様を記述するよりも、
"/"が末尾に含まれる場合でも正常に動作する方が良いと思いますので、対応可能なように修正を行います。

OpenPNE.ymlがロードされるタイミング(opProjectConfiguration::setOpenPNEConfiguration())で、"/"が含まれていた場合に除去を行います。

この実装の場合、添付されているテストコードではsfConfigを直接書き換えているためエラーのままとなります。
また、op_base_url を書き換えた場合も"/"の除去は機能しません。

URL生成メソッドの opApplicationConfiguration::getAppRouting() 中に処理を記述することで対応は可能ですが、
この場合URL生成時に毎回処理を実行するため無駄な処理が走る事、他の箇所で参照されていた場合を考慮してこの実装は行いません。
(op_base_url を書き換えることは通常考えられない、この実装でも問題ないと考えます)

関連するチケット

関係しているリビジョン

リビジョン ea3e9480 (差分)
Masato Nagasawa がほぼ13年前に追加

fixed remove when "/" was included end of the url in the "op_base_url" config(fixes #2076 BP from #1687)

リビジョン 470652ae (差分)
Masato Nagasawa がほぼ13年前に追加

Revert "fixed remove when "/" was included end of the url in the "op_base_url" config(fixes #2076, BP from #1687)"

This reverts commit 53ddf52e4bca623f388c4a9841a3129d45d1664e.

リビジョン c61bb52d (差分)
Masato Nagasawa がほぼ13年前に追加

fixed remove when "/" was included end of the url in the "op_base_url" config(fixes #2076, BP from #1687)

リビジョン ad9696ce (差分)
Masato Nagasawa がほぼ13年前に追加

fixed remove when "/" was included end of the url in the "op_base_url" config(fixes #2076 BP from #1687)
(cherry picked from commit ea3e9480c3fc8e1d83993b9849f226a0dbb1e7e4)

リビジョン b3e347c5 (差分)
Masato Nagasawa がほぼ13年前に追加

Revert "fixed remove when "/" was included end of the url in the "op_base_url" config(fixes #2076, BP from #1687)"

This reverts commit 53ddf52e4bca623f388c4a9841a3129d45d1664e.
(cherry picked from commit 470652ae2c86d7b72d70dfcfc1a6e5888c3cddea)

リビジョン 64a1b40d (差分)
Masato Nagasawa がほぼ13年前に追加

fixed remove when "/" was included end of the url in the "op_base_url" config(fixes #2076, BP from #1687)
(cherry picked from commit c61bb52ddb7839ac223d758c691eaec5b4064ec9)