enhance: account migration #10592

nmkj-io · 2023-04-12T01:04:32Z

What

#10507 で導入したアカウント引っ越し機能を強化する。

Resolves: #5475
Resolves: #10515

Why

#10507 の通り

Additional info (optional)

引っ越し発生時のフォロー・フォロー解除をジョブキューにする feat: queueing bulk follow/unfollow and block/unblock #10544
同じサーバー内のアカウントに引っ越しできるようにする
引っ越し発生時に旧アカウントに対するブロック・ミュートを新アカウントに向けて再登録
- 引っ越し元サーバーおよびMove受信サーバーはシンプルにそのまま再登録
- フォロワーがサーバーにおらずMoveを受信しなかった場合でも既知のリモートアカウントがfetchされた場合に新アカウントを検知できるか？
  - apPersonServiceのupdatePersonでmovedToがデータベースと比較して新しく設定されていた場合ブロック・ミュートを再登録
  - 毎回ブロック・ミュートをチェックすることになり負荷が増えないか？
    - 毎回は起こらない（movedToが新しく設定されていた場合のみ）
  - 新アカウントが同一サーバーでも旧リモートアカウントをfetchする必要あり。（alsoKnownAsだけでは信用できない）
  - 他の実装が正しい（引越し時にちゃんとalsoKnownAsをチェックしている）ならmovedToは信用できる。
旧アカウントが登録されていたリスト ~~とアンテナ~~ へ新アカウントを追加
- 新アカウントが検知できた場合のみ可能
- 負荷が大きいか？
  - アンテナのユーザーリスト更新は大変だがリストはそうでもない
引っ越し元のアカウントはフォローを解除する
- 次のフォローリクエスト自動許可との整合性が取れないとか、フォローエクスポートを忘れていた場合を鑑み、Bullの機能を使い、24時間後にフォロー解除を始める（テストでは10秒後）
鍵アカウントで旧アカウントをフォロー許可していた場合に新アカウントからのフォローリクエストを自動許可する
- フォローリクエスト受信時にalsoKnownAsの旧アカウントを再fetchしてmovedToが正しいことを確認
- 新アカウントが引っ越し実行前にフォローリクエストしてきた場合は不可能（旧アカウントと関連が無いため）
  - そもそもalsoKnownAsは他人のアカウントでも設定可能なため、それだけで信用してはならない
  - 引っ越し完了後にもう一度リクエストされれば確実に許可できる
旧アカウントの使用を制限する
- 禁止するもの
  - ノート作成
  - リアクション
  - フォロー
  - リスト作成・編集
  - アンテナ作成・編集
  - チャンネルの作成
  - クリップの作成・編集
  - ギャラリーの作成・編集
  - ページの作成・編集
  - ドライブへのアップロード
  - フォロー、リストなどのインポート
  - 実績解除
  - ピン留め
  - など
- 旧アカウントに対するフォローは拒否しない
  - ~~フォロワー限定のノートなどが見れなくなるのでフォローは引き続き受け付ける？~~
    - ~~旧アカウントをunfollowしない方がよいか？~~
  - 旧アカウントをunfollowしない
  - 旧アカウントのフォロワーカウントを増やさない
    - 引っ越し後は旧アカウントのフォロー・フォロワーカウントを0に固定する
  - movedToがあるアカウントはフォローとして数えない
    - 引っ越し時に-1してしまい、follow・ ~~unfollow~~ でmovedToがある場合はカウントを動かさない。
    - unfollow のときは引っ越し実装前にすでに旧アカウントをフォローしているアカウントがあり得るため movedTo が無いフォロー・フォロワーの数でアップデートする。
- 旧アカウントが作成済みのリスト・アンテナは残す？
  - 将来的にインポート・エクスポートが実装されたときのために残す
- ドライブの閲覧・ダウンロードはOK
- ~~設定画面以外を見えないようにする？~~
リレーションジョブキューの実行制限設定を追加
- ブロック, フォロー, ミュート, リストのインポートファイルの制限を32MiBに
~~引っ越し後に新アカウントのフォロー通知を抑制~~ 引っ越し先サーバーにとって通常のフォローとアカウント引っ越しによるフォローの見分けがつかないため難しい
- ~~一定時間抑制する？~~ いつ解除するのかわからない。引っ越し直後にも通常のフォローを受信する可能性あり。

Checklist

Read the contribution guide
Test working in a local environment
(If needed) Update CHANGELOG.md
(If possible) Add tests

codecov · 2023-04-12T01:08:37Z

Codecov Report

Merging #10592 (037727a) into develop (149ddeb) will increase coverage by 2.24%.
The diff coverage is 60.52%.

