Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

通報を受けた時の通知機能を拡充する #13705

Closed
1 task done
samunohito opened this issue Apr 13, 2024 · 12 comments · Fixed by #13758
Closed
1 task done

通報を受けた時の通知機能を拡充する #13705

samunohito opened this issue Apr 13, 2024 · 12 comments · Fixed by #13758
Assignees
@samunohito
Labels
✨Feature This adds/improves/enhances a feature

Comments

@samunohito
Copy link
Member

samunohito commented Apr 13, 2024 •
edited

Summary

サーバのユーザから通報を受けた際、モデレータがそれを知る手段が乏しく以下に限定されています。

  • meta.email に登録されているメールアドレス
  • Streamのadminチャンネルを購読しているクライアント（Misskey本体には使ってる場所が無い？）
  • モデレータ自身がコンパネを目視で監視する

迅速なモデレーションを行うために、通知機能（とその付帯機能）の拡充をした方が良いと考えました。
具体的な案としては以下のようなイメージです。

  • meta.emailだけではなくモデレータにもメールを送信
  • モデレータがログインした瞬間に気づけるような仕組み
    左部メニューにあるコントロールパネルのインジケータ点灯など
  • 通報をすぐに開けるURLの発行とフロント側のルーティング
  • webhookでの通知
    連携先は無限に考えられるので、必要な情報を変数として提供し、それを使ってペイロードのテンプレートを記述してもらうようなイメージで考えている

～↓も考えたけど微妙かも？～

  • モデレータへのDM（rootアカウント→アカウントのDMと任意ののAP連合先アカウント）
  • PWA/ブラウザでの通知

通報がひっきりなしに来るようなシチュエーションも考えられるので、メール・webhook・DM・ブラウザ/PWAでの通知をそれぞれOFFに出来るようなオプションもあると尚良いと思います。

Purpose

  • モデレータが通報に気付きやすくなり、サーバの治安悪化を最小限に抑えられる

Do you want to implement this feature yourself?

  • Yes, I will implement this by myself and send a pull request
@samunohito samunohito added the ✨Feature This adds/improves/enhances a feature label Apr 13, 2024
@syuilo
Copy link
Member

syuilo commented Apr 16, 2024

通報の対処が必要になる規模のサーバーの運営者はSlackやDiscordを使っている場合がほとんどだからwebhookがあれば良さそう

@samunohito samunohito self-assigned this Apr 19, 2024
@samunohito
Copy link
Member Author

やるか…

@samunohito
Copy link
Member Author

今回やる予定のやつ

  • meta.emailだけではなくモデレータにもメールを送信
  • webhookでの通知
  • 通報をすぐに開けるURLの発行とフロント側のルーティング

インジケータ表示も要望あったけど別途やる。
通報の有無を含めて通知表示が4種類あり、これらも含めたほうが良いかどうか検討したいので

misskey/packages/frontend/src/pages/admin/index.vue

Lines 15 to 18 in 860e8bb

<MkInfo v-if="thereIsUnresolvedAbuseReport" warn class="info">{{ i18n.ts.thereIsUnresolvedAbuseReportWarning }} <MkA to="/admin/abuses" class="_link">{{ i18n.ts.check }}</MkA></MkInfo>
<MkInfo v-if="noMaintainerInformation" warn class="info">{{ i18n.ts.noMaintainerInformationWarning }} <MkA to="/admin/settings" class="_link">{{ i18n.ts.configure }}</MkA></MkInfo>
<MkInfo v-if="noBotProtection" warn class="info">{{ i18n.ts.noBotProtectionWarning }} <MkA to="/admin/security" class="_link">{{ i18n.ts.configure }}</MkA></MkInfo>
<MkInfo v-if="noEmailServer" warn class="info">{{ i18n.ts.noEmailServerWarning }} <MkA to="/admin/email-settings" class="_link">{{ i18n.ts.configure }}</MkA></MkInfo>

@syuilo
Copy link
Member

syuilo commented Apr 20, 2024

メールで受け取るかどうかはオプションにする必要があるわね
規模が大きいとものすごい量の(対処不要な)通報がくるため

@samunohito
Copy link
Member Author

webhookにも同様のことが言えると思うので（というかそういう話を聞いたので）、ここで追加するものはon/off切り替え可能なように実装します。

@samunohito
Copy link
Member Author

連携先は無限に考えられるので、必要な情報を変数として提供し、それを使ってペイロードのテンプレートを記述してもらうようなイメージで考えている

これは下記の理由からやめる。ペイロードの形式は固定で。

  • 脆弱性を作りこみそうで怖い
  • 既存のWebhook送信の仕組みを流用したい
  • アプリとしての責任範囲を最小にしたい（参考：GitHubやBacklogなど、大手が提供するサービスの実装の大半は固定）

@samunohito

This comment was marked as off-topic.

Sign in to view
@samunohito
Copy link
Member Author

管理系のwebhookを持つテーブルを新設して、その上に通報のwebhook機能を載せるような感じで考えている
（送信は既存のwebhook送信の仕組みを間借りするようなイメージ）

@syuilo
Copy link
Member

syuilo commented Apr 21, 2024

管理系のwebhookを持つテーブル というと？

@samunohito
Copy link
Member Author

管理系のwebhookを持つテーブル

もともとあるwebhookテーブルではなく、システム契機で作動するwebhookを登録しておくためのsystem_webhook（仮）を作ろうとしていました。システム契機の場合はユーザIDを持たず、またイベントの種類も既存のwebhookとは異なるため、既存のものと混ぜたくないなという思いから。

@syuilo
Copy link
Member

syuilo commented Apr 21, 2024

ほむん

@samunohito
Copy link
Member Author

通報をすぐに開けるURLの発行とフロント側のルーティング

通報一覧画面に対象のIDを引き込めても、バックエンド側との噛み合わせがイマイチで難しいのでいったんオミットする
（対象IDをピンポイントで取得する機能が無い）

@samunohito samunohito mentioned this issue Apr 28, 2024
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@samunohito samunohito

Labels
✨Feature This adds/improves/enhances a feature
Projects
[FEATURE] Admin
Status: Done
Milestone
No milestone
2 participants
@syuilo @samunohito