Project

General

Profile

Backport(バックポート) #2725

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

Added by Yuya Watanabe over 7 years ago. Updated about 7 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2011-04-19
Due date:
% Done:

100%


Description

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

Related issues

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

Associated revisions

Revision a1cc33ba (diff)
Added by Yuya Watanabe about 7 years ago

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

BP from #2008
bdca873709d66f3fb999b3fd1efd824a4e1b8ecc

Revision 0e755f38 (diff)
Added by Yuya Watanabe about 7 years ago

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

BP from #2008
d65df79b24955e1302c89b65d982a44758c957a7

Conflicts:

lib/config/opApplicationConfiguration.class.php

Revision fa3559b8 (diff)
Added by Yuya Watanabe about 7 years ago

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

BP from #2008
2ed9c936f62482e4c3d5ea91954b01943873a47c

History

#1 Updated by Yuya Watanabe about 7 years ago

  • Description updated (diff)
  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Yuya Watanabe

#2 Updated by Yuya Watanabe about 7 years ago

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

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

#3 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Fixing(修正待ち)
  • % Done changed from 50 to 0

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

#4 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Pending Fixing(修正待ち) to Accepted(着手)

#5 Updated by Yuya Watanabe about 7 years ago

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

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

#6 Updated by Kousuke Ebihara about 7 years ago

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

#7 Updated by Yuma Sakata about 7 years ago

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

テストOKです。

#8 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Fixed(完了) to Pending Fixing(修正待ち)
  • % Done changed from 100 to 0

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

#9 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Pending Fixing(修正待ち) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#10 Updated by Kousuke Ebihara about 7 years ago

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

#11 Updated by Yuma Sakata about 7 years ago

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

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

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

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

Also available in: Atom PDF