fix: coalesce falsy body to {} (#4929)

server/middlewares/coaleseBody.ts

@@ -0,0 +1,12 @@
+import { Next } from "koa";
+import { AppContext } from "@server/types";
+
+// remove after https://github.com/koajs/koa-body/issues/218 is resolved
+export default function coalesceBody() {
+  return function coalesceBodyMiddleware(ctx: AppContext, next: Next) {
+    if (!ctx.request.body) {
+      ctx.request.body = {};
+    }
+    return next();
+  };
+}

server/routes/api/index.ts

@@ -7,6 +7,7 @@ import userAgent, { UserAgentContext } from "koa-useragent";
 import env from "@server/env";
 import { NotFoundError } from "@server/errors";
 import Logger from "@server/logging/Logger";
+import coalesceBody from "@server/middlewares/coaleseBody";
 import { AppState, AppContext } from "@server/types";
 import apiKeys from "./apiKeys";
 import attachments from "./attachments";
@@ -45,6 +46,7 @@ api.use(
     },
   })
 );
+api.use(coalesceBody());
 api.use<BaseContext, UserAgentContext>(userAgent);
 api.use(apiWrapper());
 api.use(editor());

server/routes/auth/index.ts

@@ -5,6 +5,7 @@ import bodyParser from "koa-body";
 import Router from "koa-router";
 import { AuthenticationError } from "@server/errors";
 import auth from "@server/middlewares/authentication";
+import coalesceBody from "@server/middlewares/coaleseBody";
 import { Collection, Team, View } from "@server/models";
 import AuthenticationHelper from "@server/models/helpers/AuthenticationHelper";
 import { AppState, AppContext, APIContext } from "@server/types";
@@ -71,6 +72,7 @@ router.get("/redirect", auth(), async (ctx: APIContext) => {
 });
 
 app.use(bodyParser());
+app.use(coalesceBody());
 app.use(router.routes());
 
 export default app;