Backport(バックポート) #1638
The Parameter will be escape in the mail (メールに含まれるパラメータがエスケープされている)
100%
説明
Overview (現象)¶
ニックネームに「<」や「'」などの記号が含まれると、送られてきたデイリーニュースのメールを表示した際に、HTMLエンティティ(< ' など)に変換されている。
また、ニックネーム以外でもHTMLエンティティ変換される記号が含まれる場合、同様の現象が起こると思われる。
refs: htmlspecialchars
http://php.net/manual/ja/function.htmlspecialchars.php
出力される文字列がこの関数を"何度か"通っていると考えられる。
再現方法¶
- ニックネームを !"#$%&'<<picomu>>?|\-!"#$ にする
- このメンバーでデイリーニュースを受信するように設定しておく
- デイリーニュースタスクを実行する
- 送られてきたデイリーニュースのニックネーム部分を見ると
!"#$%&'<<picomu>>?|\-!"#$
となっている
再現バージョン¶
- OpenPNE3.4.6-dev
他のバージョンでは未確認
Causes (原因)¶
メール本文はHTML形式でないのにも関わらず、エスケープ処理がされている。
Way to fix (修正内容)¶
メールテンプレートではHTMLのようにエスケープ処理を施す必要がない。
エスケープ処理を行っているのは sfViewParameterHolder クラスで、
sfView クラス内でこのインスタンスを生成する際に sfConfig の値を参照してエスケープの有無を決定している。
修正内容としては、sfView で生成された sfViewParameterHolder のインスタンスを、
結果を出力する前に sfViewParameterHolder::setEscaping(false) とする事で、エスケープ処理を行わないようにする。
関連するチケット
関係しているリビジョン
履歴
#1 Masato Nagasawa が13年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 d02fbe8559d029ec9c04ba9376ac6246fa7074d6 で適用されました。
#2 Hiroki Mogi が13年以上前に更新
テスターテストを実施しました。正常に動作していました。
#3 Shogo Kawahara が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
親チケット差し戻しによるステータス変更
#4 Masato Nagasawa が13年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 a3c844c59d3ad9a15a4508dd1219f0c50eac3069 で適用されました。
#5 Masato Nagasawa が13年以上前に更新
更新履歴 f786f143b9a826b032351b8f1a028b639b92c899 で適用されました。
#6 Shogo Kawahara が13年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
#7 Mutsumi Imamura が13年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
再テスト実施しました。テストOKです。