-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix(backend): ブロックした相手から自分のノートが見えないように(/users/featured-notes, /users/notes) #12511
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #12511 +/- ##
===========================================
- Coverage 78.76% 78.75% -0.01%
===========================================
Files 955 955
Lines 103873 103889 +16
Branches 8337 8332 -5
===========================================
+ Hits 81814 81819 +5
- Misses 22059 22070 +11 ☔ View full report in Codecov by Sentry. |
このPRによるapi.jsonの差分 差分はこちら |
kanarikanaru
changed the title
fix: ブロックした相手から自分のノートが見えないように(ユーザー,チャンネル)
fix(backend): ブロックした相手から自分のノートが見えないように(ユーザー,チャンネル)
Nov 30, 2023
運営しているBackspaceKeyで適用しましたが、特に問題なく動いています。 |
anatawa12
reviewed
Dec 4, 2023
kanarikanaru
changed the title
fix(backend): ブロックした相手から自分のノートが見えないように(ユーザー,チャンネル)
fix(backend): ブロックした相手から自分のノートが見えないように(/users/featured-notes, /users/notes)
Dec 4, 2023
anatawa12
reviewed
Dec 4, 2023
- ブロックされている場合は、/users/notesでノートが表示されない - ミュートしている場合は、ノートが表示される
@syuilo こちらも確認していただけないでしょうか。何度も、また他人のPRでpingすみません。 |
🙏🏻 🙏🏻 |
Merged
5 tasks
camilla-ett
pushed a commit
to kaseiski/misskey
that referenced
this pull request
Jan 2, 2024
…notes) (misskey-dev#12511) * fix: ブロックした相手から自分のノートが見えないように(ユーザー,チャンネル) * Update CHANGELOG.md * /users/featured-notesでもブロックを考慮するように * cacheServiceを使うように * /channels/timeline.tsで必要のないnoteFilterを持たないように * Update CHANGELOG.md * FanoutTimelineEndpointServiceへの対応 - ブロックされている場合は、/users/notesでノートが表示されない - ミュートしている場合は、ノートが表示される
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What
/users/featured-notes, /users/notesのFTT有効時に被ブロック判定を追加し、特定のユーザーにブロックされている場合にブロックを行ったユーザーのノートを含んで返さないようにしました。
Why
FTTが導入されてから、Redisにキャッシュされたノートに限定して起きる問題です。
ユーザー1が、ユーザー2をブロックしたとして、
ユーザー2からは、ユーザー1のノートが、ユーザー画面では閲覧できるようになっていたため。
Additional info (optional)
Checklist