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.

Sign up for GitHub

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

Jump to bottom

HTLに未フォローチャンネルノートが含まれてしまう #12181

Closed
setaria430 opened this issue Oct 29, 2023 · 3 comments
Closed

HTLに未フォローチャンネルノートが含まれてしまう #12181

setaria430 opened this issue Oct 29, 2023 · 3 comments
Assignees
@samunohito
Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR

Comments

@setaria430
Copy link
Contributor

💡 Summary

ホームタイムラインに、フォローアカウントの未フォローチャンネルが含まれる
streamingでは発生せず、遡った時のapi/notes/timelineのレスポンスに混入する
未フォローアカウントの未フォローチャンネルノートは含まれない
ホームタイムライン上部の設定（リノートを表示・ファイル付きのみ）を何れの組み合わせにしても影響しない
フォローが少ない別のアカウントでは再現しなかったため、条件が絞りきれていない

🥰 Expected Behavior

未フォローチャンネルはHTLに含まれない

🤬 Actual Behavior

含まれる

📝 Steps to Reproduce

  1. HTLを遡る（もっと見る）
  2. 以上

📌 Environment

Image 220

💻 Frontend

  • Model and OS of the device(s):
    Windows 10pro
  • Browser:
    Chrome 118.0.5993.89
  • Server URL:
    misskey.gamelore.fun
  • Misskey:
    2023.11.0-beta.4

🛰 Backend (for server admin)

  • Installation Method or Hosting Service:
  • Misskey: 13.x.x
  • Node: 20.x.x
  • PostgreSQL: 15.x.x
  • Redis: 7.x.x
  • OS and Architecture:
@setaria430 setaria430 added the ⚠️bug? This might be a bug label Oct 29, 2023
@samunohito
Copy link
Member

@syuilo
#12179 の対処が甘かったようです。すみません。
自分が引き続き対処しても問題ありませんか？

@syuilo
Copy link
Member

syuilo commented Oct 29, 2023

@syuilo #12179 の対処が甘かったようです。すみません。 自分が引き続き対処しても問題ありませんか？

お願いします 🙏🏻

@syuilo syuilo added 🐛Bug Unexpected behavior packages/backend Server side specific issue/PR and removed ⚠️bug? This might be a bug labels Oct 29, 2023
samunohito added a commit to samunohito/misskey that referenced this issue Oct 29, 2023
@samunohito
misskey-dev#12181 の問題に対処
50eea64 
「ユーザー・チャンネルともにフォローあり」のときの絞り込みに不備があり、フォローしているユーザの投稿であればチャンネルのフォロー関係なく通過するようになってしまっていた
@samunohito samunohito mentioned this issue Oct 29, 2023
Merged
5 tasks
syuilo pushed a commit that referenced this issue Oct 29, 2023
@samunohito
Fix: フォローしたユーザが、自分のフォローしていないチャンネルにノートを投稿した時、そのノートがHTLで見えてしまう (#12186)
2de4d33 
* #12181 の問題に対処
「ユーザー・チャンネルともにフォローあり」のときの絞り込みに不備があり、フォローしているユーザの投稿であればチャンネルのフォロー関係なく通過するようになってしまっていた

* fix CHANGELOG.md

---------

Co-authored-by: osamu <46447427+sam-osamu@users.noreply.github.com>
@syuilo syuilo mentioned this issue Oct 30, 2023
Merged
@tai-cha
Copy link
Member

tai-cha commented Nov 3, 2023

修正済みなので閉じます
#12186

@tai-cha tai-cha closed this as completed Nov 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@samunohito samunohito

Labels
🐛Bug Unexpected behavior packages/backend Server side specific issue/PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@syuilo @tai-cha @samunohito @setaria430