「信頼レベル unless mediaをレビュー」設定により絵文字付きのレビューが発生

バグ
1

当サイトでは、特定の信頼レベルのユーザーが画像を投稿した場合にフォーラム投稿をレビューするために、review_media_unless_trust_level 設定を使用しています。これはうまく機能しますが、絵文字とプレーンテキストのみを含む投稿のレビューも引き起こすため、このフィルターは私の意見では強すぎます。

埋め込みリンクの場合のように、レビューがそれ以外の場合は「非画像ケース」でトリガーされるのは非常に良いことなので、フィルターを一般的に非常に敏感に保つことは素晴らしいことです。しかし、私が知る限り、サイト上のすべての絵文字はフォーラム自体によってホストされ、決定されているため、人々が好きな絵文字を投稿することを許可することに危険はないと思います。:rocket: :sunglasses:

この設定で、絵文字とプレーンテキストのみを含む投稿を無視することは可能でしょうか？

関連情報：

[編集：この投稿のタイトルで、単語間にアンダースコアを使用して、適切な設定名を使用できませんでした。フォーラムは、投稿のタイトルが曖昧で「長すぎる単語」のようだと私に伝えました。]

「いいね！」 7
2

さらに悪いことに、非ASCII Unicode文字のために投稿が保留されている可能性があります。レビューキューに入った投稿があり、原因として考えられるのは引用符に“と”を使用していることだけです。

あと、名前の一部としてãも使われています。それが原因でないことを願います！

4

返信が遅くなり申し訳ありません。テストサイトでこれを実行してみたところ、絵文字でトリガーすることができ、これは過敏すぎるように思えます。それらが不可分に結びついているかどうかはわかりませんが、調べてみます。:+1:

しかし、引用符や ã ではトリガーできませんでした。何か見落としていることはありますか？

image
image1035×653 18.2 KB

「いいね！」 3
5

よくわかりません――レビューキューにある投稿で、それらしか考えられないと思ったのですが。

「いいね！」 1
11

そうは思えません。以下をご覧ください。

画像サイズが存在することを要求しています…残念ながら、絵文字はサイズを持つため画像とみなされ、トリガーされてしまいます。

修正は比較的簡単に行えるはずですが、サイズを決定する際に画像を絵文字から分離する内部トラッキングを導入する必要があります。

12

こちらがその例です：

https://meta.discourse.org/t/why-did-this-post-get-held-for-moderation/253048/2?u=mattdm

（一般ユーザーの方には、公開トピックにプライベートリンクを貼ってしまい申し訳ありません。誰かの名前が含まれているため、機密情報として扱っています）

13

いいえ、含んでいません。でも、どうでもいい :wink:

14

はい、ご指摘の通りです。:slight_smile: リンク自体には名前がありませんが、リンク先のコンテンツには名前があります。:slight_smile:

15

引用内のアバター（スクリーンショットでは機密情報のため非表示になっています :man_facepalming:_）のことですね。
同じ問題ですが、別のファミリーです。そちらも対応します。作業は今後20営業日分キューに入っています。

  • 絵文字画像はカウントしない
  • 引用内のユーザーアバター画像はカウントしない
「いいね！」 3
17

これをざっと見てみたのですが、問題は FEATURE: Review posts with media. (#10693) · discourse/discourse@f85f73b · GitHub にあるようです。なぜなら、クライアントから送信された imageSizes を使用して review media unless trust level をチェックしているからです。

代わりに、サーバーにチェックを任せるべきで、これに似たことを行うべきです。

ユーザーが投稿に追加したメディアをチェックする前に。

ただし、NewPostManager で「nokogiri-ed」バージョンの調理済み投稿にアクセスできるかどうかわかりません。もしアクセスできない場合、パフォーマンスへの影響はどれくらいでしょうか？ :thinking:

「いいね！」 2
19

これは少し奇妙なやり方です。クライアント側では、プレビューをクエリすることによって「画像」を検出していますが、これも理想的ではありません。
https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/controllers/composer.js#L1008-L1018

ここは大幅なリファクタリングの余地がありますが、とりあえず、絵文字やアバターの imageSizes をペイロードに送信しないようにすることができます。絵文字やアバターには imageSizes が全く使用されていないことを確認したので、これは安全なようです: FIX: Don't send image sizes for emojis/avatars by pmusaraj · Pull Request #20589 · discourse/discourse · GitHub

PRの説明にあるように、onebox の画像も投稿をレビューキューに送信しますが、この設定が有効になっている場合、それは望ましい副作用のように思えます。

「いいね！」 3