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

refactor(backend): meUpdatedイベントに更新後のアカウント情報を含まないように #15063

Draft
wants to merge 5 commits into
base: develop
Choose a base branch
from

Commits on Nov 27, 2024

  1. WebSocketのmainStreamに流れるmeUpdatedイベントでアップデート後のアカウント情報を流さないように

    フロントエンドが`/api/i`を呼び出すことでアカウント情報を得るようにした。これにより
    
    - Redis PubSubストリーム内を巨大なデータが行き来する回数を減らしパフォーマンスが向上することが考えられる
    - 各`publish`処理が正確なアカウント情報を得るために`UserEntityService`に依存する状況を解消できる
    
    特に後者は重要で、`UserEntityService`という巨大なサービスへの依存を減らせるのはコードの見通しもよくなる。現に`AccountMoveService`は`UserEntityService`への依存のせいで、`ApPersonService`と合わせて循環依存を作ってしまっている。これが解消できるのは大きい。
    
    (cherry picked from commit 0d4cf43e61887c110a840f108ea5fea9afb1263b)
    okayurisotto authored and kakkokari-gtyih committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    a32092a View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    353882e View commit details
    Browse the repository at this point in the history
  3. build misskey-js

    kakkokari-gtyih committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    edd9a94 View commit details
    Browse the repository at this point in the history
  4. fix

    kakkokari-gtyih committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    fa56d6e View commit details
    Browse the repository at this point in the history
  5. Update Changelog

    kakkokari-gtyih committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    4628a8f View commit details
    Browse the repository at this point in the history