Actions
Backport（バックポート） #2726
closed
config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
Start date:
2011-04-19
Due date:
% Done:
100%
Estimated time:
Description
Overview (現象)¶
config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
Environment (再現バージョン)¶
OpenPNE3.6beta8
Way to repro (再現手順)¶
- config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定
- 招待メールを送ってみる
- 招待状本文中の「〜〜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 と表示される。
Updated by Yuya Watanabe almost 13 years ago
- Status changed from New（新規） to Accepted（着手）
- Assignee set to Yuya Watanabe
Updated by Yuya Watanabe almost 13 years ago
バックポート元では ssl_base_url も考慮した実装を行ったが， OpenPNE 3.4 ではSSLに対応した設定項目が無いため一部コミットのみを適用する．
Updated by Yuya Watanabe almost 13 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
更新履歴 94c54045f28df79c9e79567eea8678ad81db93aa で適用されました。
Updated by Kousuke Ebihara almost 13 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
Updated by Yuma Sakata almost 13 years ago
- Status changed from Pending Testing（テスト待ち） to Fixed（完了）
- % Done changed from 70 to 100
テストOKです。
携帯メールに署名が反映されないバグがありました。
実装者と検討した結果、別チケット #2783 で対応することになりました。
Actions