広告アラートで意図したwp_ksesフィルタリングが適用されていなかったのを修正 - Githubissues

vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support

https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/

GNU General Public License v2.0

8 stars 1 forks source link

広告アラートで意図したwp_ksesフィルタリングが適用されていなかったのを修正 #1120

Closed mtdkei closed 1 month ago

mtdkei commented 1 month ago

チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）

広告アラートで意図したwp_ksesフィルタリングが適用されていなかったため、iframe等の不要なタグが通過する問題が発生していました。

どういう変更をしたか？

wp_ksesフィルタリングを適切に適用し、許可リストに含まれないタグが通過しないよう修正
data-nosnippet属性がwp_ksesフィルタリングで失われないよう、divタグに属性を追加

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

ソースコードについて

[ ] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[ ] 関数名 / 変数名 / クラス名 / 保存値名はそれだけで内容が想像できるものになっているか？紛らわしい命名になっていないか？
[ ] 関数名 / 変数名 / クラス名 / 保存値名は既存のコードの命名規則に沿ったものになっているか？

デザイン・UI

[ ] 初見のユーザーが予備知識無しで使っても使いやすいようになっているか？
[ ] 情報意味を考慮した意味グルーピング・余白になっているか？
[ ] アラートの表示など追加した場合は他の同様の表示と同じデザインになっているか？

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。書いていない場合は書かない理由を記載してください。

[ ] 書けそうなテストは書いたか？
[ ] 表示要素が仕様通りに表示されない不具合の修正ではない or 表示要素に関する不具合修正の場合テストは書いたか？

その他

[ ] readme.txt に変更内容は書いたか？
[ ] Files changed (変更ファイル)の内容は目視でちゃんと確認したか？
[ ] このチェック項目を機械的にチェックするのではなく本当にちゃんと確認をしたか？
[ ] レビュワーが確認しないでリリースしてしまっても問題ないレベルまでちゃんと作りこみ・確認をしたか？

変更内容について何を確認したか、どういう方法で確認をしたかなど

「ExUnit」→「広告アラート」→「カスタムアラートコンテント」になどのアラート機能に不要なHTMLタグが入った状態で保存した後、「カスタムアラートコンテント」からが消えていることを確認しました。
ソース上では
<iframe src="XXX.XX.XX.XXX:XXXX"></iframe>
のようになっていることを確認しました。

確認URL

（　どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL　）

レビュワーの確認方法・確認する内容など

レビュワーに回す前の確認事項

[ ] このテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー向け

確認して変更が反映されていない場合の確認事項

プルしたか？
ビルドしたか？
ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
npm install したか？
composer install したか？

mtdkei commented 1 month ago

このブランチは不要になったため閉じます