Project

General

Profile

Backport(バックポート) #2088

/default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される

Added by Masato Nagasawa over 11 years ago. Updated over 11 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2011-03-30
Due date:
2011-06-24
% Done:

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

Related to OpenPNE 3 - Bug(バグ) #1987: /default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される Fixed(完了) 2011-03-30
Related to OpenPNE 3 - Backport(バックポート) #2296: /default/indexなどのdefaultモジュールのページにアクセスするとsymfonyのデフォルト画面が表示される Fixed(完了) 2011-03-30

Associated revisions

Revision 5d2626a3 (diff)
Added by Masato Nagasawa over 11 years ago

fixed had access the not defined the default modules in the routing rules(fixes #2088 BP from #1987)

Revision 12b07027 (diff)
Added by Shingo Yamada over 11 years ago

fixed had access the not defined the default modules in the routing rules for non the pc_frontend app (fixes #2088, BP from #1987)

Conflicts:

apps/smartphone_frontend/config/routing.yml

Revision 40a694da (diff)
Added by Masato Nagasawa over 11 years ago

fixed had access the not defined the default modules in the routing rules(fixes #2088 BP from #1987)
(cherry picked from commit 5d2626a3cfcd172c1553290004cca3bd9b22bb20)

Revision 977fc154 (diff)
Added by Shingo Yamada over 11 years ago

fixed had access the not defined the default modules in the routing rules for non the pc_frontend app (fixes #2088, BP from #1987)

Conflicts:

apps/smartphone_frontend/config/routing.yml
(cherry picked from commit 12b070271c6ce1a284681d68ca1c779e8012e6b7)

History

#1 Updated by Masato Nagasawa over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 5d2626a3cfcd172c1553290004cca3bd9b22bb20 で適用されました。

#2 Updated by Kousuke Ebihara over 11 years ago

  • Due date set to 2011-06-24

#3 Updated by Kousuke Ebihara over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

元チケットでおこなわれた指摘事項に関する追加の対応が取り込まれていません。

#4 Updated by Shingo Yamada over 11 years ago

  • Assignee changed from Masato Nagasawa to Shingo Yamada

引き継ぎます

#5 Updated by Shingo Yamada over 11 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

12b070271c6ce1a284681d68ca1c779e8012e6b7 で対応しました。
apps/smartphone_frontend/config/routing.yml の対応は取り込んでおりません。

#6 Updated by Shingo Yamada over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)

#7 Updated by Shingo Yamada over 11 years ago

更新履歴 12b070271c6ce1a284681d68ca1c779e8012e6b7 で適用されました。

#8 Updated by Naoya Tozuka over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

更新箇所確認しました。レビューOKです。

#9 Updated by Mutsumi Imamura over 11 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

動作確認しました。
問題ありません。

Also available in: Atom PDF