Merge 400aafddd7 into 763c708253

CHANGELOG.md

@@ -65,6 +65,8 @@
   (Cherry-picked from https://activitypub.software/TransFem-org/Sharkey/-/merge_requests/709)
 - Fix: User Webhookテスト機能のMock Payloadを修正  
 - Fix: アカウント削除のモデレーションログが動作していないのを修正 (#14996)  
+- Fix: ユーザーミュートにおいて、ノート内のメンションが考慮されていなかった問題を修正
+  - これにより、第三者から自分に対するノートを意図せず取り逃してしまう可能性があったため、通知欄ではメンションを考慮しないままになっています
 
 ### Misskey.js
 - Fix: Stream初期化時、別途WebSocketを指定する場合の型定義を修正

packages/backend/src/core/QueryService.ts

@@ -127,13 +127,18 @@ export class QueryService {
 	}
 
 	@bindThis
-	public generateMutedUserQuery(q: SelectQueryBuilder<any>, me: { id: MiUser['id'] }, exclude?: { id: MiUser['id'] }): void {
+	public generateMutedUserQuery(q: SelectQueryBuilder<any>, me: { id: MiUser['id'] }, exclude?: { id: MiUser['id'] }, checkMentions: boolean = true): void {
 		const mutingQuery = this.mutingsRepository.createQueryBuilder('muting')
 			.select('muting.muteeId')
 			.where('muting.muterId = :muterId', { muterId: me.id });
 
+		const mutingArrayQuery = this.mutingsRepository.createQueryBuilder('muting')
+			.select('array_agg(muting.muteeId)', 'muting.muteeIdArray')
+			.where('muting.muterId = :muterId', { muterId: me.id });
+
 		if (exclude) {
 			mutingQuery.andWhere('muting.muteeId != :excludeId', { excludeId: exclude.id });
+			mutingArrayQuery.andWhere('muting.muteeId != :excludeId', { excludeId: exclude.id });
 		}
 
 		const mutingInstanceQuery = this.userProfilesRepository.createQueryBuilder('user_profile')
@@ -172,7 +177,17 @@ export class QueryService {
 					.orWhere(`NOT ((${ mutingInstanceQuery.getQuery() })::jsonb ? note.renoteUserHost)`);
 			}));
 
+		// 投稿に含まれるメンションの相手をミュートしていない
+		if (checkMentions) {
+			q.andWhere(new Brackets(qb => {
+				qb
+					.where('note.mentions IS NULL')
+					.orWhere(`NOT (note.mentions && (${ mutingArrayQuery.getQuery() }))`);
+			}));
+		}
+
 		q.setParameters(mutingQuery.getParameters());
+		q.setParameters(mutingArrayQuery.getParameters());
 		q.setParameters(mutingInstanceQuery.getParameters());
 	}
 

packages/backend/src/misc/is-user-related.ts

@@ -12,6 +12,10 @@ export function isUserRelated(note: any, userIds: Set<string>, ignoreAuthor = fa
 		return true;
 	}
 
+	if (note.mentions != null && note.mentions.filter((userId: string) => userId !== note.userId && userIds.has(userId)).length > 0) {
+		return true;
+	}
+
 	if (note.reply != null && note.reply.userId !== note.userId && userIds.has(note.reply.userId)) {
 		return true;
 	}

packages/backend/src/server/api/endpoints/notes/mentions.ts

@@ -74,7 +74,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
 				.leftJoinAndSelect('renote.user', 'renoteUser');
 
 			this.queryService.generateVisibilityQuery(query, me);
-			this.queryService.generateMutedUserQuery(query, me);
+			this.queryService.generateMutedUserQuery(query, me, undefined, false); // 通知系ではメンションを確認しないようにする
 			this.queryService.generateMutedNoteThreadQuery(query, me);
 			this.queryService.generateBlockedUserQuery(query, me);