Project

General

Profile

Backport(バックポート) #2726

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

Added by Yuya Watanabe over 9 years ago. Updated about 9 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 を保持。(lib/config/opApplicationConfiguration.class.php)
  • sfPatternRoutingを継承したクラス opPatternRouting で修正。symfony側(sfRouting)には手をつけない。

修正パッチ: https://github.com/tozuka/OpenPNE3/commit/1da0e7306a7815a8502f7524a6b26f1436096bd7

備考

config/OpenPNE.yml.sample のように

base_url: "http://example.com" 
が指定されていると、実際のリクエストからURLをポート番号込みで読み取って絶対URLを生成してくれる実装になっているが、op_base_url までは書き換えてくれないのでメールのフッタには http://example.com と表示される。


Related issues

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

Associated revisions

Revision 94c54045 (diff)
Added by Yuya Watanabe about 9 years ago

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

BP from #2008
modified bdca873709d66f3fb999b3fd1efd824a4e1b8ecc

History

#1 Updated by Yuya Watanabe about 9 years ago

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

#2 Updated by Yuya Watanabe about 9 years ago

バックポート元では ssl_base_url も考慮した実装を行ったが, OpenPNE 3.4 ではSSLに対応した設定項目が無いため一部コミットのみを適用する.

#3 Updated by Yuya Watanabe about 9 years ago

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

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

#4 Updated by Kousuke Ebihara about 9 years ago

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

#5 Updated by Yuma Sakata about 9 years ago

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

テストOKです。

携帯メールに署名が反映されないバグがありました。
実装者と検討した結果、別チケット #2783 で対応することになりました。

Also available in: Atom PDF