Enforce n - 1 deletion requirement server side
This commit is contained in:
parent
ba4932958c
commit
bd50acfe1d
@ -2,8 +2,12 @@ import { jsonError } from "../../../common.js";
|
||||
|
||||
export async function onRequestDelete(context: RequestContext) {
|
||||
const eventId = context.params.id as string;
|
||||
const eventData = await context.env.D1.prepare(
|
||||
"SELECT created_by FROM events WHERE id = ?;",
|
||||
const eventData:
|
||||
| ({
|
||||
[k: string]: number;
|
||||
} & { created_by: string })
|
||||
| null = await context.env.D1.prepare(
|
||||
"SELECT created_by, day, month, year FROM events WHERE id = ?;",
|
||||
)
|
||||
.bind(eventId)
|
||||
.first();
|
||||
@ -11,14 +15,26 @@ export async function onRequestDelete(context: RequestContext) {
|
||||
if (!eventData) return jsonError("No event exists with that ID", 404);
|
||||
|
||||
const { current_user: currentUser } = context.data;
|
||||
const isETM = [1 << 4, 1 << 12].find((int) => currentUser.permissions & int);
|
||||
|
||||
if (
|
||||
eventData.created_by !== currentUser.id &&
|
||||
![1 << 4, 1 << 12].find((int) => currentUser.permissions & int)
|
||||
)
|
||||
if (eventData.created_by !== currentUser.id && !isETM)
|
||||
return jsonError("You are not authorized to delete that event", 403);
|
||||
|
||||
await context.env.DATA.delete(`event_${eventId}`);
|
||||
const now = new Date();
|
||||
now.setUTCHours(0, 0, 0, 0);
|
||||
|
||||
const eventDate = new Date(
|
||||
eventData.year,
|
||||
eventData.month - 1,
|
||||
eventData.day,
|
||||
);
|
||||
|
||||
if (!isETM && now.getTime() <= eventDate.getTime())
|
||||
return jsonError(
|
||||
"Event cannot be deleted on or after the scheduled date",
|
||||
403,
|
||||
);
|
||||
|
||||
await context.env.D1.prepare("DELETE FROM events WHERE id = ?;")
|
||||
.bind(eventId)
|
||||
.run();
|
||||
|
Loading…
x
Reference in New Issue
Block a user