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

プロフィールページのノートタブで誤ってリノートが含まれることがある #12306

Closed
tkmrgit opened this issue Nov 11, 2023 · 3 comments
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR

Comments

@tkmrgit
Copy link

tkmrgit commented Nov 11, 2023

💡 Summary

2023.11.0(stable)で確認。プロフィールページで「全て」タブではなく「ノート」タブではリノートが表示されないはずなのだが、何らかの条件によりリノートが誤って含まれている
rsyeryey

🥰 Expected Behavior

「ノート」タブにリノートは含まれない

🤬 Actual Behavior

「ノート」タブにリノートが含まれる

📝 Steps to Reproduce

※発生条件不明。考えられるのはアップデート前にしか投稿が存在しないこと。
ex : https://misskey.systems/@misskey_dev

なお、アップデート後に何らかの投稿操作を行うと解消される(他アカウントにてそのような動作を確認)
Redisが関係している模様※下記参考情報を参考

💻 Frontend Environment

* Model and OS of the device(s): All
* Browser: All
* Server URL: misskey.systems
* Misskey: 2023.11.0

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:
@tkmrgit tkmrgit added the ⚠️bug? This might be a bug label Nov 11, 2023
@tkmrgit tkmrgit changed the title プロフィールページのノートタブでリノート誤って含まれることがある プロフィールページのノートタブで誤ってリノートが含まれることがある Nov 11, 2023
@tkmrgit
Copy link
Author

tkmrgit commented Nov 11, 2023

参考情報

みてきた。バグです。

  • 「リノートを含むか」というフラグが2種類あり、Redisから取る時とDBから取るときで違うフラグを見ていた
  • フロントから渡しているフラグは1種類のみ(Redisから取る時にのみ使用されるフラグだけ)

1度投稿すると直るのは、Redisにキャッシュされて意図したフラグを使う方の処理に流れるからだと思う。過去のにリノートが紛れ込むのは、Redisにノートが無くてDBまでフォールバックし、正しくないフラグを使って「リノートを含むか」を判定したから
https://misskey.systems/notes/9lx3q8ofhm

@samunohito
Copy link
Member

↑のノートは自分の書き込みなので、このまま自分が対応します。

@samunohito
Copy link
Member

@tkmrgit
develop最新にて対応済みです:pray:

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