TheCloud/PronounsPage
1
0
Fork 0
mirror of https://gitlab.com/PronounsPage/PronounsPage.git synced 2025-09-27 23:13:01 -04:00
Code Issues Packages Projects Releases Wiki Activity

[perf] remove n+1 on /admin/list

Browse Source
This commit is contained in:
Andrea Vos 2021-11-23 20:50:14 +01:00
parent 133d8c4dbc
commit 4c48e886ca
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
server/avatar.js
View File

@ -7,6 +7,9 @@ export default async (db, user) => {
} else if (user.avatarSource === 'gravatar') {
return gravatar(user);
} else if (user.avatarSource) {
if (user.payload) {
return JSON.parse(user.payload).avatar;
}
const auth = await db.get(SQL`
SELECT payload FROM authenticators
WHERE type = ${user.avatarSource}

6
server/routes/admin.js
View File

@ -13,9 +13,10 @@ const router = Router();
router.get('/admin/list', handleErrorAsync(async (req, res) => {
return res.json(await caches.admins.fetch(async () => {
const admins = await req.db.all(SQL`
SELECT u.username, p.teamName, p.locale, u.id, u.email, u.avatarSource, p.credentials, p.credentialsLevel, p.credentialsName
SELECT u.username, p.teamName, p.locale, u.id, u.email, u.avatarSource, p.credentials, p.credentialsLevel, p.credentialsName, a.payload
FROM users u
LEFT JOIN profiles p ON p.userId = u.id
LEFT JOIN profiles p ON p.userId = u.id
LEFT JOIN authenticators a ON u.id = a.userId AND a.type = u.avatarSource AND (a.validUntil IS NULL OR a.validUntil > ${now()})
WHERE p.teamName IS NOT NULL
AND p.teamName != ''
ORDER BY RANDOM()
@ -34,6 +35,7 @@ router.get('/admin/list', handleErrorAsync(async (req, res) => {
admin.avatar = await avatar(req.db, admin);
delete admin.id;
delete admin.email;
delete admin.payload;
if (admin.credentials) {
admin.credentials = admin.credentials.split('|');
}