instance.actorが消せてしまう

[syuilo]

💡 Summary

instance.actorが消せてしまい、連合ができなくなる

🥰 Expected Behavior

消せない

🤬 Actual Behavior

消せる

📝 Steps to Reproduce

No response

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey:

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

• Yes, I will patch the bug myself and send a pull request

[5ouma]

現状の解決策: データベースを直接編集して、確保されているIDから@instance.actorを削除

このアカウントが何か明記されていないため、不用意な削除を招く可能性があります。(1敗)
役割を明示する必要もあるかもしれないですね。

[samunohito]

relay.actorについても同様のことが言えると思うので、createSystemUserService.createSystemUser()経由で作成したアカウントは

• 削除不能にする（アカウント削除APIでエラーにする）
• アカウントの一覧に表示しない

が必要だと思います。

現状、システムアカウントかどうかのフラグは無いので

• userテーブルにisSystemみたいなのを持つ
• 特殊なシステムアカウント名をハードコードして除く

のどちらかの対応になるかと・・・（前者を推したい）

[tai-cha]

現状Misskeyは.付きのusernameをユーザーは使えないはずなので、それらをisSystemなどに移行することで前者への移行自体はできそうですよね

[syuilo]

現状の解決策: データベースを直接編集して、確保されているIDから@instance.actorを削除

この場合、署名鍵が変わるので今まで連合していた相手と連合できなくなる可能性がありそうですね

[anatawa12]

当面の対処として、instance.actorとrelay.actorをハードコードで禁じていい気がする。

[syuilo]

今あるシステムアカウントって instance.actor と relay.actor だけ？

[syuilo]

さきに #15026 入れた方がコンフリクト少なくて済むかも

[KisaragiEffective]

「.」を含むハンドルをユーザーが作れないのはそうだけど、「『.』を含んだらシステムアカウント」はFed BridgeとかのMisskeyが想定してなかったケースで壊れそうなのでやめたほうが良さそう

[tai-cha]

「.」を含むハンドルをユーザーが作れないのはそうだけど、「『.』を含んだらシステムアカウント」はFed BridgeとかのMisskeyが想定してなかったケースで壊れそうなのでやめたほうが良さそう

マイグレーションの話であればあくまでhost=nullの場合の話のつもりでした

[syuilo]

「.」を含むハンドルをユーザーが作れないのはそうだけど、「『.』を含んだらシステムアカウント」はFed BridgeとかのMisskeyが想定してなかったケースで壊れそうなのでやめたほうが良さそう

Misskeyはこれからも . を許可することはないから大丈夫だわね