fix: Multi-method handlers, regressed in methodOverride removal

server/routes/api/attachments.ts

@@ -147,7 +147,7 @@ router.post("attachments.delete", auth(), async (ctx) => {
 });
 
 const handleAttachmentsRedirect = async (ctx: ContextWithState) => {
-  const { id } = (ctx.request.body ?? ctx.request.query) as { id?: string };
+  const id = ctx.request.body?.id ?? ctx.request.query?.id;
   assertUuid(id, "id is required");
 
   const { user } = ctx.state;

server/routes/api/cron.ts

@@ -12,7 +12,9 @@ import InviteReminderTask from "@server/queues/tasks/InviteReminderTask";
 const router = new Router();
 
 const cronHandler = async (ctx: Context) => {
-  const { token, limit = 500 } = (ctx.request.body ?? ctx.request.query) as {
+  const { token, limit = 500 } = (ctx.method === "POST"
+    ? ctx.request.body
+    : ctx.request.query) as {
     token?: string;
     limit: number;
   };

server/routes/api/fileOperations.ts

@@ -70,7 +70,7 @@ router.post(
 );
 
 const handleFileOperationsRedirect = async (ctx: ContextWithState) => {
-  const { id } = (ctx.request.body ?? ctx.request.query) as { id?: string };
+  const id = ctx.request.body?.id ?? ctx.request.query?.id;
   assertUuid(id, "id is required");
 
   const { user } = ctx.state;

server/routes/api/notificationSettings.ts

@@ -57,7 +57,9 @@ router.post("notificationSettings.delete", auth(), async (ctx) => {
 });
 
 const handleUnsubscribe = async (ctx: ContextWithState) => {
-  const { id, token } = (ctx.request.body ?? ctx.request.query) as {
+  const { id, token } = (ctx.method === "POST"
+    ? ctx.request.body
+    : ctx.request.query) as {
     id?: string;
     token?: string;
   };