Ryan/drop
1
0
Fork 0
mirror of https://github.com/Drop-OSS/drop.git synced 2025-11-10 04:22:09 +10:00
Code Issues Packages Projects Releases Wiki Activity

fix: remove old requiredPerms field

Browse Source
This commit is contained in:
Huskydog9988
2025-05-15 14:21:12 -04:00
parent 59c3b9b76e
commit 831b20d737
3 changed files with 1 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
server/api/v1/notifications/index.get.ts
View File

@ -1,4 +1,4 @@
import aclManager, { type SystemACL } from "~/server/internal/acls";
import aclManager from "~/server/internal/acls";
import prisma from "~/server/internal/db/database";
export default defineEventHandler(async (h3) => {
@ -24,36 +24,5 @@ export default defineEventHandler(async (h3) => {
},
});
let i = notifications.length;
while (i--) {
const notif = notifications[i];
const hasPermsForNotif = await aclManager.allowSystemACL(
h3,
notif.requiredPerms as SystemACL,
);
if (!hasPermsForNotif) {
// remove element
console.log(
userId,
"did not have perms to access",
notif.id,
"based on",
notif.requiredPerms,
);
notifications.splice(i, 1);
} else {
console.log(
userId,
"had perms to access",
notif.id,
"based on",
notif.requiredPerms,
);
}
}
return notifications;
});