操作
Backport（バックポート） #2726
完了
config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
開始日:
2011-04-19
期日:
進捗率:
100%
予定工数:
説明
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 と表示される。
Yuya Watanabe さんがほぼ13年前に更新
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Yuya Watanabe にセット
Yuya Watanabe さんがほぼ13年前に更新
バックポート元では ssl_base_url も考慮した実装を行ったが， OpenPNE 3.4 ではSSLに対応した設定項目が無いため一部コミットのみを適用する．
Yuya Watanabe さんがほぼ13年前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 94c54045f28df79c9e79567eea8678ad81db93aa で適用されました。
Kousuke Ebihara さんがほぼ13年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
Yuma Sakata さんがほぼ13年前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
テストOKです。
携帯メールに署名が反映されないバグがありました。
実装者と検討した結果、別チケット #2783 で対応することになりました。
操作