凍結されたユーザーが含まれている場合一部の機能が誤動作する不具合を修正 (MisskeyIO#161)

z-6561

packages/backend/src/core/entities/NoteReactionEntityService.ts

@@ -64,4 +64,17 @@ export class NoteReactionEntityService implements OnModuleInit {
 			} : {}),
 		};
 	}
+
+	@bindThis
+	public async packMany(
+		reactions: (MiNoteReaction['id'] | MiNoteReaction)[],
+		me: { id: MiUser['id'] } | null | undefined,
+		options?: {
+			withNote: boolean;
+		},
+	) : Promise<Packed<'NoteReaction'>[]> {
+		return (await Promise.allSettled(reactions.map(x => this.pack(x, me, options))))
+			.filter(result => result.status === 'fulfilled')
+			.map(result => (result as PromiseFulfilledResult<Packed<'NoteReaction'>>).value);
+	}
 }

packages/backend/src/server/api/endpoints/drive/files/find.ts

@@ -55,7 +55,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> {
 				folderId: ps.folderId ?? IsNull(),
 			});
 
-			return await Promise.all(files.map(file => this.driveFileEntityService.pack(file, me, { self: true })));
+			return await this.driveFileEntityService.packMany(files, me, { self: true });
 		});
 	}
 }

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

@@ -83,7 +83,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> {
 				relations: ['user', 'note'],
 			});
 
-			return await Promise.all(reactions.map(reaction => this.noteReactionEntityService.pack(reaction, me)));
+			return await this.noteReactionEntityService.packMany(reactions, me);
 		});
 	}
 }

packages/backend/src/server/api/endpoints/users/reactions.ts

@@ -81,7 +81,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> {
 				.limit(ps.limit)
 				.getMany();
 
-			return await Promise.all(reactions.map(reaction => this.noteReactionEntityService.pack(reaction, me, { withNote: true })));
+			return await this.noteReactionEntityService.packMany(reactions, me, { withNote: true });
 		});
 	}
 }

packages/backend/src/server/api/endpoints/users/show.ts

@@ -113,12 +113,13 @@ export default class extends Endpoint<typeof meta, typeof paramDef> {
 				// リクエストされた通りに並べ替え
 				const _users: MiUser[] = [];
 				for (const id of ps.userIds) {
-					_users.push(users.find(x => x.id === id)!);
+					const user = users.find((u) => u.id === id);
+					if (user) _users.push(user);
 				}
 
-				return await Promise.all(_users.map(u => this.userEntityService.pack(u, me, {
+				return await this.userEntityService.packMany(_users, me, {
 					detail: true,
-				})));
+				});
 			} else {
 				// Lookup user
 				if (typeof ps.host === 'string' && typeof ps.username === 'string') {