Bug(バグ) #1987
/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される
100%
Description
Overview (現象)¶
/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される。
動作として特に問題があるわけではないが、OpenPNEのアプリケーションとして想定していない画面が表示されてしまうのはあまり適切ではないと思われる。
再現バージョン¶
- OpenPNE3.6beta8
再現手順¶
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 (修正内容)¶
ルーティングルールの既存defaultルール以外はdefault/errorに遷移させる。
その際、defaultアクションのルーティングを無効にして問題が起きないか調査する必要がある。
調査結果¶
URI | 調査結果 |
default/index | 恐らく使用箇所はないため問題ない |
default/module | アクション自動生成時にforwardされるため無効にして問題ない |
default/error404 | forwardのみのアクセスため問題ない |
default/login | 恐らく使用箇所はないため問題ない |
default/disabled | モジュール無効時のforwardのみのアクセスのため問題ない |
default/secure | SNSMemberクレデンシャル存在しない状態の場合のforwardのみのアクセスのため問題ない |
default/customizingCss | カスタマイズCSSの結果を返すが、customizing_cssのルールを使用しているため無効にして問題ない |
以上の結果から新規に追加する必要のあるルールは存在しないことが分かったので、
以下を既存defaultルールの最後に記述して、無効化を行う。
no_default: url: /default/* param: { module: default, action: error }
Related issues
Associated revisions
fixed had access the not defined the default modules in the routing rules(fixes #1987)
fixed had access the not defined the default modules in the routing rules for non the pc_frontend app (fixes #1987)
History
#1
Updated by Shingo Yamada over 8 years ago
- Priority changed from Normal(通常) to High(高め)
#2
Updated by Shingo Yamada over 8 years ago
- Assignee set to Masato Nagasawa
#3
Updated by Masato Nagasawa over 8 years ago
- Status changed from New(新規) to Accepted(着手)
- Target version set to OpenPNE 3.7.0
#4
Updated by Masato Nagasawa over 8 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
更新履歴 c5457aff5130f6afa431e23fa9965024d55f737f で適用されました。
#5
Updated by Naoya Tozuka over 8 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
変更点確認しました。OKです。
#6
Updated by Mutsumi Imamura over 8 years ago
- Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
- % Done changed from 70 to 50
再現手順にはpc_frontendのみ記述していましたが、pc_backend、mobile_frontendでも再現しますので対応をお願いします。
#7
Updated by Masato Nagasawa over 8 years ago
- Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
更新履歴 4cd558bca89b8adb9dd24ad77bcfbb91e1acfa2f で適用されました。
api, mobile_frontend, pc_backend, smartphone_frontend に対して同様の対応を行いました。
#8
Updated by Naoya Tozuka over 8 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
修正OKです。
// smartphone_frontend につきましては、現状ではメンテナンスされておらずリリースにも含まれないため、対応・テストは不要と思われます。
#9
Updated by Fumie Toyooka about 8 years ago
- Status changed from Pending Testing(テスト待ち) to Fixed(完了)
- % Done changed from 70 to 100
対象のページにアクセスした際の動作結果は以下の通りです。
■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 | アクセスエラーページに遷移する | ○ |
テスト無事完了しました。
#10
Updated by kaoru n about 4 years ago
- 3.8 で発生するか set to Unknown (未調査)