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

削除されたユーザーへ配信したとき、sharedInboxがnullであるとサーバー全体について配送停止される #10499

Closed
m-hayabusa opened this issue Apr 6, 2023 · 2 comments
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR

Comments

@m-hayabusa
Copy link
Contributor

💡 Summary

https://mewl.me で、mstdn.jp / pawoo.net / ... に投稿を配信したとき、一部の削除済みユーザーのInboxから410 Goneが返ってくるため、それらサーバー宛の配送が停止される

🥰 Expected Behavior

アカウント単位では削除されているが、全体では稼働中であるため、停止されない

🤬 Actual Behavior

if (job.data.isSharedInbox && res.statusCode === 410) {
this.federatedInstanceService.fetch(host).then(i => {
this.instancesRepository.update(i.id, {
isSuspended: true,
});
this.federatedInstanceService.updateCachePartial(host, {
isSuspended: true,
});
});
のif文内に追記した console.log('makeSuspended', host, job.data.to);

makeSuspended mstdn.jp https://mstdn.jp/users/<ユーザー名>/inbox

のように出力される (該当ユーザは削除済み)

📝 Steps to Reproduce

不明

📌 Environment

Misskey version: v13.10.3 (当該サーバーはv8.33.xあたりから稼動している)
Your OS: Ubuntu Server 22.04

@m-hayabusa m-hayabusa added the ⚠️bug? This might be a bug label Apr 6, 2023
@m-hayabusa
Copy link
Contributor Author

userテーブル内sharedInboxnullになっている場合、inboxのURLへ配信をするが、その場合でもjob.data.isSharedInboxtrueである ということのようです

@m-hayabusa m-hayabusa changed the title ユーザー単位で410 Goneを返すサーバーが自動で配信停止になる場合がある 削除されたユーザーへ配信したとき、sharedInboxがnullであるとサーバー全体について配送停止される Apr 7, 2023
@CyberRex0
Copy link
Contributor

恐らく、ここの式の===は!=じゃないといけなかったかもしれない

inboxes.set(inbox, following.followerSharedInbox === null);

@syuilo syuilo added 🐛Bug Unexpected behavior packages/backend Server side specific issue/PR and removed ⚠️bug? This might be a bug labels Apr 7, 2023
@syuilo syuilo closed this as completed in 59d0d50 Apr 9, 2023
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

No branches or pull requests

3 participants