プロジェクト

全般

プロフィール

Bug(バグ) #1632

[PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分

Hidenori Goto13年以上前に追加. 約7年前に更新.

ステータス:
Won't fix(対応せず)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-10-01
期日:
進捗率:

0%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

説明

状況

1. 末尾スラッシュの除去処理がサブディレクトリのプレフィックスを考慮していない。

opApplicationConfiguration::getAppScriptName()の以下の処理

    if ('/' === $prefix)
    {
      $prefix = '';
    }

は、後続の処理でスラッシュとファイル名を連結するために$prefixの末尾のスラッシュを除去するための処理と思われるのですが、ルートディレクトリのパターンでしか処理されていません。

2. $isNoScriptNameがtrueの場合とのインターフェイスの整合性

$isNoScriptNameがtrueの場合、$prefixをそのままreturnしていますが、$prefixの末尾のスラッシュの扱いについて、このメソッドの戻り値の一貫性が崩れていると思われます。
(このメソッドからは、末尾スラッシュなしのパスが返されるというインターフェイスになっているという認識)

※sfRouting::fixGeneratedUrl()でprefixとの結合を行っていますが、ここでprefixの末尾にスラッシュがないことを前提としているのかどうかまでは分からなかったため、調べています。

対応

パッチを添付しました。

0001-fixed-opApplicationConfiguration.patch 表示 (1.01 KB) Hidenori Goto, 2010-10-01 10:16


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1577: OpenPNE.ymlのbase_url にパスが含まれない場合に、デバッグモードでwarningメッセージが表示される場合がある Fixed(完了) 2010-09-10
コピー先 OpenPNE 3 - Backport(バックポート) #3633: [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 Fixed(完了) 2010-10-01
コピー先 OpenPNE 3 - Backport(バックポート) #3658: [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 Fixed(完了) 2014-07-15

関係しているリビジョン

リビジョン a9de2d32 (差分)
誠二 天重ほぼ10年前に追加

fixed removal of trailing slash is not enough. (refs #1632)

リビジョン 598b1822
Shinichi Urabeほぼ10年前に追加

Merge pull request #143 from amashigeseiji/t-1632-trailing-slash

fixed removal of trailing slash is not enough. (refs #1632)

履歴

#2 Kousuke Ebihara13年以上前に更新

  • 対象バージョンOpenPNE 3.7.0 にセット

#3 Shogo Kawaharaほぼ13年前に更新

  • 題名opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 から [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 に変更

#4 Shouta Kashiwagi約12年前に更新

  • 対象バージョンOpenPNE 3.7.0 から 252 に変更

#5 Shouta Kashiwagi約12年前に更新

  • 対象バージョン252 から OpenPNE 3.8.x に変更

#6 誠二 天重ほぼ10年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 担当者誠二 天重 にセット
  • 進捗率0 から 50 に変更
  • 3.6 で発生するかUnknown (未調査) にセット
  • 3.8 で発生するかUnknown (未調査) にセット

パッチの内容で以下にプルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/143

#7 Shinichi Urabeほぼ10年前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.12 に変更

#8 誠二 天重ほぼ10年前に更新

#9 誠二 天重ほぼ10年前に更新

  • 対象バージョンOpenPNE 3.8.12 から OpenPNE 3.8.x に変更

#10 誠二 天重ほぼ10年前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.9.0-old に変更

#11 Shinichi Urabeほぼ10年前に更新

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

レビューOKです https://github.com/openpne/OpenPNE3/pull/143 を取り込みました

#12 isao sanoほぼ10年前に更新

#14 isao sano約7年前に更新

  • ステータスPending Testing(テスト待ち) から Won't fix(対応せず) に変更
  • 進捗率70 から 0 に変更

OpenPNE 3.8.12 にて対応済みであったため、対応せずとします。

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