@@             Coverage Diff             @@
##           develop   #10592      +/-   ##
===========================================
+ Coverage    76.08%   78.33%   +2.24%     
===========================================
  Files          899      166     -733     
  Lines        88855    20660   -68195     
  Branches      6280      374    -5906     
===========================================
- Hits         67603    16183   -51420     
+ Misses       21252     4477   -16775

Impacted Files	Coverage Δ
packages/frontend/src/scripts/achievements.ts	`96.28% <0.00%> (-0.20%)`	⬇️
packages/frontend/src/scripts/please-login.ts	`28.57% <0.00%> (ø)`
packages/frontend/src/scripts/show-moved-dialog.ts	`31.25% <31.25%> (ø)`
packages/frontend/src/os.ts	`17.04% <33.33%> (+0.07%)`	⬆️
...ackages/frontend/src/components/MkAccountMoved.vue	`100.00% <100.00%> (ø)`
packages/frontend/src/components/MkInfo.vue	`100.00% <100.00%> (ø)`
packages/frontend/src/components/MkNote.vue	`100.00% <100.00%> (ø)`
packages/frontend/src/pages/user/home.vue	`92.69% <100.00%> (ø)`

... and 733 files with indirect coverage changes

tamaina · 2023-04-12T02:07:11Z

引っ越し発生時に旧アカウントに対するブロック・ミュートを新アカウントに向けて再登録

めちゃくちゃすごい

naskya · 2023-04-13T12:57:53Z

ここまで徹底的にお引っ越しを実装するのであれば、旧アカウントが追加されていたリストに新アカウントを追加するというのもいかがでしょうか
人数が上限まで追加されているリストでは失敗してしまいそうですが（あるいはその場合には旧アカウントは使用されないという前提でリストから旧アカウントを削除する？これは破壊的な操作なので慎重に議論されるべきですが、実用的ではありそうです）

nmkj-io · 2023-04-13T13:15:57Z

旧アカウントが追加されていたリストに新アカウントを追加する

新アカウントが検知できた場合は可能だと思いますが、リスト・アンテナから旧アカウントでfindBy→新アカウントと入れ替えてupdateをすると負荷が大きいかもしれません。

操作の結果を待たなくていいので高負荷の場合はジョブにするという手もあるかと思いますが、負荷についての助言があるとありがたいです。

naskya · 2023-04-13T13:43:30Z

リストも引っ越しに対応してくれると便利だろうと思ってコメントしましたが、確かに更新すべきリストが明らかでないので小規模なサーバーでない限りその機能は現実的ではなさそうですね……。

tamaina · 2023-04-13T13:54:48Z

全リスト・アンテナから旧アカウントを検索→新アカウントと入れ替えをすると負荷が大きいかも

リストはuserListJoiningをいじればいいのでそこまで大変ではない気がする

アンテナのusersを更新するのは超大変

nmkj-io · 2023-04-13T17:57:53Z

引っ越し後の旧アカウントに対するフォローなのですが、フォロワー限定ノートなどを後から見たいという可能性を考えると

旧アカウントへのフォローを拒否しないでおくか
引っ越し実行・Move受信時に旧アカウントをunfollowしないでおくか

について、意見をもらえるとありがたいです。
（個人的にはフォロー拒否せずunfollowも要らないのではと思ってます）

tamaina · 2023-04-13T18:00:16Z

旧アカウントへのフォローを拒否しないでおくか

拒否しない方が良さそう（フォローの承認制や許可拒否もいじれるようにした方がいい）

引っ越し実行・Move受信時に旧アカウントをunfollowしないでおくか

unfollowしないけど、フォローカウントを増やさないみたいな処置が欲しいかもしれない

tamaina · 2023-04-13T18:01:59Z

引越し元のアカウント(movedToのあるアカウント)をフォローしてもフォローカウントは増やさないでおくという処置が望ましいかなと個人的に思った

This reverts commit 2321214.

…into pr/nmkj-io/10592

tamaina · 2023-04-27T03:50:06Z

マージするか

…into pr/nmkj-io/10592

tamaina · 2023-04-29T15:10:07Z

実験的って書いてマージした()

saschanaz · 2023-04-29T15:31:25Z

実験的って書いて

良さそう

locales/ja-JP.yml

tamaina · 2023-05-02T09:18:32Z

どうしてテストが落ちているの〜

#10738

