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

リストタイムラインが殆どのミュートを貫通する #10443

Closed
EbiseLutica opened this issue Mar 31, 2023 · 10 comments · Fixed by #12534
Closed

リストタイムラインが殆どのミュートを貫通する #10443

EbiseLutica opened this issue Mar 31, 2023 · 10 comments · Fixed by #12534
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR

Comments

@EbiseLutica
Copy link
Contributor

💡 Summary

リストタイムラインが以下のミュートを尊重せず、投稿を貫通してしまう不具合がある。HTTP APIとストリーミングの両方で確認済み。

  • ユーザーミュート
    • RenoteやReply経由で貫通する
  • ワードミュート
  • サーバーミュート
  • チャンネル投稿
    • ミュートとは違うが、そもそもフォローしていないチャンネル投稿が引っかかるのはあんまり良くなさそう

🥰 Expected Behavior

🤬 Actual Behavior

📝 Steps to Reproduce

📌 Environment

Misskey version: Misskey 13.10.3
Your OS: Windows 11
Your browser: Chrome

@tamaina
Copy link
Contributor

tamaina commented Mar 31, 2023

user-list-timelineエンドポイントだけブロック・ミュートのクエリが書かれていない(どうしてかは知らないので手出ししてない

@EbiseLutica
Copy link
Contributor Author

joinとサブクエリ地獄になって機能しなくなるから説はありそう

@EbiseLutica
Copy link
Contributor Author

直すためにはおそらく #9325 が必須?

@tamaina
Copy link
Contributor

tamaina commented Mar 31, 2023

必須?

必須というわけではないのでは(ただ単に忘れたと推測してる

@anatawa12
Copy link
Member

anatawa12 commented Aug 8, 2023

issue探し忘れてましたが #11480 で治ってるかもです

@anatawa12
Copy link
Member

streamの方のワードミュートは確認してないので治ってないかもですがuser muteはそれっぽい実装あります

// 流れてきたNoteがミュートしているユーザーが関わるものだったら無視する
if (isUserRelated(note, this.userIdsWhoMeMuting)) return;
// 流れてきたNoteがブロックされているユーザーが関わるものだったら無視する
if (isUserRelated(note, this.userIdsWhoBlockingMe)) return;

nirila鯖でのユーザからのレポートによりユーザミュートはstreamでは動いてるっぽいという報告があります

@samunohito
Copy link
Member

(ミュート貫通系はちょっと気になってる)

@syuilo
2点ほど確認させてほしいです(良さげなら👍だけでも)

①あえて現状にしているわけではない…で相違ございませんか?

②①がYesの場合、手を入れる方針としては以下で問題ございませんか?

  • ユーザーミュート
    →考慮する。Renote先やReply先にミュートユーザがいる場合はリストに流さない
  • ワードミュート
    →考慮しない。フロント側に移ったので
  • サーバーミュート
    →考慮する。ミュートしたサーバであればリストに流さない
  • チャンネル投稿
    →考慮する。自分がフォローしていないチャンネルに対しての投稿はリストに流さない

@anatawa12
Copy link
Member

チャンネルについてはチャンネルフォローしたくないけどみてたい、または逆があるから選択可能にしてほしいです

@syuilo syuilo added 🐛Bug Unexpected behavior packages/backend Server side specific issue/PR and removed ⚠️bug? This might be a bug labels Nov 29, 2023
@syuilo
Copy link
Member

syuilo commented Nov 29, 2023

①あえて現状にしているわけではない…で相違ございませんか?

yes

②①がYesの場合、手を入れる方針としては以下で問題ございませんか?

yes

@syuilo
Copy link
Member

syuilo commented Nov 29, 2023

チャンネルについてはチャンネルフォローしたくないけどみてたい、または逆があるから選択可能にしてほしいです

これは他のTLでも言える話だから別Issueかな

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants