プロジェクト

全般

プロフィール

Backport(バックポート) #2725

config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない

Yuya Watanabe12年以上前に追加. 約12年前に更新.

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

100%


説明

Overview (現象)

config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない

Environment (再現バージョン)

OpenPNE3.6beta8

Way to repro (再現手順)

  1. config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定
  2. 招待メールを送ってみる
  3. 招待状本文中の「〜〜SNS に参加する」URLのリンクにポート番号が含まれない

例:

base_url: "http://sns.openpne.jp:8080" 
...
■ ××××SNS に参加する
http://sns.openpne.jp/member/register/token/xyxyxyxyxyxyxyxy
...

Causes (原因)

parse_url(sfConfig::get('op_base_url')) の戻り値配列に含まれる port が使われていない

Way to fix (修正内容)

  • parse_url() の戻り値配列に含まれる port を保持。
diff --git a/lib/config/opApplicationConfiguration.class.php b/lib/config/opApplicationConfiguration.class.php
index f1e8d92..cd29665 100644
--- a/lib/config/opApplicationConfiguration.class.php
+++ b/lib/config/opApplicationConfiguration.class.php
@@ -598,6 +598,10 @@ abstract class opApplicationConfiguration extends sfApplicationConfiguration
       if (isset($parts['host']))
       {
         $options['context']['host'] = $parts['host'];
+        if (isset($parts['port']))
+        {
+          $options['context']['host'] .= ':'.$parts['port'];
+        }
       }
     }
     else

関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない Fixed(完了) 2011-04-19

関係しているリビジョン

リビジョン a1cc33ba (差分)
Yuya Watanabe約12年前に追加

(fixes #2725, BP from #2008) add port to generated url

BP from #2008
bdca873709d66f3fb999b3fd1efd824a4e1b8ecc

リビジョン 0e755f38 (差分)
Yuya Watanabe約12年前に追加

(fixes #2725, BP from #2008) fixed to generate url considering SSL and the port

BP from #2008
d65df79b24955e1302c89b65d982a44758c957a7

Conflicts:

lib/config/opApplicationConfiguration.class.php

リビジョン fa3559b8 (差分)
Yuya Watanabe約12年前に追加

(fixes #2725, BP from #2008) fixed to flip default flag to evaluate correct boolean

BP from #2008
2ed9c936f62482e4c3d5ea91954b01943873a47c

履歴

#1 Yuya Watanabe約12年前に更新

  • 説明 を更新 (diff)
  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Yuya Watanabe にセット

#2 Yuya Watanabe約12年前に更新

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

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

#3 Yuya Watanabe約12年前に更新

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

バックポート元のステータスが「Pending Testing」になるまで本チケットのステータスを「Pending Fixing」にしておきます.

#4 Yuya Watanabe約12年前に更新

  • ステータスPending Fixing(修正待ち) から Accepted(着手) に変更

#5 Yuya Watanabe約12年前に更新

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

更新履歴 0e755f38d24a3f40026db2a9ab3e1b9cf97f578c で適用されました。

#6 Kousuke Ebihara約12年前に更新

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

#7 Yuma Sakata約12年前に更新

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

テストOKです。

#8 Yuya Watanabe約12年前に更新

  • ステータスFixed(完了) から Pending Fixing(修正待ち) に変更
  • 進捗率100 から 0 に変更

バックポート元で差し戻されているため,ステータスを「Pending Fixing」に戻します.

#9 Yuya Watanabe約12年前に更新

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

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

#10 Kousuke Ebihara約12年前に更新

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

#11 Yuma Sakata約12年前に更新

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

テストOKですが、気になる点がありましたので確認お願いします。

ポート設定しない場合、デフォルト値と同じホストが表示されないか確認した場合

  • 試験手順
    0. config/OpenPNE.yml でポート番号を含まないデフォルト値と同じURL(例:http://example.com)をbase_urlに指定しておく
    1. 招待ページ(/invite)にアクセスする
    2. PCメールアドレスに招待メール送信する
    3. 受信した招待メールを確認する
  • 試験結果
    アクセスするホストと同じホストが表示されるが、署名URLが http://example.com で表示されます。
  • 備考
    ssl_base_urlでも同様の現象が発生します。

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