* copy block and mute then create follow and unfollow jobs * copy block and mute and update lists when detecting an account has moved * no need to care promise orders * refactor updating actor and target * automatically accept if a locked account had accepted an old account * fix exception format * prevent the old account from calling some endpoints * do not unfollow when moving * adjust following and follower counts * check movedToUri when receiving a follow request * skip if no need to adjust * Revert "disable account migration" This reverts commit 2321214. * fix translation specifier * fix checking alsoKnownAs and uri * fix updating account * fix refollowing locked account * decrease followersCount if followed by the old account * adjust following and followers counts when unfollowing * fix copying mutings * prohibit moved account from moving again * fix move service * allow app creation after moving * fix lint * remove unnecessary field * fix cache update * add e2e test * add e2e test of accepting the new account automatically * force follow if any error happens * remove unnecessary joins * use Array.map instead of for const of * ユーザーリストの移行は追加のみを行う * nanka iroiro * fix misskey-js? * ✌️ * 移行を行ったアカウントからのフォローリクエストの自動許可を調整 * newUriを外に出す * newUriを外に出す2 * clean up * fix newUri * prevent moving if the destination account has already moved * set alsoKnownAs via /i/update * fix database initialization * add return type * prohibit updating alsoKnownAs after moving * skip to add to alsoKnownAs if toUrl is known * skip adding to the list if it already has * use Acct.parse instead * rename error code * 🎨 * 制限を5から10に緩和 * movedTo(Uri), alsoKnownAsはユーザーidを返すように * test api res * fix * 元アカウントはミュートし続ける * 🎨 * unfollow * fix * getUserUriをUserEntityServiceに * ? * job! * 🎨 * instance => server * accountMovedShort, forbiddenBecauseYouAreMigrated * accountMovedShort * fix test * import, pin禁止 * 実績を凍結する * clean up * ✌️ * change message * ブロック, フォロー, ミュート, リストのインポートファイルの制限を32MiBに * Revert "ブロック, フォロー, ミュート, リストのインポートファイルの制限を32MiBに" This reverts commit 3bd7be3. * validateAlsoKnownAs * 移行後2時間以内はインポート可能なファイルサイズを拡大 * clean up * どうせactorをupdatePersonで更新するならupdatePersonしか移行処理を発行しないことにする * handle error? * リモートからの移行処理の条件を是正 * log, port * fix * fix * enhance(dev): non-production環境でhttpサーバー間でもユーザー、ノートの連合が可能なように * refactor (use checkHttps) * MISSKEY_WEBFINGER_USE_HTTP * Environment Variable readme * NEVER USE IN PRODUCTION * fix punyHost * fix indent * fix * experimental --------- Co-authored-by: tamaina <[email protected]> Co-authored-by: syuilo <[email protected]>

tamaina · 2023-07-06T08:14:13Z

isRootをマイグレーションできなくする意味ってある？

tamaina · 2023-07-06T08:15:55Z

（APIがprohibitMovedでブラックリスト形式になっているため管理が滞るようなことはなさそう）

