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

V13.11.x系のサーバーにおいて、他のサーバーの情報が取得できない #10631

Open
sasagar opened this issue Apr 14, 2023 · 10 comments
Assignees
Labels
⚠️bug? This might be a bug

Comments

@sasagar
Copy link
Contributor

sasagar commented Apr 14, 2023

💡 Summary

v13.11.x系のサーバーから他のサーバーのNoteを表示させたときに、
Noteに表示されるべきサーバー名・アイコン・サーバーカラーが反映されない

🥰 Expected Behavior

全てのリモートユーザーのNoteに適切なサーバー名が表示され、サーバーカラーの背景がつき、サーバーアイコンが表示される

🤬 Actual Behavior

A -> B を「AのサーバーでBのNoteを見た際」としたとき、以下の条件でサーバー情報がNoteに表示されない

v13.11.x -> v13.11.x
v13.11.x -> v13.10.x
v13.11.x -> v.13.9.x
v13.11.x -> Mastodon系サーバー

なお、逆向きは反映される。(検証不十分な可能性はあり)
ただし、どうやら以前から連合になっているサーバーの情報などは問題なく表示されている。

📝 Steps to Reproduce

  1. v13.11.x系でc13.11.xへのアップデート以前から連合関係にないサーバーのNoteを閲覧する
  2. サーバー情報がない

📌 Environment

Misskey version: v13.11.3 / v13.10.3
Your OS: CentOS Stream 8
Your browser: Chrome 112.0.5615.87

@sasagar sasagar added the ⚠️bug? This might be a bug label Apr 14, 2023
@nexryai
Copy link
Contributor

nexryai commented Apr 16, 2023

v13.11.x → dolphinで再現しました(v13.10.xからは正常に取得できているのでDolphin側での問題ではなさそう)

https://misskey.io/@nexryai@s.nexryai.online

@nexryai
Copy link
Contributor

nexryai commented Apr 16, 2023

/instance-infoから見ると普通に取得できてるっぽいです

image

@GrapeApple0
Copy link
Contributor

こちらの方でいろいろ試した結果、1377ea4 が原因になっていました(これ以前のコミットをチェックアウトするとちゃんと動いたため)。

@tamaina
Copy link
Member

tamaina commented Apr 18, 2023

(RedisKVCache系の実装、副作用多くない?)

@sasagar
Copy link
Contributor Author

sasagar commented Apr 18, 2023

packages/backend/src/core/entities/UserEntityService.ts
において、 1377ea4 で大幅に削除されただけの部分があるのは良いのでしょうかね...?
L122あたりからの部分が削除されて代替の記述は増えていない様子でした。

@GrapeApple0
Copy link
Contributor

初期状態(id/host/firstRetrievedAtのみ書き込まれている)のデータを削除して再度取得させると正しく表示されたので、おそらく更新に失敗している感じだと思います。

@tar-bin
Copy link
Contributor

tar-bin commented Apr 22, 2023

こちら自分も再現しました。
image
Redisの<domain>:kvcache:federatedInstance:<host> にnullが入ってるようですね。
連合先のサーバーで redis-cli --scan --pattern '<domain>:kvcache:federatedInstance:*' | xargs redis-cli DEL でキャッシュを消すと解消しました。

@GrapeApple0
Copy link
Contributor

GrapeApple0 commented Apr 22, 2023

強制的にアップデートするようにした結果ちゃんと動くようになった(GrapeApple0@e366aa9 )ので作成時のデータをずっと引きずってる感じです。

@syuilo syuilo self-assigned this Apr 22, 2023
@syuilo
Copy link
Member

syuilo commented Apr 22, 2023

メモリキャッシュを3min、Redisキャッシュを30minに短くしたのと、更新があったときにRedisキャッシュにも反映させるようにしてみた

sasagar pushed a commit to sasagar/misskey that referenced this issue Apr 26, 2023
sasagar pushed a commit to sasagar/misskey that referenced this issue May 9, 2023
na2na-p pushed a commit to na2na-p/misskey that referenced this issue May 10, 2023
@fu-sen
Copy link

fu-sen commented Oct 19, 2023

古い Issues を上げますが。サーバー管理者関連の Discord でこの問題が出ている報告を受けています。
この原因がどうも単に OS パッケージによる Redis 6 系を維持していて Redis 7 へ更新していないのが原因のようです。
これを経験している場合、公式の最新 Redis を使用するようにして下さい。
https://misskey-hub.net/docs/releases.html#_13-11-0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

No branches or pull requests

7 participants