プロジェクト

全般

プロフィール

Backport(バックポート) #2088

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

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

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2011-03-30
期日:
2011-06-24
進捗率:

100%


説明

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 }


関連するチケット

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

関係しているリビジョン

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

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

リビジョン 12b07027 (差分)
Shingo Yamadaほぼ13年前に追加

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

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

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

リビジョン 977fc154 (差分)
Shingo Yamadaほぼ13年前に追加

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)

履歴

#1 Masato Nagasawaほぼ13年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#2 Kousuke Ebiharaほぼ13年前に更新

  • 期日2011-06-24 にセット

#3 Kousuke Ebiharaほぼ13年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

#4 Shingo Yamadaほぼ13年前に更新

  • 担当者Masato Nagasawa から Shingo Yamada に変更

引き継ぎます

#5 Shingo Yamadaほぼ13年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更

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

#6 Shingo Yamadaほぼ13年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更

#7 Shingo Yamadaほぼ13年前に更新

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

#8 Naoya Tozukaほぼ13年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

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

#9 Mutsumi Imamuraほぼ13年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

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

他の形式にエクスポート: Atom PDF