* cleanup(`ApImageService.ts`) * refactor(`ApImageService.ts`) * cleanup(`check-https.ts`) * cleanup(`ApMentionService.ts`) * refactor(`ApMentionService.ts`) * cleanup(`ApNoteService.ts`): unneeded `eslint-disable-next-line` * cleanup(`ApNoteService.ts`) * WIP(`ApImageService.ts`): `image.url`を`getApHrefNullable()`に通すかどうか悩んでいる * refactor(`ApNoteService.ts`): function return type * cleanup(`ApNoteService.ts`): deadcode * refactor(`ApNoteService.ts`): `eslint-disable-next-line` * refactor(`ApNoteService.ts`): non-null assertion これまでは`getApId()`の方でエラーがスローされていた。 * cleanup(`ApNoteService.ts`): unneeded await * refactor(`ApNoteService.ts`): note.attachment - `toArray()`を使うように - よくわからない条件式を整理 - `as`をなくすために`promiseLimit()`でジェネリクスを使うように * cleanup(`ApNoteService.ts`) * refactor(`ApNoteService.ts`): よりよい型定義 `res`が`null`でないことは確認されているようだったので`null`とのunionはなくした * refactor(`ApNoteService.ts`): 不要な条件を削除 * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`): 重要でない`as`を削除 * refactor(`ApNoteService.ts`): `eslint-disable-next-line` * cleanup(`ApNoteService.ts`): deadcode * cleanup(`ApNoteService.ts`): unneeded non-null assertion * refactor(`ApNoteService.ts`): 不要な条件を削除 * WIP(`ApNoteService.ts`): `as`をなくすエラーメッセージを考える * cleanup(`ApNoteService.ts`): 不要な`as`を削除 * cleanup(`ApPersonService.ts`): `no-unused-vars` * cleanup(`ApPersonService.ts`): deadcode * refactor(`ApPersonService.ts`): function return type * cleanup(`ApPersonService.ts`): deadcode * cleanup(`ApPersonService.ts`): deadcode * WIP(`ApPersonService.ts`): `as`を調整 `null`でないか確認する処理が続いていたので型アサーションは`null`とのunionにした。より本質的な改善の余地があるように感じるのでひとまずWIPとしてコミット。 * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * WIP(`ApPersonService.ts`): `as any`をなくしたエラーをスローするようにせざるを得なかったのでエラーメッセージを考える必要がある。 * WIP(`ApNoteService.ts`): non-null assertion non-nullアサーションを減らすために事前に存在確認をするようにした。エラーをスローするようにしたのでメッセージを考えなければならない。 * refactor(`ApNoteService.ts`): non-null assertion -> optional chaining * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * refactor(`ApPersonService.ts`): function return type * refactor(`ApPersonService.ts`): type guardによるnon-null assertionの削除 * WIP(`ApPersonService.ts`): `analyzeAttachments` - Field型を事前に定義しておくように - `attachments`が`IObject`だった場合、返り値が`{ fields: [] }`になるようだが構わないのか？ - `toArray()`を通すべきでは？ * Revert "WIP(`ApImageService.ts`): `image.url`を`getApHrefNullable()`に通すかどうか悩んでいる" This reverts commit aeefb84. * cleanup(`ApImageService.ts`): `import` * refactor(`ApImageService.ts`): 冗長だった部分を短く * cleanup(`ApMentionService.ts`): `import` * refactor(`ApImageService.ts`): `JSON.stringify()`でのindentationを追加 * cleanup(`ApNoteService.ts`): `import` * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`): `any`に対するnon-null assertion * refactor(`ApNoteService.ts`): 添付ファイル * cleanup(`ApPersonService.ts`): `import` * refactor(`ApPersonService.ts`): より実情に即した`as`に * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 冗長だった部分を修正 * cleanup(`ApPersonService.ts`): deadcode * cleanup(`ApPersonService.ts`) * cleanup(`ApQuestionService.ts`): `import` * refactor(`ApQuestionService.ts`): `eslint-disable-next-line` * refactor(`ApQuestionService.ts`): `eslint-disable-next-line` * cleanup(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`): non-null assertionを消した * cleanup(`ApQuestionService.ts`) * WIP(`ApQuestionService.ts`): non-null assertionを消すエラーメッセージを考える必要がある。 * refactor(`ApQuestionService.ts`): `any`を消す * refactor(`ApQuestionService.ts`): function return type * WIP(`ApPersonService.ts`): 可読性の低い三項演算子を削除しつつnon-null assertionを回避エラーメッセージを考える必要がある。 * cleanup(`ApPersonService.ts`): 不必要な三項演算子を削除 * cleanup(`ApPersonService.ts`): 不要な`as` * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 可読性の低い三項演算子を削除元の実装が悪いと判断し`null`かどうかの確認をより厳密に行うようにした。 * cleanup(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 返り値を`void`に統一この返り値を参照しているコードは見当たらなかった。また、普通に意味がない値であるように見受けられた。 * fixup! refactor(`ApPersonService.ts`): 返り値を`void`に統一 * refactor(`ApNoteService.ts`) * refactor(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 返り値の`void`統一と条件式の調整この返り値を参照しているコードは見当たらなかった。また、普通に意味がない値であるように見受けられた。 * cleanup(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`) * refactor(`tag.ts`): function return type * fixup! enhance: account migration (#10592) * fixup! WIP(`ApPersonService.ts`): 可読性の低い三項演算子を削除しつつnon-null assertionを回避 * fixup! cleanup(`ApPersonService.ts`): 不要な`as` * refactor: エラーメッセージを見繕った * Revert "cleanup(`ApImageService.ts`): `import`" This reverts commit 1454d04. * Revert "cleanup(`ApMentionService.ts`): `import`" This reverts commit 244f672. * Revert "cleanup(`ApNoteService.ts`): `import`" This reverts commit d8f0d76. * Revert "cleanup(`ApPersonService.ts`): `import`" This reverts commit 5190ef9. # Conflicts: # packages/backend/src/core/activitypub/models/ApPersonService.ts * Revert "cleanup(`ApQuestionService.ts`): `import`" This reverts commit 778585e. * processRemoteMoveはそのままにしてほしい * Revert "fixup! refactor(`ApPersonService.ts`): 返り値を`void`に統一" This reverts commit 083cd67. * Revert "refactor(`ApPersonService.ts`): 返り値を`void`に統一" This reverts commit bfa0fcd. --------- Co-authored-by: tamaina <[email protected]>

* cleanup(`ApImageService.ts`) * refactor(`ApImageService.ts`) * cleanup(`check-https.ts`) * cleanup(`ApMentionService.ts`) * refactor(`ApMentionService.ts`) * cleanup(`ApNoteService.ts`): unneeded `eslint-disable-next-line` * cleanup(`ApNoteService.ts`) * WIP(`ApImageService.ts`): `image.url`を`getApHrefNullable()`に通すかどうか悩んでいる * refactor(`ApNoteService.ts`): function return type * cleanup(`ApNoteService.ts`): deadcode * refactor(`ApNoteService.ts`): `eslint-disable-next-line` * refactor(`ApNoteService.ts`): non-null assertion これまでは`getApId()`の方でエラーがスローされていた。 * cleanup(`ApNoteService.ts`): unneeded await * refactor(`ApNoteService.ts`): note.attachment - `toArray()`を使うように - よくわからない条件式を整理 - `as`をなくすために`promiseLimit()`でジェネリクスを使うように * cleanup(`ApNoteService.ts`) * refactor(`ApNoteService.ts`): よりよい型定義 `res`が`null`でないことは確認されているようだったので`null`とのunionはなくした * refactor(`ApNoteService.ts`): 不要な条件を削除 * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`): 重要でない`as`を削除 * refactor(`ApNoteService.ts`): `eslint-disable-next-line` * cleanup(`ApNoteService.ts`): deadcode * cleanup(`ApNoteService.ts`): unneeded non-null assertion * refactor(`ApNoteService.ts`): 不要な条件を削除 * WIP(`ApNoteService.ts`): `as`をなくすエラーメッセージを考える * cleanup(`ApNoteService.ts`): 不要な`as`を削除 * cleanup(`ApPersonService.ts`): `no-unused-vars` * cleanup(`ApPersonService.ts`): deadcode * refactor(`ApPersonService.ts`): function return type * cleanup(`ApPersonService.ts`): deadcode * cleanup(`ApPersonService.ts`): deadcode * WIP(`ApPersonService.ts`): `as`を調整 `null`でないか確認する処理が続いていたので型アサーションは`null`とのunionにした。より本質的な改善の余地があるように感じるのでひとまずWIPとしてコミット。 * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * WIP(`ApPersonService.ts`): `as any`をなくしたエラーをスローするようにせざるを得なかったのでエラーメッセージを考える必要がある。 * WIP(`ApNoteService.ts`): non-null assertion non-nullアサーションを減らすために事前に存在確認をするようにした。エラーをスローするようにしたのでメッセージを考えなければならない。 * refactor(`ApNoteService.ts`): non-null assertion -> optional chaining * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * refactor(`ApPersonService.ts`): function return type * refactor(`ApPersonService.ts`): type guardによるnon-null assertionの削除 * WIP(`ApPersonService.ts`): `analyzeAttachments` - Field型を事前に定義しておくように - `attachments`が`IObject`だった場合、返り値が`{ fields: [] }`になるようだが構わないのか？ - `toArray()`を通すべきでは？ * Revert "WIP(`ApImageService.ts`): `image.url`を`getApHrefNullable()`に通すかどうか悩んでいる" This reverts commit aeefb84. * cleanup(`ApImageService.ts`): `import` * refactor(`ApImageService.ts`): 冗長だった部分を短く * cleanup(`ApMentionService.ts`): `import` * refactor(`ApImageService.ts`): `JSON.stringify()`でのindentationを追加 * cleanup(`ApNoteService.ts`): `import` * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`): `any`に対するnon-null assertion * refactor(`ApNoteService.ts`): 添付ファイル * cleanup(`ApPersonService.ts`): `import` * refactor(`ApPersonService.ts`): より実情に即した`as`に * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 冗長だった部分を修正 * cleanup(`ApPersonService.ts`): deadcode * cleanup(`ApPersonService.ts`) * cleanup(`ApQuestionService.ts`): `import` * refactor(`ApQuestionService.ts`): `eslint-disable-next-line` * refactor(`ApQuestionService.ts`): `eslint-disable-next-line` * cleanup(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`): non-null assertionを消した * cleanup(`ApQuestionService.ts`) * WIP(`ApQuestionService.ts`): non-null assertionを消すエラーメッセージを考える必要がある。 * refactor(`ApQuestionService.ts`): `any`を消す * refactor(`ApQuestionService.ts`): function return type * WIP(`ApPersonService.ts`): 可読性の低い三項演算子を削除しつつnon-null assertionを回避エラーメッセージを考える必要がある。 * cleanup(`ApPersonService.ts`): 不必要な三項演算子を削除 * cleanup(`ApPersonService.ts`): 不要な`as` * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 可読性の低い三項演算子を削除元の実装が悪いと判断し`null`かどうかの確認をより厳密に行うようにした。 * cleanup(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 返り値を`void`に統一この返り値を参照しているコードは見当たらなかった。また、普通に意味がない値であるように見受けられた。 * fixup! refactor(`ApPersonService.ts`): 返り値を`void`に統一 * refactor(`ApNoteService.ts`) * refactor(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 返り値の`void`統一と条件式の調整この返り値を参照しているコードは見当たらなかった。また、普通に意味がない値であるように見受けられた。 * cleanup(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`) * refactor(`tag.ts`): function return type * fixup! enhance: account migration (misskey-dev#10592) * fixup! WIP(`ApPersonService.ts`): 可読性の低い三項演算子を削除しつつnon-null assertionを回避 * fixup! cleanup(`ApPersonService.ts`): 不要な`as` * refactor: エラーメッセージを見繕った * Revert "cleanup(`ApImageService.ts`): `import`" This reverts commit 1454d04. * Revert "cleanup(`ApMentionService.ts`): `import`" This reverts commit 244f672. * Revert "cleanup(`ApNoteService.ts`): `import`" This reverts commit d8f0d76. * Revert "cleanup(`ApPersonService.ts`): `import`" This reverts commit 5190ef9. * Revert "cleanup(`ApQuestionService.ts`): `import`" This reverts commit 778585e. * processRemoteMoveはそのままにしてほしい * Revert "fixup! refactor(`ApPersonService.ts`): 返り値を`void`に統一" This reverts commit 083cd67. * Revert "refactor(`ApPersonService.ts`): 返り値を`void`に統一" This reverts commit bfa0fcd. --------- Co-authored-by: tamaina <[email protected]>

* cleanup(`ApImageService.ts`) * refactor(`ApImageService.ts`) * cleanup(`check-https.ts`) * cleanup(`ApMentionService.ts`) * refactor(`ApMentionService.ts`) * cleanup(`ApNoteService.ts`): unneeded `eslint-disable-next-line` * cleanup(`ApNoteService.ts`) * WIP(`ApImageService.ts`): `image.url`を`getApHrefNullable()`に通すかどうか悩んでいる * refactor(`ApNoteService.ts`): function return type * cleanup(`ApNoteService.ts`): deadcode * refactor(`ApNoteService.ts`): `eslint-disable-next-line` * refactor(`ApNoteService.ts`): non-null assertion これまでは`getApId()`の方でエラーがスローされていた。 * cleanup(`ApNoteService.ts`): unneeded await * refactor(`ApNoteService.ts`): note.attachment - `toArray()`を使うように - よくわからない条件式を整理 - `as`をなくすために`promiseLimit()`でジェネリクスを使うように * cleanup(`ApNoteService.ts`) * refactor(`ApNoteService.ts`): よりよい型定義 `res`が`null`でないことは確認されているようだったので`null`とのunionはなくした * refactor(`ApNoteService.ts`): 不要な条件を削除 * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`): 重要でない`as`を削除 * refactor(`ApNoteService.ts`): `eslint-disable-next-line` * cleanup(`ApNoteService.ts`): deadcode * cleanup(`ApNoteService.ts`): unneeded non-null assertion * refactor(`ApNoteService.ts`): 不要な条件を削除 * WIP(`ApNoteService.ts`): `as`をなくすエラーメッセージを考える * cleanup(`ApNoteService.ts`): 不要な`as`を削除 * cleanup(`ApPersonService.ts`): `no-unused-vars` * cleanup(`ApPersonService.ts`): deadcode * refactor(`ApPersonService.ts`): function return type * cleanup(`ApPersonService.ts`): deadcode * cleanup(`ApPersonService.ts`): deadcode * WIP(`ApPersonService.ts`): `as`を調整 `null`でないか確認する処理が続いていたので型アサーションは`null`とのunionにした。より本質的な改善の余地があるように感じるのでひとまずWIPとしてコミット。 * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * WIP(`ApPersonService.ts`): `as any`をなくしたエラーをスローするようにせざるを得なかったのでエラーメッセージを考える必要がある。 * WIP(`ApNoteService.ts`): non-null assertion non-nullアサーションを減らすために事前に存在確認をするようにした。エラーをスローするようにしたのでメッセージを考えなければならない。 * refactor(`ApNoteService.ts`): non-null assertion -> optional chaining * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * refactor(`ApPersonService.ts`): `eslint-disable-next-line` * refactor(`ApPersonService.ts`): function return type * refactor(`ApPersonService.ts`): type guardによるnon-null assertionの削除 * WIP(`ApPersonService.ts`): `analyzeAttachments` - Field型を事前に定義しておくように - `attachments`が`IObject`だった場合、返り値が`{ fields: [] }`になるようだが構わないのか？ - `toArray()`を通すべきでは？ * Revert "WIP(`ApImageService.ts`): `image.url`を`getApHrefNullable()`に通すかどうか悩んでいる" This reverts commit aeefb84. * cleanup(`ApImageService.ts`): `import` * refactor(`ApImageService.ts`): 冗長だった部分を短く * cleanup(`ApMentionService.ts`): `import` * refactor(`ApImageService.ts`): `JSON.stringify()`でのindentationを追加 * cleanup(`ApNoteService.ts`): `import` * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`) * cleanup(`ApNoteService.ts`): `any`に対するnon-null assertion * refactor(`ApNoteService.ts`): 添付ファイル * cleanup(`ApPersonService.ts`): `import` * refactor(`ApPersonService.ts`): より実情に即した`as`に * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 冗長だった部分を修正 * cleanup(`ApPersonService.ts`): deadcode * cleanup(`ApPersonService.ts`) * cleanup(`ApQuestionService.ts`): `import` * refactor(`ApQuestionService.ts`): `eslint-disable-next-line` * refactor(`ApQuestionService.ts`): `eslint-disable-next-line` * cleanup(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`): non-null assertionを消した * cleanup(`ApQuestionService.ts`) * WIP(`ApQuestionService.ts`): non-null assertionを消すエラーメッセージを考える必要がある。 * refactor(`ApQuestionService.ts`): `any`を消す * refactor(`ApQuestionService.ts`): function return type * WIP(`ApPersonService.ts`): 可読性の低い三項演算子を削除しつつnon-null assertionを回避エラーメッセージを考える必要がある。 * cleanup(`ApPersonService.ts`): 不必要な三項演算子を削除 * cleanup(`ApPersonService.ts`): 不要な`as` * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 可読性の低い三項演算子を削除元の実装が悪いと判断し`null`かどうかの確認をより厳密に行うようにした。 * cleanup(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 返り値を`void`に統一この返り値を参照しているコードは見当たらなかった。また、普通に意味がない値であるように見受けられた。 * fixup! refactor(`ApPersonService.ts`): 返り値を`void`に統一 * refactor(`ApNoteService.ts`) * refactor(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * cleanup(`ApPersonService.ts`) * refactor(`ApPersonService.ts`): 返り値の`void`統一と条件式の調整この返り値を参照しているコードは見当たらなかった。また、普通に意味がない値であるように見受けられた。 * cleanup(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`) * refactor(`ApQuestionService.ts`) * refactor(`tag.ts`): function return type * fixup! enhance: account migration (misskey-dev#10592) * fixup! WIP(`ApPersonService.ts`): 可読性の低い三項演算子を削除しつつnon-null assertionを回避 * fixup! cleanup(`ApPersonService.ts`): 不要な`as` * refactor: エラーメッセージを見繕った * Revert "cleanup(`ApImageService.ts`): `import`" This reverts commit 1454d04. * Revert "cleanup(`ApMentionService.ts`): `import`" This reverts commit 244f672. * Revert "cleanup(`ApNoteService.ts`): `import`" This reverts commit d8f0d76. * Revert "cleanup(`ApPersonService.ts`): `import`" This reverts commit 5190ef9. # Conflicts: # packages/backend/src/core/activitypub/models/ApPersonService.ts * Revert "cleanup(`ApQuestionService.ts`): `import`" This reverts commit 778585e. * processRemoteMoveはそのままにしてほしい * Revert "fixup! refactor(`ApPersonService.ts`): 返り値を`void`に統一" This reverts commit 083cd67. * Revert "refactor(`ApPersonService.ts`): 返り値を`void`に統一" This reverts commit bfa0fcd. --------- Co-authored-by: tamaina <[email protected]>

tamaina · 2024-02-19T18:43:22Z

isRootをマイグレーションできなくする意味ってある？

なさそうだけど、isRootならapi.prohibitMovedをバイパスできるといい気がした

github-actions bot added the packages/backend Server side specific issue/PR label Apr 12, 2023

Namekuji added 20 commits April 15, 2023 07:28

copy block and mute then create follow and unfollow jobs

8c031e9

copy block and mute and update lists when detecting an account has moved

91fcad0

no need to care promise orders

0474efc

refactor updating actor and target

75c2c10

automatically accept if a locked account had accepted an old account

71a2fbd

fix exception format

91a59da

prevent the old account from calling some endpoints

5e845f1

do not unfollow when moving

75d02a5

adjust following and follower counts

fc327f0

check movedToUri when receiving a follow request

942d5b6

skip if no need to adjust

aaf098f

Revert "disable account migration"

69a1dad

This reverts commit 2321214.

fix translation specifier

33851fb

fix checking alsoKnownAs and uri

0e088cb

fix updating account

17c80e8

fix refollowing locked account

2f5d9b8

decrease followersCount if followed by the old account

83d70f3

adjust following and followers counts when unfollowing

6a2752e

fix copying mutings

71e9b22

prohibit moved account from moving again

21677aa

tamaina added 6 commits April 26, 2023 11:25

Merge branch 'lhap' into pr/nmkj-io/10592

8a4ba00

Merge branch 'develop' into enhance-migration

d6b2797

NEVER USE IN PRODUCTION

4ee95e3

fix punyHost

a75350d

Merge branch 'lhap' into pr/nmkj-io/10592

c65f0d6

Merge branch 'enhance-migration' of https://github.com/nmkj-io/misskey …

f29239b

…into pr/nmkj-io/10592

tamaina added 3 commits April 27, 2023 03:53

fix indent

a44a73f

fix

abeffd7

Merge branch 'develop' into enhance-migration

058ee55

tamaina mentioned this pull request Apr 27, 2023

fix(backend): alsoKnownAsの誤った定義を修正 #10725

Merged

5 tasks

tamaina added 4 commits April 29, 2023 22:57

Merge branch 'develop' into enhance-migration

2408c04

Merge branch 'develop' into enhance-migration

037727a

experimental

74014f9

Merge branch 'enhance-migration' of https://github.com/nmkj-io/misskey …

10edaba

…into pr/nmkj-io/10592

tamaina merged commit d28866f into misskey-dev:develop Apr 29, 2023

syuilo reviewed Apr 29, 2023

View reviewed changes

locales/ja-JP.yml Show resolved Hide resolved

tassoman mentioned this pull request Nov 8, 2023

Fediverse/ActivityPub account migration #10426

Closed

tamaina mentioned this pull request Apr 24, 2024

rootアカウントもアカウントの引っ越しができるようにしてほしい #13740

Open

1 task

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

enhance: account migration #10592

enhance: account migration #10592

nmkj-io commented Apr 12, 2023 •

edited by tamaina

Loading

codecov bot commented Apr 12, 2023 •

edited

Loading

tamaina commented Apr 12, 2023

naskya commented Apr 13, 2023 •

edited

Loading

nmkj-io commented Apr 13, 2023 •

edited

Loading

naskya commented Apr 13, 2023

tamaina commented Apr 13, 2023

nmkj-io commented Apr 13, 2023 •

edited

Loading

tamaina commented Apr 13, 2023

tamaina commented Apr 13, 2023 •

edited

Loading

tamaina commented Apr 27, 2023

tamaina commented Apr 29, 2023

saschanaz commented Apr 29, 2023

tamaina commented May 2, 2023

tamaina commented Jul 6, 2023

tamaina commented Jul 6, 2023 •

edited

Loading

tamaina commented Feb 19, 2024

enhance: account migration #10592

enhance: account migration #10592

Conversation

nmkj-io commented Apr 12, 2023 • edited by tamaina Loading

What

Why

Additional info (optional)

Checklist

codecov bot commented Apr 12, 2023 • edited Loading

Codecov Report

tamaina commented Apr 12, 2023

naskya commented Apr 13, 2023 • edited Loading

nmkj-io commented Apr 13, 2023 • edited Loading

naskya commented Apr 13, 2023

tamaina commented Apr 13, 2023

nmkj-io commented Apr 13, 2023 • edited Loading

tamaina commented Apr 13, 2023

tamaina commented Apr 13, 2023 • edited Loading

tamaina commented Apr 27, 2023

tamaina commented Apr 29, 2023

saschanaz commented Apr 29, 2023

tamaina commented May 2, 2023

tamaina commented Jul 6, 2023

tamaina commented Jul 6, 2023 • edited Loading

tamaina commented Feb 19, 2024

nmkj-io commented Apr 12, 2023 •

edited by tamaina

Loading

codecov bot commented Apr 12, 2023 •

edited

Loading

naskya commented Apr 13, 2023 •

edited

Loading

nmkj-io commented Apr 13, 2023 •

edited

Loading

nmkj-io commented Apr 13, 2023 •

edited

Loading

tamaina commented Apr 13, 2023 •

edited

Loading

tamaina commented Jul 6, 2023 •

edited

Loading