Backport(バックポート) #2296
完了/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される
100%
説明
Overview (現象)¶
/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される。
動作として特に問題があるわけではないが、OpenPNEのアプリケーションとして想定していない画面が表示されてしまうのはあまり適切ではないと思われる。
再現バージョン¶
- OpenPNE3.4.15-dev
再現手順¶
1.pc_frontendにログインする
2.下記のURLにアクセスする
default/index default/module default/error404 default/login default/disabled default/customizingCss ← このURLはホワイトアウトする(devモードでも通常でも)
3.symfonyのデフォルト画面が表示される
Causes (原因)¶
ルーティングルールでdefaultモジュールがすべて有効な状態になっている。
Way to fix (修正内容)¶
調査結果¶
ackによる調査
- default/index
主にユニットテストでの使用のため問題なし。
- default/module
スケルトン作成タスク用のテンプレートで用いられているのみのため動作には問題なし。
- default/error404
主にユニットテストでの使用のため問題なし。
- default/login
pc_backendで用いられているように見えるが、バックポート元と同じ対処を行った場合でも動作には問題がないように思われる。
- default/disabled
ユニットテストでの使用のため問題なし。
- default/secure
該当なし。
- default/customizingCss
該当なし。
Yuya Watanabe さんが13年以上前に更新
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を Yuya Watanabe にセット
Yuya Watanabe さんが13年以上前に更新
本チケットの対象バージョンにおいてはバックポート元の修正方針では不備が生じる可能性があるため再調査を行う必要があります。
Yuya Watanabe さんが13年以上前に更新
バックポート元の調査方法が不明だったため、バックポート元の調査結果に基づいた調査としてforwardやリンクが作成されそうな場所についてマッチするような検索を行いました。以下に調査結果を示します。しかし、下記の調査方法ではバックポート元で示されるような結果が得られなかったため調査方法について検討する必要があると思います。
ackによる調査¶
- default/index
$ ack "'default', *'index'|default/index" -l lib/vendor/symfony/test/unit/routing/sfPatternRoutingTest.php lib/vendor/symfony/test/functional/genericTest.php test/functional/pc_backend/defaultActionsTest.php test/functional/mobile_frontend/defaultActionsTest.php
主にユニットテストでの使用のため問題なし。
- default/module
$ ack "'default', *'module'|default/module" -l lib/task/skeleton/module/module/actions/actions.class.php lib/vendor/symfony/lib/task/generator/skeleton/module/module/actions/actions.class.php
スケルトン作成タスク用のテンプレートで用いられているのみのため動作には問題なし。
- default/error404
$ ack "'default', *'error404'|default/error404" -l lib/vendor/symfony/test/functional/prodTest.php
主にユニットテストでの使用のため問題なし。
- default/login
$ ack "'default', *'login'|default/login" -l lib/vendor/symfony/test/functional/genericTest.php apps/pc_backend/modules/default/templates/loginSuccess.php apps/pc_backend/modules/default/actions/logoutAction.class.php test/functional/pc_backend/snsActionsTest.php test/functional/pc_backend/designActionsTest.php
pc_backendで用いられているように見えるが、バックポート元と同じ対処を行った場合でも動作には問題がないようにみえた。(要検証)
- default/disabled
$ ack "'default', *'disabled'|default/disabled" -l lib/vendor/symfony/test/functional/genericTest.php
ユニットテストでの使用のため問題なし。
- default/secure
$ ack "'default', *'secure'|default/secure" -l
該当なし。
- default/customizingCss
$ ack "'default', *'customizingCss'|default/customizingCss" -l
該当なし。
wa ta さんが13年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 a28d9bad753f83710a2051bdb598cebcd49c320f で適用されました。
Mutsumi Imamura さんが13年以上前に更新
Seleniumにてテストケースを作成し、対象のページにアクセスした際の動作結果は以下の通りです。
■mobile_frontend¶
No. | 確認したページ | 期待結果 | 結果 |
1 | default/csrfError | アクセスエラーページに遷移する | ○ |
2 | default/disabled | アクセスエラーページに遷移する | ○ |
3 | default/error404 | アクセスエラーページに遷移する | ○ |
4 | default/error | アクセスエラーページに遷移する | ○ |
5 | default/index | アクセスエラーページに遷移する | ○ |
6 | default/login | アクセスエラーページに遷移する | ○ |
7 | default/module | アクセスエラーページに遷移する | ○ |
8 | default/nonMobileError | アクセスエラーページに遷移する | ○ |
9 | default/privacyPolicy | アクセスエラーページに遷移する | ○ |
10 | default/secure | アクセスエラーページに遷移する | ○ |
11 | default/urlProxy | アクセスエラーページに遷移する | ○ |
12 | default/userAgreement | アクセスエラーページに遷移する | ○ |
■pc_frontend¶
No. | 確認したページ | 期待結果 | 結果 |
1 | default/csrfError | アクセスエラーページに遷移する | ○ |
2 | default/customizingCss | アクセスエラーページに遷移する | ○ |
3 | default/disabled | アクセスエラーページに遷移する | ○ |
4 | default/error404 | アクセスエラーページに遷移する | ○ |
5 | default/index | アクセスエラーページに遷移する | ○ |
6 | default/login | アクセスエラーページに遷移する | ○ |
7 | default/module | アクセスエラーページに遷移する | ○ |
8 | default/search | アクセスエラーページに遷移する | ○ |
9 | default/secure | アクセスエラーページに遷移する | ○ |
10 | default/privacyPolicy | プライバシーポリシーのページに遷移する | ○ |
11 | default/userAgreement | 利用規約のページ遷移するに遷移する | ○ |
■pc_backend¶
No. | 確認したページ | 期待結果 | 結果 |
1 | csrfError | アクセスエラーページに遷移する | ○ |
2 | default/customizingCss | アクセスエラーページに遷移する | ○ |
3 | default/disabled | アクセスエラーページに遷移する | ○ |
4 | default/error404 | アクセスエラーページに遷移する | ○ |
5 | default/index | アクセスエラーページに遷移する | ○ |
6 | default/login | アクセスエラーページに遷移する | ○ |
7 | default/module | アクセスエラーページに遷移する | ○ |
8 | default/search | アクセスエラーページに遷移する | ○ |
9 | default/secure | アクセスエラーページに遷移する | ○ |
10 | default/top | アクセスエラーページに遷移する | ○ |
Minoru Takai さんが13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
レビューしました。
バージョン 3.4.x に限らず、この修正内容は修正方針として妥当だと思います。仮にデフォルトルートを用いている箇所があったとしても、この修正内容を変更するのではなく、デフォルトルートを用いている側を修正することが適切でしょう。 stable-3.4.x への修正内容の取り込みも問題なく行なわれていることを確認しました。修正 OK です。
Mutsumi Imamura さんが13年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
http://redmine.openpne.jp/issues/2296#note-8 でテスト済みです。
問題ありあませんので完了とします。