Fix: フォローしたユーザが、自分のフォローしていないチャンネルにノートを投稿した時、そのノートがHTLで見えてしまう

[samunohito]

What

#12181 の対処です。
もともと #12179 の対応を入れ込みましたが、このときに作成したクエリに漏れがありました。

フォローしたユーザ or フォローしたチャンネル

という条件だったのですが、上記だとフォローしたユーザのノートであれば未フォローのチャンネルに対するノートでも拾ってしまうようになってしまっていました。

（フォローしたユーザ and チャンネル投稿ではない） or フォローしたチャンネル

上記のようにすることでこの現象に対処しています。

Why

フォローしているユーザとはいえ、フォローしていないチャンネルの内容が見えるのは良くない

Additional info (optional)

以下をローカルで確認しました

• 下記の両方を満たすパターンのとき、HTLに該当ノートが流れてこないこと
  • 自分が別ユーザをフォローしている
  • 自分がフォローしていないチャンネル内で↑のユーザがノート
• 下記の両方を満たすパターンのとき、HTLに該当ノートが流れてくること
  • 自分が別ユーザをフォローしている
  • 自分がフォローしているチャンネル内で↑のユーザがノート
• リグレッションとして フォローしたチャンネルのノートを常にHTLに流したい #11765 で行った動作確認も再実施

Checklist

• Read the contribution guide
• Test working in a local environment　※oauth.tsがタイムアウト
• (If needed) Add story of storybook
• (If needed) Update CHANGELOG.md
• (If possible) Add tests

[codecov]

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (cb1449b) 79.16% compared to head (83c9326) 79.18%.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #12186      +/-   ##
===========================================
+ Coverage    79.16%   79.18%   +0.01%     
===========================================
  Files          941      942       +1     
  Lines       101019   101087      +68     
  Branches      8215     8220       +5     
===========================================
+ Hits         79973    80043      +70     
+ Misses       21046    21044       -2     

Files	Coverage Δ
...backend/src/server/api/endpoints/notes/timeline.ts	79.71% <0.00%> (-1.18%)	⬇️

... and 4 files with indirect coverage changes

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

[github-actions]

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

[syuilo]

👍