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.

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

enhance(backend): スレッドミュートにおいて、RN/引用/リアクションの通知もミュートするように #15271

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Sayamame-beans
Copy link
Member

@Sayamame-beans Sayamame-beans commented Jan 13, 2025

What

スレッドミュートはこれまで、対象スレッドでの返信通知とメンション通知にのみ作用していましたが、リノート通知と引用通知、リアクション通知に対しても作用するようにしました。

なお、以下のケースでは通知がミュートされません。(ミュートを行っているユーザーをAとします)

  • ミュート対象ノートを、当該スレッドの外にあるAへの返信/メンション付きノートにおいて引用する

これは、ミュートした意図と関係ない会話の中で引用した場合にミュートされてしまうことが無いようにするためです。
逆に言えば、そのようにすることでスレッドミュートを迂回して通知を発生させられてしまうということですが、それが悪質なユーザーによるものであれば、ユーザーのブロック/ミュート、及び通知受け取り範囲の調整が可能であるため問題ないと判断しました。

Why

fix: #13965

Additional info (optional)

  • ユーザー側のWebhookにおいて、ユーザーミュートは作用していないという情報を耳にしました。
    スレッドミュートはWebhookにも適用される処理に(以前から)なっているのですが、これは適用しないべきなのでしょうか?
  • ユーザーの「投稿を通知」において、返信も通知できるように #15269 で新規投稿通知が返信を通知するようになった場合、スレッドミュートはそこにも適用する必要が生じます。
    (現状は返信が含まれないため、スレッドミュートを適用する必要のある箇所がありません。)
  • 本仕様変更に関するテストの追加が出来ていません

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Jan 13, 2025
@Sayamame-beans Sayamame-beans changed the title Enhance thread mute スレッドミュートにおいて、RN/引用/リアクションの通知もミュートするように Jan 13, 2025
@Sayamame-beans
Copy link
Member Author

タイトルを変え忘れたので修正しました…

Copy link
Contributor

このPRによるapi.jsonの差分
差分はありません。
Get diff files from Workflow Page

Copy link

codecov bot commented Jan 13, 2025

Codecov Report

Attention: Patch coverage is 10.25641% with 35 lines in your changes missing coverage. Please review.

Project coverage is 40.33%. Comparing base (d60c307) to head (fb43b2d).

Files with missing lines Patch % Lines
packages/backend/src/core/NoteCreateService.ts 0.00% 22 Missing ⚠️
packages/backend/src/core/ReactionService.ts 23.52% 13 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #15271      +/-   ##
===========================================
+ Coverage    40.22%   40.33%   +0.10%     
===========================================
  Files         1564     1564              
  Lines       198080   198104      +24     
  Branches      3844     3857      +13     
===========================================
+ Hits         79682    79904     +222     
+ Misses      117793   117597     -196     
+ Partials       605      603       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kakkokari-gtyih kakkokari-gtyih changed the title スレッドミュートにおいて、RN/引用/リアクションの通知もミュートするように enhance(backend): スレッドミュートにおいて、RN/引用/リアクションの通知もミュートするように Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR
Projects
Development

Successfully merging this pull request may close these issues.

スレッドをミュートでRNやリアクションの通知も抑制して欲しい
1 participant