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

ユーザーの凍結も削除も連合ではDeleteとして扱われるが、凍結解除した場合フォロー関係がリセットされた状態で復活する #13256

Open
1 task
tamaina opened this issue Feb 11, 2024 · 12 comments
Labels
✨Feature This adds/improves/enhances a feature 🌌Federation The Federation/ActivityPub feature 🔥high priority

Comments

@tamaina
Copy link
Member

tamaina commented Feb 11, 2024

Summary

連合で凍結を識別したい

Purpose

ユーザーが削除されてしまうため、凍結解除した際にフォロー関係を修復できない

Do you want to implement this feature yourself?

  • Yes, I will implement this by myself and send a pull request
@tamaina tamaina added the ✨Feature This adds/improves/enhances a feature label Feb 11, 2024
@kakkokari-gtyih
Copy link
Contributor

強火

@tamaina tamaina changed the title ユーザーを凍結も削除も連合ではDeleteとして扱われるが、凍結の場合フォロー関係を修復できない ユーザーの凍結も削除も連合ではDeleteとして扱われるが、凍結の場合フォロー関係を修復できない Feb 11, 2024
@syuilo syuilo added the 🌌Federation The Federation/ActivityPub feature label Feb 11, 2024
@tamaina tamaina changed the title ユーザーの凍結も削除も連合ではDeleteとして扱われるが、凍結の場合フォロー関係を修復できない ユーザーの凍結も削除も連合ではDeleteとして扱われるが、凍結解除した場合フォロー関係を修復できない Feb 11, 2024
@mei23
Copy link
Contributor

mei23 commented Feb 11, 2024

Mastodonには、Deleteだけど本Deleteじゃない プロパティがあった気がするわね。
そのあたりをSuspendとマップすればいいかもだわ。
でも、ローカルで行ったSuspendをリモートから戻されるわけにはいかないので、リモートSuspendみたいなフラグ必要かもだわね。

@Sayamame-beans
Copy link
Member

現状だとノートも消えるので、ちょっとつらそうですね…

@mei23
Copy link
Contributor

mei23 commented Feb 12, 2024

フォロー関係を修復できない

曖昧な表現
フォロー関係に不整合が出るのではなく、フォロー関係がリセットされた状態で復活する。

@kakkokari-gtyih kakkokari-gtyih changed the title ユーザーの凍結も削除も連合ではDeleteとして扱われるが、凍結解除した場合フォロー関係を修復できない ユーザーの凍結も削除も連合ではDeleteとして扱われるが、凍結解除した場合フォロー関係がリセットされる Feb 12, 2024
@kakkokari-gtyih kakkokari-gtyih changed the title ユーザーの凍結も削除も連合ではDeleteとして扱われるが、凍結解除した場合フォロー関係がリセットされる ユーザーの凍結も削除も連合ではDeleteとして扱われるが、凍結解除した場合フォロー関係がリセットされた状態で復活する Feb 12, 2024
@Sayamame-beans
Copy link
Member

フォロー関係に不整合が出るのではなく、

不整合が出るかと思います

フォローしようとしても多分リモート側でフォロー済扱い受けてるからかフォローできないのですが
https://misskey.niri.la/notes/9pl1csori1

@mei23
Copy link
Contributor

mei23 commented Feb 12, 2024

不整合が出るかと思います

通常MisskeyでDeleteを送られてきて成功する限り、フォロー関係も含めて削除されるので出ないはず。

@anatawa12
Copy link
Member

anatawa12 commented Feb 12, 2024

フォローする側ではフォロー解除されるのですが(確認済)、フォローされる側 = 凍結解除された側で不整合が起きるのだと推測します

@mei23
Copy link
Contributor

mei23 commented Feb 12, 2024

フォローしようとしても多分リモート側でフォロー済扱い受けてるからかフォローできないのですが
https://misskey.niri.la/notes/9pl1csori1

リモートユーザー削除しても (おそらく削除済みレコードを参照している状態の) キャッシュを残してしまっているらしく
そこで返ってきたAccept Followを弾いてしまって不整合になっていると思っているわ。

@anatawa12
Copy link
Member

anatawa12 commented Feb 12, 2024

ブロ解して再フォローでうまく行ったのは運とか時間経過という感じですかね あれうまく行ってなかった

@anatawa12
Copy link
Member

リモートユーザー削除しても (おそらく削除済みレコードを参照している状態の) キャッシュを残してしまっているらしく
そこで返ってきたAccept Followを弾いてしまって不整合になっていると思っているわ。

今確認したらなんかうまく行ってなかったので確かのそうなのかもです

@mei23
Copy link
Contributor

mei23 commented Feb 12, 2024

実際試した感じこうなったわ

Delete => Undo Deleteの段階ではどちらもフォロー関係は0

Delete => Undo Deleteを受け取った方のサーバーからフォローした場合
相手からAccept Followが返って来てもError: No follow request.でエラーにしてしまう (inboxジョブとして何回か再試行される)

受け取った側のMisskeyを再起動して (ここでキャッシュも消える) ジョブキューを再試行すると成功する

@mei23
Copy link
Contributor

mei23 commented Feb 12, 2024

あと本題の方だと
新生ユーザーとして復活するので、フォロー関係だけではなくミュートブロックやモデレーション系もリセットされてしまう。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature 🌌Federation The Federation/ActivityPub feature 🔥high priority
Projects
Status: Triage
Development

No branches or pull requests

6 participants