summaryrefslogtreecommitdiff
path: root/packages/backend/src/core/entities
diff options
context:
space:
mode:
Diffstat (limited to 'packages/backend/src/core/entities')
-rw-r--r--packages/backend/src/core/entities/AbuseUserReportEntityService.ts36
-rw-r--r--packages/backend/src/core/entities/InstanceEntityService.ts36
-rw-r--r--packages/backend/src/core/entities/NoteEntityService.ts337
-rw-r--r--packages/backend/src/core/entities/UserEntityService.ts304
4 files changed, 465 insertions, 248 deletions
diff --git a/packages/backend/src/core/entities/AbuseUserReportEntityService.ts b/packages/backend/src/core/entities/AbuseUserReportEntityService.ts
index 70ead890ab..c1d877aa12 100644
--- a/packages/backend/src/core/entities/AbuseUserReportEntityService.ts
+++ b/packages/backend/src/core/entities/AbuseUserReportEntityService.ts
@@ -5,13 +5,14 @@
import { Inject, Injectable } from '@nestjs/common';
import { DI } from '@/di-symbols.js';
-import type { AbuseUserReportsRepository } from '@/models/_.js';
+import type { AbuseUserReportsRepository, InstancesRepository, MiInstance, MiUser } from '@/models/_.js';
import { awaitAll } from '@/misc/prelude/await-all.js';
import type { MiAbuseUserReport } from '@/models/AbuseUserReport.js';
import { bindThis } from '@/decorators.js';
import { IdService } from '@/core/IdService.js';
import type { Packed } from '@/misc/json-schema.js';
import { UserEntityService } from './UserEntityService.js';
+import { InstanceEntityService } from './InstanceEntityService.js';
@Injectable()
export class AbuseUserReportEntityService {
@@ -19,6 +20,10 @@ export class AbuseUserReportEntityService {
@Inject(DI.abuseUserReportsRepository)
private abuseUserReportsRepository: AbuseUserReportsRepository,
+ @Inject(DI.instancesRepository)
+ private instancesRepository: InstancesRepository,
+
+ private readonly instanceEntityService: InstanceEntityService,
private userEntityService: UserEntityService,
private idService: IdService,
) {
@@ -30,11 +35,14 @@ export class AbuseUserReportEntityService {
hint?: {
packedReporter?: Packed<'UserDetailedNotMe'>,
packedTargetUser?: Packed<'UserDetailedNotMe'>,
+ packedTargetInstance?: Packed<'FederationInstance'>,
packedAssignee?: Packed<'UserDetailedNotMe'>,
},
+ me?: MiUser | null,
) {
const report = typeof src === 'object' ? src : await this.abuseUserReportsRepository.findOneByOrFail({ id: src });
+ // noinspection ES6MissingAwait
return await awaitAll({
id: report.id,
createdAt: this.idService.parse(report.id).date.toISOString(),
@@ -43,13 +51,22 @@ export class AbuseUserReportEntityService {
reporterId: report.reporterId,
targetUserId: report.targetUserId,
assigneeId: report.assigneeId,
- reporter: hint?.packedReporter ?? this.userEntityService.pack(report.reporter ?? report.reporterId, null, {
+ reporter: hint?.packedReporter ?? this.userEntityService.pack(report.reporter ?? report.reporterId, me, {
schema: 'UserDetailedNotMe',
}),
- targetUser: hint?.packedTargetUser ?? this.userEntityService.pack(report.targetUser ?? report.targetUserId, null, {
+ targetUser: hint?.packedTargetUser ?? this.userEntityService.pack(report.targetUser ?? report.targetUserId, me, {
schema: 'UserDetailedNotMe',
}),
- assignee: report.assigneeId ? hint?.packedAssignee ?? this.userEntityService.pack(report.assignee ?? report.assigneeId, null, {
+ // return hint, or pack by relation, or fetch and pack by id, or null
+ targetInstance: hint?.packedTargetInstance ?? (
+ report.targetUserInstance
+ ? this.instanceEntityService.pack(report.targetUserInstance, me)
+ : report.targetUserHost
+ ? this.instancesRepository.findOneBy({ host: report.targetUserHost }).then(instance => instance
+ ? this.instanceEntityService.pack(instance, me)
+ : null)
+ : null),
+ assignee: report.assigneeId ? hint?.packedAssignee ?? this.userEntityService.pack(report.assignee ?? report.assigneeId, me, {
schema: 'UserDetailedNotMe',
}) : null,
forwarded: report.forwarded,
@@ -61,21 +78,28 @@ export class AbuseUserReportEntityService {
@bindThis
public async packMany(
reports: MiAbuseUserReport[],
+ me?: MiUser | null,
) {
const _reporters = reports.map(({ reporter, reporterId }) => reporter ?? reporterId);
const _targetUsers = reports.map(({ targetUser, targetUserId }) => targetUser ?? targetUserId);
const _assignees = reports.map(({ assignee, assigneeId }) => assignee ?? assigneeId).filter(x => x != null);
const _userMap = await this.userEntityService.packMany(
[..._reporters, ..._targetUsers, ..._assignees],
- null,
+ me,
{ schema: 'UserDetailedNotMe' },
).then(users => new Map(users.map(u => [u.id, u])));
+ const _targetInstances = reports
+ .map(({ targetUserInstance, targetUserHost }) => targetUserInstance ?? targetUserHost)
+ .filter((i): i is MiInstance | string => i != null);
+ const _instanceMap = await this.instanceEntityService.packMany(await this.instanceEntityService.fetchInstancesByHost(_targetInstances), me)
+ .then(instances => new Map(instances.map(i => [i.host, i])));
return Promise.all(
reports.map(report => {
const packedReporter = _userMap.get(report.reporterId);
const packedTargetUser = _userMap.get(report.targetUserId);
+ const packedTargetInstance = report.targetUserHost ? _instanceMap.get(report.targetUserHost) : undefined;
const packedAssignee = report.assigneeId != null ? _userMap.get(report.assigneeId) : undefined;
- return this.pack(report, { packedReporter, packedTargetUser, packedAssignee });
+ return this.pack(report, { packedReporter, packedTargetUser, packedAssignee, packedTargetInstance }, me);
}),
);
}
diff --git a/packages/backend/src/core/entities/InstanceEntityService.ts b/packages/backend/src/core/entities/InstanceEntityService.ts
index fcc9bed3bd..4ca4ff650b 100644
--- a/packages/backend/src/core/entities/InstanceEntityService.ts
+++ b/packages/backend/src/core/entities/InstanceEntityService.ts
@@ -4,6 +4,7 @@
*/
import { Inject, Injectable } from '@nestjs/common';
+import { In } from 'typeorm';
import type { Packed } from '@/misc/json-schema.js';
import type { MiInstance } from '@/models/Instance.js';
import { bindThis } from '@/decorators.js';
@@ -11,7 +12,7 @@ import { UtilityService } from '@/core/UtilityService.js';
import { RoleService } from '@/core/RoleService.js';
import { MiUser } from '@/models/User.js';
import { DI } from '@/di-symbols.js';
-import { MiMeta } from '@/models/_.js';
+import type { InstancesRepository, MiMeta } from '@/models/_.js';
@Injectable()
export class InstanceEntityService {
@@ -19,6 +20,9 @@ export class InstanceEntityService {
@Inject(DI.meta)
private meta: MiMeta,
+ @Inject(DI.instancesRepository)
+ private readonly instancesRepository: InstancesRepository,
+
private roleService: RoleService,
private utilityService: UtilityService,
@@ -43,7 +47,7 @@ export class InstanceEntityService {
isNotResponding: instance.isNotResponding,
isSuspended: instance.suspensionState !== 'none',
suspensionState: instance.suspensionState,
- isBlocked: this.utilityService.isBlockedHost(this.meta.blockedHosts, instance.host),
+ isBlocked: instance.isBlocked,
softwareName: instance.softwareName,
softwareVersion: instance.softwareVersion,
openRegistrations: instance.openRegistrations,
@@ -51,8 +55,8 @@ export class InstanceEntityService {
description: instance.description,
maintainerName: instance.maintainerName,
maintainerEmail: instance.maintainerEmail,
- isSilenced: this.utilityService.isSilencedHost(this.meta.silencedHosts, instance.host),
- isMediaSilenced: this.utilityService.isMediaSilencedHost(this.meta.mediaSilencedHosts, instance.host),
+ isSilenced: instance.isSilenced,
+ isMediaSilenced: instance.isMediaSilenced,
iconUrl: instance.iconUrl,
faviconUrl: instance.faviconUrl,
themeColor: instance.themeColor,
@@ -62,6 +66,7 @@ export class InstanceEntityService {
rejectReports: instance.rejectReports,
rejectQuotes: instance.rejectQuotes,
moderationNote: iAmModerator ? instance.moderationNote : null,
+ isBubbled: this.utilityService.isBubbledHost(instance.host),
};
}
@@ -72,5 +77,28 @@ export class InstanceEntityService {
) {
return Promise.all(instances.map(x => this.pack(x, me)));
}
+
+ @bindThis
+ public async fetchInstancesByHost(instances: (MiInstance | MiInstance['host'])[]): Promise<MiInstance[]> {
+ const result: MiInstance[] = [];
+
+ const toFetch: string[] = [];
+ for (const instance of instances) {
+ if (typeof(instance) === 'string') {
+ toFetch.push(instance);
+ } else {
+ result.push(instance);
+ }
+ }
+
+ if (toFetch.length > 0) {
+ const fetched = await this.instancesRepository.findBy({
+ host: In(toFetch),
+ });
+ result.push(...fetched);
+ }
+
+ return result;
+ }
}
diff --git a/packages/backend/src/core/entities/NoteEntityService.ts b/packages/backend/src/core/entities/NoteEntityService.ts
index 6ada5463a3..4248fde77f 100644
--- a/packages/backend/src/core/entities/NoteEntityService.ts
+++ b/packages/backend/src/core/entities/NoteEntityService.ts
@@ -11,12 +11,13 @@ import type { Packed } from '@/misc/json-schema.js';
import { awaitAll } from '@/misc/prelude/await-all.js';
import type { MiUser } from '@/models/User.js';
import type { MiNote } from '@/models/Note.js';
-import type { UsersRepository, NotesRepository, FollowingsRepository, PollsRepository, PollVotesRepository, NoteReactionsRepository, ChannelsRepository, MiMeta } from '@/models/_.js';
+import type { UsersRepository, NotesRepository, FollowingsRepository, PollsRepository, PollVotesRepository, NoteReactionsRepository, ChannelsRepository, MiMeta, MiPollVote, MiPoll, MiChannel, MiFollowing } from '@/models/_.js';
import { bindThis } from '@/decorators.js';
import { DebounceLoader } from '@/misc/loader.js';
import { IdService } from '@/core/IdService.js';
import { ReactionsBufferingService } from '@/core/ReactionsBufferingService.js';
import { isPackedPureRenote } from '@/misc/is-renote.js';
+import type { Config } from '@/config.js';
import type { OnModuleInit } from '@nestjs/common';
import type { CacheService } from '../CacheService.js';
import type { CustomEmojiService } from '../CustomEmojiService.js';
@@ -25,13 +26,13 @@ import type { UserEntityService } from './UserEntityService.js';
import type { DriveFileEntityService } from './DriveFileEntityService.js';
// is-renote.tsとよしなにリンク
-function isPureRenote(note: MiNote): note is MiNote & { renoteId: MiNote['id']; renote: MiNote } {
+function isPureRenote(note: MiNote): note is MiNote & { renoteId: MiNote['id'] } {
return (
- note.renote != null &&
- note.reply == null &&
+ note.renoteId != null &&
+ note.replyId == null &&
note.text == null &&
note.cw == null &&
- (note.fileIds == null || note.fileIds.length === 0) &&
+ note.fileIds.length === 0 &&
!note.hasPoll
);
}
@@ -92,6 +93,9 @@ export class NoteEntityService implements OnModuleInit {
@Inject(DI.channelsRepository)
private channelsRepository: ChannelsRepository,
+ @Inject(DI.config)
+ private readonly config: Config,
+
//private userEntityService: UserEntityService,
//private driveFileEntityService: DriveFileEntityService,
//private customEmojiService: CustomEmojiService,
@@ -128,7 +132,10 @@ export class NoteEntityService implements OnModuleInit {
}
@bindThis
- public async hideNote(packedNote: Packed<'Note'>, meId: MiUser['id'] | null): Promise<void> {
+ public async hideNote(packedNote: Packed<'Note'>, meId: MiUser['id'] | null, hint?: {
+ myFollowing?: ReadonlyMap<string, unknown>,
+ myBlockers?: ReadonlySet<string>,
+ }): Promise<void> {
if (meId === packedNote.userId) return;
// TODO: isVisibleForMe を使うようにしても良さそう(型違うけど)
@@ -184,14 +191,9 @@ export class NoteEntityService implements OnModuleInit {
} else if (packedNote.renote && (meId === packedNote.renote.userId)) {
hide = false;
} else {
- // フォロワーかどうか
- // TODO: 当関数呼び出しごとにクエリが走るのは重そうだからなんとかする
- const isFollowing = await this.followingsRepository.exists({
- where: {
- followeeId: packedNote.userId,
- followerId: meId,
- },
- });
+ const isFollowing = hint?.myFollowing
+ ? hint.myFollowing.has(packedNote.userId)
+ : (await this.cacheService.userFollowingsCache.fetch(meId)).has(packedNote.userId);
hide = !isFollowing;
}
@@ -207,7 +209,8 @@ export class NoteEntityService implements OnModuleInit {
}
if (!hide && meId && packedNote.userId !== meId) {
- const isBlocked = (await this.cacheService.userBlockedCache.fetch(meId)).has(packedNote.userId);
+ const blockers = hint?.myBlockers ?? await this.cacheService.userBlockedCache.fetch(meId);
+ const isBlocked = blockers.has(packedNote.userId);
if (isBlocked) hide = true;
}
@@ -231,8 +234,11 @@ export class NoteEntityService implements OnModuleInit {
}
@bindThis
- private async populatePoll(note: MiNote, meId: MiUser['id'] | null) {
- const poll = await this.pollsRepository.findOneByOrFail({ noteId: note.id });
+ private async populatePoll(note: MiNote, meId: MiUser['id'] | null, hint?: {
+ poll?: MiPoll,
+ myVotes?: MiPollVote[],
+ }) {
+ const poll = hint?.poll ?? await this.pollsRepository.findOneByOrFail({ noteId: note.id });
const choices = poll.choices.map(c => ({
text: c,
votes: poll.votes[poll.choices.indexOf(c)],
@@ -241,7 +247,7 @@ export class NoteEntityService implements OnModuleInit {
if (meId) {
if (poll.multiple) {
- const votes = await this.pollVotesRepository.findBy({
+ const votes = hint?.myVotes ?? await this.pollVotesRepository.findBy({
userId: meId,
noteId: note.id,
});
@@ -251,7 +257,7 @@ export class NoteEntityService implements OnModuleInit {
choices[myChoice].isVoted = true;
}
} else {
- const vote = await this.pollVotesRepository.findOneBy({
+ const vote = hint?.myVotes ? hint.myVotes[0] : await this.pollVotesRepository.findOneBy({
userId: meId,
noteId: note.id,
});
@@ -313,7 +319,12 @@ export class NoteEntityService implements OnModuleInit {
}
@bindThis
- public async isVisibleForMe(note: MiNote, meId: MiUser['id'] | null): Promise<boolean> {
+ public async isVisibleForMe(note: MiNote, meId: MiUser['id'] | null, hint?: {
+ myFollowing?: ReadonlySet<string>,
+ myBlocking?: ReadonlySet<string>,
+ myBlockers?: ReadonlySet<string>,
+ me?: Pick<MiUser, 'host'> | null,
+ }): Promise<boolean> {
// This code must always be synchronized with the checks in generateVisibilityQuery.
// visibility が specified かつ自分が指定されていなかったら非表示
if (note.visibility === 'specified') {
@@ -341,16 +352,16 @@ export class NoteEntityService implements OnModuleInit {
return true;
} else {
// フォロワーかどうか
- const [blocked, following, user] = await Promise.all([
- this.cacheService.userBlockingCache.fetch(meId).then((ids) => ids.has(note.userId)),
- this.followingsRepository.count({
- where: {
- followeeId: note.userId,
- followerId: meId,
- },
- take: 1,
- }),
- this.usersRepository.findOneByOrFail({ id: meId }),
+ const [blocked, following, userHost] = await Promise.all([
+ hint?.myBlocking
+ ? hint.myBlocking.has(note.userId)
+ : this.cacheService.userBlockingCache.fetch(meId).then((ids) => ids.has(note.userId)),
+ hint?.myFollowing
+ ? hint.myFollowing.has(note.userId)
+ : this.cacheService.userFollowingsCache.fetch(meId).then(ids => ids.has(note.userId)),
+ hint?.me !== undefined
+ ? (hint.me?.host ?? null)
+ : this.cacheService.findUserById(meId).then(me => me.host),
]);
if (blocked) return false;
@@ -362,12 +373,13 @@ export class NoteEntityService implements OnModuleInit {
in which case we can never know the following. Instead we have
to assume that the users are following each other.
*/
- return following > 0 || (note.userHost != null && user.host != null);
+ return following || (note.userHost != null && userHost != null);
}
}
if (meId != null) {
- const isBlocked = (await this.cacheService.userBlockedCache.fetch(meId)).has(note.userId);
+ const blockers = hint?.myBlockers ?? await this.cacheService.userBlockedCache.fetch(meId);
+ const isBlocked = blockers.has(note.userId);
if (isBlocked) return false;
}
@@ -404,6 +416,12 @@ export class NoteEntityService implements OnModuleInit {
packedFiles: Map<MiNote['fileIds'][number], Packed<'DriveFile'> | null>;
packedUsers: Map<MiUser['id'], Packed<'UserLite'>>;
mentionHandles: Record<string, string | undefined>;
+ userFollowings: Map<string, Map<string, Omit<MiFollowing, 'isFollowerHibernated'>>>;
+ userBlockers: Map<string, Set<string>>;
+ polls: Map<string, MiPoll>;
+ pollVotes: Map<string, Map<string, MiPollVote[]>>;
+ channels: Map<string, MiChannel>;
+ notes: Map<string, MiNote>;
};
},
): Promise<Packed<'Note'>> {
@@ -433,9 +451,7 @@ export class NoteEntityService implements OnModuleInit {
}
const channel = note.channelId
- ? note.channel
- ? note.channel
- : await this.channelsRepository.findOneBy({ id: note.channelId })
+ ? (opts._hint_?.channels.get(note.channelId) ?? note.channel ?? await this.channelsRepository.findOneBy({ id: note.channelId }))
: null;
const reactionEmojiNames = Object.keys(reactions)
@@ -481,7 +497,10 @@ export class NoteEntityService implements OnModuleInit {
mentionHandles: note.mentions.length > 0 ? this.getUserHandles(note.mentions, options?._hint_?.mentionHandles) : undefined,
uri: note.uri ?? undefined,
url: note.url ?? undefined,
- poll: note.hasPoll ? this.populatePoll(note, meId) : undefined,
+ poll: note.hasPoll ? this.populatePoll(note, meId, {
+ poll: opts._hint_?.polls.get(note.id),
+ myVotes: opts._hint_?.pollVotes.get(note.id)?.get(note.userId),
+ }) : undefined,
...(meId && Object.keys(reactions).length > 0 ? {
myReaction: this.populateMyReaction({
@@ -495,14 +514,14 @@ export class NoteEntityService implements OnModuleInit {
clippedCount: note.clippedCount,
processErrors: note.processErrors,
- reply: note.replyId ? this.pack(note.reply ?? note.replyId, me, {
+ reply: note.replyId ? this.pack(note.reply ?? opts._hint_?.notes.get(note.replyId) ?? note.replyId, me, {
detail: false,
skipHide: opts.skipHide,
withReactionAndUserPairCache: opts.withReactionAndUserPairCache,
_hint_: options?._hint_,
}) : undefined,
- renote: note.renoteId ? this.pack(note.renote ?? note.renoteId, me, {
+ renote: note.renoteId ? this.pack(note.renote ?? opts._hint_?.notes.get(note.renoteId) ?? note.renoteId, me, {
detail: true,
skipHide: opts.skipHide,
withReactionAndUserPairCache: opts.withReactionAndUserPairCache,
@@ -514,7 +533,10 @@ export class NoteEntityService implements OnModuleInit {
this.treatVisibility(packed);
if (!opts.skipHide) {
- await this.hideNote(packed, meId);
+ await this.hideNote(packed, meId, meId == null ? undefined : {
+ myFollowing: opts._hint_?.userFollowings.get(meId),
+ myBlockers: opts._hint_?.userBlockers.get(meId),
+ });
}
return packed;
@@ -531,79 +553,139 @@ export class NoteEntityService implements OnModuleInit {
) {
if (notes.length === 0) return [];
- const targetNotes: MiNote[] = [];
+ const targetNotesMap = new Map<string, MiNote>();
+ const targetNotesToFetch : string[] = [];
for (const note of notes) {
if (isPureRenote(note)) {
// we may need to fetch 'my reaction' for renote target.
- targetNotes.push(note.renote);
- if (note.renote.reply) {
- // idem if the renote is also a reply.
- targetNotes.push(note.renote.reply);
+ if (note.renote) {
+ targetNotesMap.set(note.renote.id, note.renote);
+ if (note.renote.reply) {
+ // idem if the renote is also a reply.
+ targetNotesMap.set(note.renote.reply.id, note.renote.reply);
+ }
+ } else if (options?.detail) {
+ targetNotesToFetch.push(note.renoteId);
}
} else {
if (note.reply) {
// idem for OP of a regular reply.
- targetNotes.push(note.reply);
+ targetNotesMap.set(note.reply.id, note.reply);
+ } else if (note.replyId && options?.detail) {
+ targetNotesToFetch.push(note.replyId);
}
- targetNotes.push(note);
+ targetNotesMap.set(note.id, note);
}
}
- const bufferedReactions = this.meta.enableReactionsBuffering ? await this.reactionsBufferingService.getMany([...getAppearNoteIds(notes)]) : null;
+ // Don't fetch notes that were added by ID and then found inline in another note.
+ for (let i = targetNotesToFetch.length - 1; i >= 0; i--) {
+ if (targetNotesMap.has(targetNotesToFetch[i])) {
+ targetNotesToFetch.splice(i, 1);
+ }
+ }
- const meId = me ? me.id : null;
- const myReactionsMap = new Map<MiNote['id'], string | null>();
- if (meId) {
- const idsNeedFetchMyReaction = new Set<MiNote['id']>();
+ // Populate any relations that weren't included in the source
+ if (targetNotesToFetch.length > 0) {
+ const newNotes = await this.notesRepository.find({
+ where: {
+ id: In(targetNotesToFetch),
+ },
+ relations: {
+ user: {
+ userProfile: true,
+ },
+ reply: {
+ user: {
+ userProfile: true,
+ },
+ },
+ renote: {
+ user: {
+ userProfile: true,
+ },
+ reply: {
+ user: {
+ userProfile: true,
+ },
+ },
+ },
+ channel: true,
+ },
+ });
- for (const note of targetNotes) {
- const reactionsCount = Object.values(this.reactionsBufferingService.mergeReactions(note.reactions, bufferedReactions?.get(note.id)?.deltas ?? {})).reduce((a, b) => a + b, 0);
- if (reactionsCount === 0) {
- myReactionsMap.set(note.id, null);
- } else if (reactionsCount <= note.reactionAndUserPairCache.length + (bufferedReactions?.get(note.id)?.pairs.length ?? 0)) {
- const pairInBuffer = bufferedReactions?.get(note.id)?.pairs.find(p => p[0] === meId);
- if (pairInBuffer) {
- myReactionsMap.set(note.id, pairInBuffer[1]);
- } else {
- const pair = note.reactionAndUserPairCache.find(p => p.startsWith(meId));
- myReactionsMap.set(note.id, pair ? pair.split('/')[1] : null);
- }
- } else {
- idsNeedFetchMyReaction.add(note.id);
- }
+ for (const note of newNotes) {
+ targetNotesMap.set(note.id, note);
}
+ }
- const myReactions = idsNeedFetchMyReaction.size > 0 ? await this.noteReactionsRepository.findBy({
- userId: meId,
- noteId: In(Array.from(idsNeedFetchMyReaction)),
- }) : [];
+ const targetNotes = Array.from(targetNotesMap.values());
+ const noteIds = Array.from(targetNotesMap.keys());
- for (const id of idsNeedFetchMyReaction) {
- myReactionsMap.set(id, myReactions.find(reaction => reaction.noteId === id)?.reaction ?? null);
+ const usersMap = new Map<string, MiUser | string>();
+ const allUsers = notes.flatMap(note => [
+ note.user ?? note.userId,
+ note.reply?.user ?? note.replyUserId,
+ note.renote?.user ?? note.renoteUserId,
+ ]);
+
+ for (const user of allUsers) {
+ if (!user) continue;
+
+ if (typeof(user) === 'object') {
+ // ID -> Entity
+ usersMap.set(user.id, user);
+ } else if (!usersMap.has(user)) {
+ // ID -> ID
+ usersMap.set(user, user);
}
}
- await this.customEmojiService.prefetchEmojis(this.aggregateNoteEmojis(notes));
- // TODO: 本当は renote とか reply がないのに renoteId とか replyId があったらここで解決しておく
- const fileIds = notes.map(n => [n.fileIds, n.renote?.fileIds, n.reply?.fileIds]).flat(2).filter(x => x != null);
- const packedFiles = fileIds.length > 0 ? await this.driveFileEntityService.packManyByIdsMap(fileIds) : new Map();
- const users = [
- ...notes.map(({ user, userId }) => user ?? userId),
- ...notes.map(({ replyUserId }) => replyUserId).filter(x => x != null),
- ...notes.map(({ renoteUserId }) => renoteUserId).filter(x => x != null),
- ];
- const packedUsers = await this.userEntityService.packMany(users, me)
- .then(users => new Map(users.map(u => [u.id, u])));
+ const users = Array.from(usersMap.values());
+ const userIds = Array.from(usersMap.keys());
- // Recursively add all mentioned users from all notes + replies + renotes
- const allMentionedUsers = targetNotes.reduce((users, note) => {
- for (const user of note.mentions) {
- users.add(user);
- }
- return users;
- }, new Set<string>());
- const mentionHandles = await this.getUserHandles(Array.from(allMentionedUsers));
+ const fileIds = new Set(targetNotes.flatMap(n => n.fileIds));
+ const mentionedUsers = new Set(targetNotes.flatMap(note => note.mentions));
+
+ const [{ bufferedReactions, myReactionsMap }, packedFiles, packedUsers, mentionHandles, userFollowings, userBlockers, polls, pollVotes, channels] = await Promise.all([
+ // bufferedReactions & myReactionsMap
+ this.getReactions(targetNotes, me),
+ // packedFiles
+ this.driveFileEntityService.packManyByIdsMap(Array.from(fileIds)),
+ // packedUsers
+ this.userEntityService.packMany(users, me)
+ .then(users => new Map(users.map(u => [u.id, u]))),
+ // mentionHandles
+ this.getUserHandles(Array.from(mentionedUsers)),
+ // userFollowings
+ this.cacheService.userFollowingsCache.fetchMany(userIds).then(fs => new Map(fs)),
+ // userBlockers
+ this.cacheService.userBlockedCache.fetchMany(userIds).then(bs => new Map(bs)),
+ // polls
+ this.pollsRepository.findBy({ noteId: In(noteIds) })
+ .then(polls => new Map(polls.map(p => [p.noteId, p]))),
+ // pollVotes
+ this.pollVotesRepository.findBy({ noteId: In(noteIds), userId: In(userIds) })
+ .then(votes => votes.reduce((noteMap, vote) => {
+ let userMap = noteMap.get(vote.noteId);
+ if (!userMap) {
+ userMap = new Map<string, MiPollVote[]>();
+ noteMap.set(vote.noteId, userMap);
+ }
+ let voteList = userMap.get(vote.userId);
+ if (!voteList) {
+ voteList = [];
+ userMap.set(vote.userId, voteList);
+ }
+ voteList.push(vote);
+ return noteMap;
+ }, new Map<string, Map<string, MiPollVote[]>>)),
+ // channels
+ this.getChannels(targetNotes),
+ // (not returned)
+ this.customEmojiService.prefetchEmojis(this.aggregateNoteEmojis(notes)),
+ ]);
return await Promise.all(notes.map(n => this.pack(n, me, {
...options,
@@ -613,6 +695,12 @@ export class NoteEntityService implements OnModuleInit {
packedFiles,
packedUsers,
mentionHandles,
+ userFollowings,
+ userBlockers,
+ polls,
+ pollVotes,
+ channels,
+ notes: new Map(targetNotes.map(n => [n.id, n])),
},
})));
}
@@ -680,4 +768,71 @@ export class NoteEntityService implements OnModuleInit {
return map;
}, {} as Record<string, string | undefined>);
}
+
+ private async getChannels(notes: MiNote[]): Promise<Map<string, MiChannel>> {
+ const channels = new Map<string, MiChannel>();
+ const channelsToFetch = new Set<string>();
+
+ for (const note of notes) {
+ if (note.channel) {
+ channels.set(note.channel.id, note.channel);
+ } else if (note.channelId) {
+ channelsToFetch.add(note.channelId);
+ }
+ }
+
+ if (channelsToFetch.size > 0) {
+ const newChannels = await this.channelsRepository.findBy({
+ id: In(Array.from(channelsToFetch)),
+ });
+ for (const channel of newChannels) {
+ channels.set(channel.id, channel);
+ }
+ }
+
+ return channels;
+ }
+
+ private async getReactions(notes: MiNote[], me: { id: string } | null | undefined) {
+ const bufferedReactions = this.meta.enableReactionsBuffering ? await this.reactionsBufferingService.getMany([...getAppearNoteIds(notes)]) : null;
+
+ const meId = me ? me.id : null;
+ const myReactionsMap = new Map<MiNote['id'], string | null>();
+ if (meId) {
+ const idsNeedFetchMyReaction = new Set<MiNote['id']>();
+
+ for (const note of notes) {
+ const reactionsCount = Object.values(this.reactionsBufferingService.mergeReactions(note.reactions, bufferedReactions?.get(note.id)?.deltas ?? {})).reduce((a, b) => a + b, 0);
+ if (reactionsCount === 0) {
+ myReactionsMap.set(note.id, null);
+ } else if (reactionsCount <= note.reactionAndUserPairCache.length + (bufferedReactions?.get(note.id)?.pairs.length ?? 0)) {
+ const pairInBuffer = bufferedReactions?.get(note.id)?.pairs.find(p => p[0] === meId);
+ if (pairInBuffer) {
+ myReactionsMap.set(note.id, pairInBuffer[1]);
+ } else {
+ const pair = note.reactionAndUserPairCache.find(p => p.startsWith(meId));
+ myReactionsMap.set(note.id, pair ? pair.split('/')[1] : null);
+ }
+ } else {
+ idsNeedFetchMyReaction.add(note.id);
+ }
+ }
+
+ const myReactions = idsNeedFetchMyReaction.size > 0 ? await this.noteReactionsRepository.findBy({
+ userId: meId,
+ noteId: In(Array.from(idsNeedFetchMyReaction)),
+ }) : [];
+
+ for (const id of idsNeedFetchMyReaction) {
+ myReactionsMap.set(id, myReactions.find(reaction => reaction.noteId === id)?.reaction ?? null);
+ }
+ }
+
+ return { bufferedReactions, myReactionsMap };
+ }
+
+ @bindThis
+ public genLocalNoteUri(noteId: string): string {
+ return `${this.config.url}/notes/${noteId}`;
+ }
}
diff --git a/packages/backend/src/core/entities/UserEntityService.ts b/packages/backend/src/core/entities/UserEntityService.ts
index 56506a5fa4..638eaac16f 100644
--- a/packages/backend/src/core/entities/UserEntityService.ts
+++ b/packages/backend/src/core/entities/UserEntityService.ts
@@ -30,6 +30,7 @@ import type {
FollowingsRepository,
FollowRequestsRepository,
MiFollowing,
+ MiInstance,
MiMeta,
MiUserNotePining,
MiUserProfile,
@@ -42,7 +43,7 @@ import type {
UsersRepository,
} from '@/models/_.js';
import { bindThis } from '@/decorators.js';
-import { RoleService } from '@/core/RoleService.js';
+import { RolePolicies, RoleService } from '@/core/RoleService.js';
import { ApPersonService } from '@/core/activitypub/models/ApPersonService.js';
import { FederatedInstanceService } from '@/core/FederatedInstanceService.js';
import { IdService } from '@/core/IdService.js';
@@ -52,6 +53,7 @@ import { AvatarDecorationService } from '@/core/AvatarDecorationService.js';
import { ChatService } from '@/core/ChatService.js';
import { isSystemAccount } from '@/misc/is-system-account.js';
import { DriveFileEntityService } from '@/core/entities/DriveFileEntityService.js';
+import type { CacheService } from '@/core/CacheService.js';
import type { OnModuleInit } from '@nestjs/common';
import type { NoteEntityService } from './NoteEntityService.js';
import type { PageEntityService } from './PageEntityService.js';
@@ -77,7 +79,7 @@ function isRemoteUser(user: MiUser | { host: MiUser['host'] }): boolean {
export type UserRelation = {
id: MiUser['id']
- following: MiFollowing | null,
+ following: Omit<MiFollowing, 'isFollowerHibernated'> | null,
isFollowing: boolean
isFollowed: boolean
hasPendingFollowRequestFromYou: boolean
@@ -103,6 +105,7 @@ export class UserEntityService implements OnModuleInit {
private idService: IdService;
private avatarDecorationService: AvatarDecorationService;
private chatService: ChatService;
+ private cacheService: CacheService;
constructor(
private moduleRef: ModuleRef,
@@ -163,6 +166,7 @@ export class UserEntityService implements OnModuleInit {
this.idService = this.moduleRef.get('IdService');
this.avatarDecorationService = this.moduleRef.get('AvatarDecorationService');
this.chatService = this.moduleRef.get('ChatService');
+ this.cacheService = this.moduleRef.get('CacheService');
}
//#region Validators
@@ -193,16 +197,8 @@ export class UserEntityService implements OnModuleInit {
memo,
mutedInstances,
] = await Promise.all([
- this.followingsRepository.findOneBy({
- followerId: me,
- followeeId: target,
- }),
- this.followingsRepository.exists({
- where: {
- followerId: target,
- followeeId: me,
- },
- }),
+ this.cacheService.userFollowingsCache.fetch(me).then(f => f.get(target) ?? null),
+ this.cacheService.userFollowingsCache.fetch(target).then(f => f.has(me)),
this.followRequestsRepository.exists({
where: {
followerId: me,
@@ -215,45 +211,22 @@ export class UserEntityService implements OnModuleInit {
followeeId: me,
},
}),
- this.blockingsRepository.exists({
- where: {
- blockerId: me,
- blockeeId: target,
- },
- }),
- this.blockingsRepository.exists({
- where: {
- blockerId: target,
- blockeeId: me,
- },
- }),
- this.mutingsRepository.exists({
- where: {
- muterId: me,
- muteeId: target,
- },
- }),
- this.renoteMutingsRepository.exists({
- where: {
- muterId: me,
- muteeId: target,
- },
- }),
- this.usersRepository.createQueryBuilder('u')
- .select('u.host')
- .where({ id: target })
- .getRawOne<{ u_host: string }>()
- .then(it => it?.u_host ?? null),
+ this.cacheService.userBlockingCache.fetch(me)
+ .then(blockees => blockees.has(target)),
+ this.cacheService.userBlockedCache.fetch(me)
+ .then(blockers => blockers.has(target)),
+ this.cacheService.userMutingsCache.fetch(me)
+ .then(mutings => mutings.has(target)),
+ this.cacheService.renoteMutingsCache.fetch(me)
+ .then(mutings => mutings.has(target)),
+ this.cacheService.findUserById(target).then(u => u.host),
this.userMemosRepository.createQueryBuilder('m')
.select('m.memo')
.where({ userId: me, targetUserId: target })
.getRawOne<{ m_memo: string | null }>()
.then(it => it?.m_memo ?? null),
- this.userProfilesRepository.createQueryBuilder('p')
- .select('p.mutedInstances')
- .where({ userId: me })
- .getRawOne<{ p_mutedInstances: string[] }>()
- .then(it => it?.p_mutedInstances ?? []),
+ this.cacheService.userProfileCache.fetch(me)
+ .then(profile => profile.mutedInstances),
]);
const isInstanceMuted = !!host && mutedInstances.includes(host);
@@ -277,8 +250,8 @@ export class UserEntityService implements OnModuleInit {
@bindThis
public async getRelations(me: MiUser['id'], targets: MiUser['id'][]): Promise<Map<MiUser['id'], UserRelation>> {
const [
- followers,
- followees,
+ myFollowing,
+ myFollowers,
followersRequests,
followeesRequests,
blockers,
@@ -289,13 +262,8 @@ export class UserEntityService implements OnModuleInit {
memos,
mutedInstances,
] = await Promise.all([
- this.followingsRepository.findBy({ followerId: me })
- .then(f => new Map(f.map(it => [it.followeeId, it]))),
- this.followingsRepository.createQueryBuilder('f')
- .select('f.followerId')
- .where('f.followeeId = :me', { me })
- .getRawMany<{ f_followerId: string }>()
- .then(it => it.map(it => it.f_followerId)),
+ this.cacheService.userFollowingsCache.fetch(me),
+ this.cacheService.userFollowersCache.fetch(me),
this.followRequestsRepository.createQueryBuilder('f')
.select('f.followeeId')
.where('f.followerId = :me', { me })
@@ -306,34 +274,18 @@ export class UserEntityService implements OnModuleInit {
.where('f.followeeId = :me', { me })
.getRawMany<{ f_followerId: string }>()
.then(it => it.map(it => it.f_followerId)),
- this.blockingsRepository.createQueryBuilder('b')
- .select('b.blockeeId')
- .where('b.blockerId = :me', { me })
- .getRawMany<{ b_blockeeId: string }>()
- .then(it => it.map(it => it.b_blockeeId)),
- this.blockingsRepository.createQueryBuilder('b')
- .select('b.blockerId')
- .where('b.blockeeId = :me', { me })
- .getRawMany<{ b_blockerId: string }>()
- .then(it => it.map(it => it.b_blockerId)),
- this.mutingsRepository.createQueryBuilder('m')
- .select('m.muteeId')
- .where('m.muterId = :me', { me })
- .getRawMany<{ m_muteeId: string }>()
- .then(it => it.map(it => it.m_muteeId)),
- this.renoteMutingsRepository.createQueryBuilder('m')
- .select('m.muteeId')
- .where('m.muterId = :me', { me })
- .getRawMany<{ m_muteeId: string }>()
- .then(it => it.map(it => it.m_muteeId)),
- this.usersRepository.createQueryBuilder('u')
- .select(['u.id', 'u.host'])
- .where({ id: In(targets) } )
- .getRawMany<{ m_id: string, m_host: string }>()
- .then(it => it.reduce((map, it) => {
- map[it.m_id] = it.m_host;
- return map;
- }, {} as Record<string, string>)),
+ this.cacheService.userBlockedCache.fetch(me),
+ this.cacheService.userBlockingCache.fetch(me),
+ this.cacheService.userMutingsCache.fetch(me),
+ this.cacheService.renoteMutingsCache.fetch(me),
+ this.cacheService.getUsers(targets)
+ .then(users => {
+ const record: Record<string, string | null> = {};
+ for (const [id, user] of users) {
+ record[id] = user.host;
+ }
+ return record;
+ }),
this.userMemosRepository.createQueryBuilder('m')
.select(['m.targetUserId', 'm.memo'])
.where({ userId: me, targetUserId: In(targets) })
@@ -342,16 +294,13 @@ export class UserEntityService implements OnModuleInit {
map[it.m_targetUserId] = it.m_memo;
return map;
}, {} as Record<string, string | null>)),
- this.userProfilesRepository.createQueryBuilder('p')
- .select('p.mutedInstances')
- .where({ userId: me })
- .getRawOne<{ p_mutedInstances: string[] }>()
- .then(it => it?.p_mutedInstances ?? []),
+ this.cacheService.userProfileCache.fetch(me)
+ .then(p => p.mutedInstances),
]);
return new Map(
targets.map(target => {
- const following = followers.get(target) ?? null;
+ const following = myFollowing.get(target) ?? null;
return [
target,
@@ -359,14 +308,14 @@ export class UserEntityService implements OnModuleInit {
id: target,
following: following,
isFollowing: following != null,
- isFollowed: followees.includes(target),
+ isFollowed: myFollowers.has(target),
hasPendingFollowRequestFromYou: followersRequests.includes(target),
hasPendingFollowRequestToYou: followeesRequests.includes(target),
- isBlocking: blockers.includes(target),
- isBlocked: blockees.includes(target),
- isMuted: muters.includes(target),
- isRenoteMuted: renoteMuters.includes(target),
- isInstanceMuted: mutedInstances.includes(hosts[target]),
+ isBlocking: blockees.has(target),
+ isBlocked: blockers.has(target),
+ isMuted: muters.has(target),
+ isRenoteMuted: renoteMuters.has(target),
+ isInstanceMuted: hosts[target] != null && mutedInstances.includes(hosts[target]),
memo: memos[target] ?? null,
},
];
@@ -391,6 +340,7 @@ export class UserEntityService implements OnModuleInit {
return false; // TODO
}
+ // TODO optimization: make redis calls in MULTI
@bindThis
public async getNotificationsInfo(userId: MiUser['id']): Promise<{
hasUnread: boolean;
@@ -424,16 +374,14 @@ export class UserEntityService implements OnModuleInit {
@bindThis
public async getHasPendingReceivedFollowRequest(userId: MiUser['id']): Promise<boolean> {
- const count = await this.followRequestsRepository.countBy({
+ return await this.followRequestsRepository.existsBy({
followeeId: userId,
});
-
- return count > 0;
}
@bindThis
public async getHasPendingSentFollowRequest(userId: MiUser['id']): Promise<boolean> {
- return this.followRequestsRepository.existsBy({
+ return await this.followRequestsRepository.existsBy({
followerId: userId,
});
}
@@ -480,6 +428,10 @@ export class UserEntityService implements OnModuleInit {
userRelations?: Map<MiUser['id'], UserRelation>,
userMemos?: Map<MiUser['id'], string | null>,
pinNotes?: Map<MiUser['id'], MiUserNotePining[]>,
+ iAmModerator?: boolean,
+ userIdsByUri?: Map<string, string>,
+ instances?: Map<string, MiInstance | null>,
+ securityKeyCounts?: Map<string, number>,
},
): Promise<Packed<S>> {
const opts = Object.assign({
@@ -487,7 +439,10 @@ export class UserEntityService implements OnModuleInit {
includeSecrets: false,
}, options);
- const user = typeof src === 'object' ? src : await this.usersRepository.findOneByOrFail({ id: src });
+ const user = typeof src === 'object' ? src : await this.usersRepository.findOneOrFail({
+ where: { id: src },
+ relations: { userProfile: true },
+ });
// migration
if (user.avatarId != null && user.avatarUrl === null) {
@@ -518,10 +473,10 @@ export class UserEntityService implements OnModuleInit {
const isDetailed = opts.schema !== 'UserLite';
const meId = me ? me.id : null;
const isMe = meId === user.id;
- const iAmModerator = me ? await this.roleService.isModerator(me as MiUser) : false;
+ const iAmModerator = opts.iAmModerator ?? (me ? await this.roleService.isModerator(me as MiUser) : false);
const profile = isDetailed
- ? (opts.userProfile ?? await this.userProfilesRepository.findOneByOrFail({ userId: user.id }))
+ ? (opts.userProfile ?? user.userProfile ?? await this.userProfilesRepository.findOneByOrFail({ userId: user.id }))
: null;
let relation: UserRelation | null = null;
@@ -556,7 +511,7 @@ export class UserEntityService implements OnModuleInit {
}
}
- const mastoapi = !isDetailed ? opts.userProfile ?? await this.userProfilesRepository.findOneByOrFail({ userId: user.id }) : null;
+ const mastoapi = !isDetailed ? opts.userProfile ?? user.userProfile ?? await this.userProfilesRepository.findOneByOrFail({ userId: user.id }) : null;
const followingCount = profile == null ? null :
(profile.followingVisibility === 'public') || isMe || iAmModerator ? user.followingCount :
@@ -579,6 +534,9 @@ export class UserEntityService implements OnModuleInit {
const checkHost = user.host == null ? this.config.host : user.host;
const notificationsInfo = isMe && isDetailed ? await this.getNotificationsInfo(user.id) : null;
+ let fetchPoliciesPromise: Promise<RolePolicies> | null = null;
+ const fetchPolicies = () => fetchPoliciesPromise ??= this.roleService.getUserPolicies(user);
+
const packed = {
id: user.id,
name: user.name,
@@ -603,19 +561,21 @@ export class UserEntityService implements OnModuleInit {
enableRss: user.enableRss,
mandatoryCW: user.mandatoryCW,
rejectQuotes: user.rejectQuotes,
- isSilenced: user.isSilenced || this.roleService.getUserPolicies(user.id).then(r => !r.canPublicNote),
+ attributionDomains: user.attributionDomains,
+ isSilenced: user.isSilenced || fetchPolicies().then(r => !r.canPublicNote),
speakAsCat: user.speakAsCat ?? false,
approved: user.approved,
requireSigninToViewContents: user.requireSigninToViewContents === false ? undefined : true,
makeNotesFollowersOnlyBefore: user.makeNotesFollowersOnlyBefore ?? undefined,
makeNotesHiddenBefore: user.makeNotesHiddenBefore ?? undefined,
- instance: user.host ? this.federatedInstanceService.federatedInstanceCache.fetch(user.host).then(instance => instance ? {
+ instance: user.host ? Promise.resolve(opts.instances?.has(user.host) ? opts.instances.get(user.host) : this.federatedInstanceService.fetch(user.host)).then(instance => instance ? {
name: instance.name,
softwareName: instance.softwareName,
softwareVersion: instance.softwareVersion,
iconUrl: instance.iconUrl,
faviconUrl: instance.faviconUrl,
themeColor: instance.themeColor,
+ isSilenced: instance.isSilenced,
} : undefined) : undefined,
followersCount: followersCount ?? 0,
followingCount: followingCount ?? 0,
@@ -623,7 +583,7 @@ export class UserEntityService implements OnModuleInit {
emojis: this.customEmojiService.populateEmojis(user.emojis, checkHost),
onlineStatus: this.getOnlineStatus(user),
// パフォーマンス上の理由でローカルユーザーのみ
- badgeRoles: user.host == null ? this.roleService.getUserBadgeRoles(user.id).then((rs) => rs
+ badgeRoles: user.host == null ? this.roleService.getUserBadgeRoles(user).then((rs) => rs
.filter((r) => r.isPublic || iAmModerator)
.sort((a, b) => b.displayOrder - a.displayOrder)
.map((r) => ({
@@ -636,9 +596,9 @@ export class UserEntityService implements OnModuleInit {
...(isDetailed ? {
url: profile!.url,
uri: user.uri,
- movedTo: user.movedToUri ? this.apPersonService.resolvePerson(user.movedToUri).then(user => user.id).catch(() => null) : null,
+ movedTo: user.movedToUri ? Promise.resolve(opts.userIdsByUri?.get(user.movedToUri) ?? this.apPersonService.resolvePerson(user.movedToUri).then(user => user.id).catch(() => null)) : null,
alsoKnownAs: user.alsoKnownAs
- ? Promise.all(user.alsoKnownAs.map(uri => this.apPersonService.fetchPerson(uri).then(user => user?.id).catch(() => null)))
+ ? Promise.all(user.alsoKnownAs.map(uri => Promise.resolve(opts.userIdsByUri?.get(uri) ?? this.apPersonService.fetchPerson(uri).then(user => user?.id).catch(() => null))))
.then(xs => xs.length === 0 ? null : xs.filter(x => x != null))
: null,
updatedAt: user.updatedAt ? user.updatedAt.toISOString() : null,
@@ -665,8 +625,8 @@ export class UserEntityService implements OnModuleInit {
followersVisibility: profile!.followersVisibility,
followingVisibility: profile!.followingVisibility,
chatScope: user.chatScope,
- canChat: this.roleService.getUserPolicies(user.id).then(r => r.chatAvailability === 'available'),
- roles: this.roleService.getUserRoles(user.id).then(roles => roles.filter(role => role.isPublic).sort((a, b) => b.displayOrder - a.displayOrder).map(role => ({
+ canChat: fetchPolicies().then(r => r.chatAvailability === 'available'),
+ roles: this.roleService.getUserRoles(user).then(roles => roles.filter(role => role.isPublic).sort((a, b) => b.displayOrder - a.displayOrder).map(role => ({
id: role.id,
name: role.name,
color: role.color,
@@ -684,7 +644,7 @@ export class UserEntityService implements OnModuleInit {
twoFactorEnabled: profile!.twoFactorEnabled,
usePasswordLessLogin: profile!.usePasswordLessLogin,
securityKeys: profile!.twoFactorEnabled
- ? this.userSecurityKeysRepository.countBy({ userId: user.id }).then(result => result >= 1)
+ ? Promise.resolve(opts.securityKeyCounts?.get(user.id) ?? this.userSecurityKeysRepository.countBy({ userId: user.id })).then(result => result >= 1)
: false,
} : {}),
@@ -728,7 +688,7 @@ export class UserEntityService implements OnModuleInit {
emailNotificationTypes: profile!.emailNotificationTypes,
achievements: profile!.achievements,
loggedInDays: profile!.loggedInDates.length,
- policies: this.roleService.getUserPolicies(user.id),
+ policies: fetchPolicies(),
defaultCW: profile!.defaultCW,
defaultCWPriority: profile!.defaultCWPriority,
allowUnsignedFetch: user.allowUnsignedFetch,
@@ -778,57 +738,103 @@ export class UserEntityService implements OnModuleInit {
includeSecrets?: boolean,
},
): Promise<Packed<S>[]> {
+ if (users.length === 0) return [];
+
// -- IDのみの要素を補完して完全なエンティティ一覧を作る
const _users = users.filter((user): user is MiUser => typeof user !== 'string');
if (_users.length !== users.length) {
_users.push(
- ...await this.usersRepository.findBy({
- id: In(users.filter((user): user is string => typeof user === 'string')),
+ ...await this.usersRepository.find({
+ where: {
+ id: In(users.filter((user): user is string => typeof user === 'string')),
+ },
+ relations: {
+ userProfile: true,
+ },
}),
);
}
const _userIds = _users.map(u => u.id);
- // -- 実行者の有無や指定スキーマの種別によって要否が異なる値群を取得
-
- let profilesMap: Map<MiUser['id'], MiUserProfile> = new Map();
- let userRelations: Map<MiUser['id'], UserRelation> = new Map();
- let userMemos: Map<MiUser['id'], string | null> = new Map();
- let pinNotes: Map<MiUser['id'], MiUserNotePining[]> = new Map();
+ const iAmModerator = await this.roleService.isModerator(me as MiUser);
+ const meId = me ? me.id : null;
+ const isDetailed = options && options.schema !== 'UserLite';
+ const isDetailedAndMod = isDetailed && iAmModerator;
- if (options?.schema !== 'UserLite') {
- profilesMap = await this.userProfilesRepository.findBy({ userId: In(_userIds) })
- .then(profiles => new Map(profiles.map(p => [p.userId, p])));
+ const userUris = new Set(_users
+ .flatMap(user => [user.uri, user.movedToUri])
+ .filter((uri): uri is string => uri != null));
- const meId = me ? me.id : null;
- if (meId) {
- userMemos = await this.userMemosRepository.findBy({ userId: meId })
- .then(memos => new Map(memos.map(memo => [memo.targetUserId, memo.memo])));
+ const userHosts = new Set(_users
+ .map(user => user.host)
+ .filter((host): host is string => host != null));
- if (_userIds.length > 0) {
- userRelations = await this.getRelations(meId, _userIds);
- pinNotes = await this.userNotePiningsRepository.createQueryBuilder('pin')
- .where('pin.userId IN (:...userIds)', { userIds: _userIds })
- .innerJoinAndSelect('pin.note', 'note')
- .getMany()
- .then(pinsNotes => {
- const map = new Map<MiUser['id'], MiUserNotePining[]>();
- for (const note of pinsNotes) {
- const notes = map.get(note.userId) ?? [];
- notes.push(note);
- map.set(note.userId, notes);
- }
- for (const [, notes] of map.entries()) {
- // pack側ではDESCで取得しているので、それに合わせて降順に並び替えておく
- notes.sort((a, b) => b.id.localeCompare(a.id));
- }
- return map;
- });
- }
+ const _profilesFromUsers: [string, MiUserProfile][] = [];
+ const _profilesToFetch: string[] = [];
+ for (const user of _users) {
+ if (user.userProfile) {
+ _profilesFromUsers.push([user.id, user.userProfile]);
+ } else {
+ _profilesToFetch.push(user.id);
}
}
+ // -- 実行者の有無や指定スキーマの種別によって要否が異なる値群を取得
+
+ const [profilesMap, userMemos, userRelations, pinNotes, userIdsByUri, instances, securityKeyCounts] = await Promise.all([
+ // profilesMap
+ this.cacheService.userProfileCache.fetchMany(_profilesToFetch).then(profiles => new Map(profiles.concat(_profilesFromUsers))),
+ // userMemos
+ isDetailed && meId ? this.userMemosRepository.findBy({ userId: meId })
+ .then(memos => new Map(memos.map(memo => [memo.targetUserId, memo.memo]))) : new Map(),
+ // userRelations
+ isDetailed && meId ? this.getRelations(meId, _userIds) : new Map(),
+ // pinNotes
+ isDetailed ? this.userNotePiningsRepository.createQueryBuilder('pin')
+ .where('pin.userId IN (:...userIds)', { userIds: _userIds })
+ .innerJoinAndSelect('pin.note', 'note')
+ .getMany()
+ .then(pinsNotes => {
+ const map = new Map<MiUser['id'], MiUserNotePining[]>();
+ for (const note of pinsNotes) {
+ const notes = map.get(note.userId) ?? [];
+ notes.push(note);
+ map.set(note.userId, notes);
+ }
+ for (const [, notes] of map.entries()) {
+ // pack側ではDESCで取得しているので、それに合わせて降順に並び替えておく
+ notes.sort((a, b) => b.id.localeCompare(a.id));
+ }
+ return map;
+ }) : new Map(),
+ // userIdsByUrl
+ isDetailed ? this.usersRepository.createQueryBuilder('user')
+ .select([
+ 'user.id',
+ 'user.uri',
+ ])
+ .where({
+ uri: In(Array.from(userUris)),
+ })
+ .getRawMany<{ user_uri: string, user_id: string }>()
+ .then(users => new Map(users.map(u => [u.user_uri, u.user_id]))) : new Map(),
+ // instances
+ Promise.all(Array.from(userHosts).map(async host => [host, await this.federatedInstanceService.fetch(host)] as const))
+ .then(hosts => new Map(hosts)),
+ // securityKeyCounts
+ isDetailedAndMod ? this.userSecurityKeysRepository.createQueryBuilder('key')
+ .select('key.userId', 'userId')
+ .addSelect('count(key.id)', 'userCount')
+ .where({
+ userId: In(_userIds),
+ })
+ .groupBy('key.userId')
+ .getRawMany<{ userId: string, userCount: number }>()
+ .then(counts => new Map(counts.map(c => [c.userId, c.userCount])))
+ : undefined, // .pack will fetch the keys for the requesting user if it's in the _userIds
+ ]);
+
return Promise.all(
_users.map(u => this.pack(
u,
@@ -839,6 +845,10 @@ export class UserEntityService implements OnModuleInit {
userRelations: userRelations,
userMemos: userMemos,
pinNotes: pinNotes,
+ iAmModerator,
+ userIdsByUri,
+ instances,
+ securityKeyCounts,
},
)),
);