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

モデレーターをリモートからブロックすると、相手側のフォローが外れない #7898

Closed
EbiseLutica opened this issue Oct 22, 2021 · 17 comments
Assignees
Labels
🐛Bug Unexpected behavior 🌌Federation The Federation/ActivityPub feature packages/frontend Client side specific issue/PR 🔒Security Security related issue/PR

Comments

@EbiseLutica
Copy link
Contributor

💡 Summary

リモートインスタンスのユーザーをブロックしたとき、そのユーザーがインスタンスのモデレーターであった場合、相手からは自分が相手をブロックしていないことになる。また、自分に対するフォローが外れない。

これにより、本来見られないはずのフォロワー限定投稿も見られてしまう。

🙂 Expected Behavior

ブロックが正しく実行される

☹️ Actual Behavior

ブロックが相手のインスタンスでは正常に行われない

📝 Steps to Reproduce

  1. リモートインスタンスのモデレーターをブロックする

📌 Environment

Misskey 12.92.0

@EbiseLutica EbiseLutica added 🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR 🔒Security Security related issue/PR 🌌Federation The Federation/ActivityPub feature labels Oct 22, 2021
@mei23
Copy link
Contributor

mei23 commented Oct 22, 2021

ブロックに投稿を見れなくする効果を追加する

「投稿が見れなくなるとモデレーションに支障が~」という話が出てくる

とりあえずモデレーターをブロックできないようにする

モデレーターにフォローを解除させることも出来なくなる (ローカルでもリモートでも)

@EbiseLutica
Copy link
Contributor Author

リモートにまで波及するところが不具合だと思っています

@mei23
Copy link
Contributor

mei23 commented Oct 22, 2021

ローカルでもモデレーターにフォローされたらフォロー解除させられないのも微妙な感じが

@syuilo
Copy link
Member

syuilo commented Oct 22, 2021

フォロー解除についてはそもそもブロックすることでしか行えないのが問題だから、フォロー解除させる機能をブロックとは別に実装予定

@mei23
Copy link
Contributor

mei23 commented Oct 22, 2021

フォロー解除させる機能

あったはず。
リモートからフォロー解除を受信すれば解除するけどMisskeyから発信する機能はない。

@EbiseLutica
Copy link
Contributor Author

フォロー解除機能の実装は別issueで。本issueが提起している問題は、1インスタンスのモデレーターの権力がリモートユーザーにまで及ぶことにあるので...。

@syuilo
Copy link
Member

syuilo commented Oct 22, 2021

リモートユーザーがスパムである可能性もあるから、リモートユーザーであっても自インスタンスのモデレーターをブロックできない仕様で良さそうではある

@syuilo
Copy link
Member

syuilo commented Oct 22, 2021

(フォローの解除だけは可能)

@mei23
Copy link
Contributor

mei23 commented Oct 22, 2021

フォロー解除機能の実装は別issueで。本issueが提起している問題は、1インスタンスのモデレーターの権力がリモートユーザーにまで及ぶことにあるので...。

確かに、フォロー解除機能が別にあってそれが正しく動いたとしてもIssueの問題は解決しないわね。

@syuilo
Copy link
Member

syuilo commented Oct 22, 2021

このIssueで問題になっているのって

本来見られないはずのフォロワー限定投稿も見られてしまう。

じゃない?

@mei23
Copy link
Contributor

mei23 commented Oct 22, 2021

フォロー解除させる機能をブロックとは 別に 実装予定

少なくともブロックとは 別に 実装だと問題は解決しない。

@syuilo
Copy link
Member

syuilo commented Oct 22, 2021

別に実装 + リモートからブロックされた時にモデレーターであってもフォロー解除だけは実行する感じかな

@mei23
Copy link
Contributor

mei23 commented Oct 22, 2021

別に実装 + リモートからブロックされた時にモデレーターであってもフォロー解除だけは実行する感じかな

それならギリギリなんとか

@EbiseLutica
Copy link
Contributor Author

リモートユーザーがスパムである可能性

それを加味するなら #7734 のようにブロックはできれどホーム以上の投稿は見れるよ、という仕様を採用し、件のブロック拒否機能を廃止すべきかなと思う

もっと踏み込むなら、管理パネルみたいな通常アクセスしない場所じゃないとブロックされた人の投稿を見られる力を行使できなくしてほしい(工数がかかりそう)

@syuilo
Copy link
Member

syuilo commented Oct 22, 2021

モデレーターをブロックできない仕様取りやめるのが一番楽そう(モデレーションについては別途考えるとして…)

@mei23
Copy link
Contributor

mei23 commented Oct 22, 2021

そもそも「モデレーターはブロック出来ない」自体、

  • 既にブロックされていたら効果がない
  • モデレーターの状態でブロックを避けた後にモデレーターを解任されたらどうする

みたいなのがあるから微妙。

@EbiseLutica
Copy link
Contributor Author

  • モデレーターブロックブロック機能の廃止
  • モデレーターはブロックされている相手ユーザーのパブリック|ホーム投稿を見れる

という仕様でいいんじゃないかな。その上で

  • ブロックされているユーザーの投稿を見る前にワンクッション操作を必要とさせる (より良い)
  • ブロックされているユーザーの投稿を見る専用の画面を管理画面に提供する (めちゃ良い)

って感じでどうかなあ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior 🌌Federation The Federation/ActivityPub feature packages/frontend Client side specific issue/PR 🔒Security Security related issue/PR
Projects
None yet
Development

No branches or pull requests

3 participants