fix!: jsonify req.body

Coobaha · Oct 26, 2023 · 636b343 · 636b343
1 parent a7dd1b9
commit 636b343
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/typed-fastify.ts b/src/typed-fastify.ts
@@ -350,7 +350,7 @@ interface Request<
   readonly operationPath: Path;
   readonly method: ROptions['method'];
   // A payload within a GET request message has no defined semantics; sending a payload body on a GET request might cause some existing implementations to reject the request.
-  readonly body: ROptions['method'] extends 'GET' ? never : Get<Op['request'], 'body'>;
+  readonly body: ROptions['method'] extends 'GET' ? never : Jsonify<Get<Op['request'], 'body'>>;
   readonly routeOptions: Id<Readonly<ROptions>>;
   readonly routerMethod: ROptions['method'];
   readonly headers: Get<Op['request'], 'headers'>;

diff --git a/test/fixtures.ts b/test/fixtures.ts
@@ -30,7 +30,8 @@ export const defaultService: Service<TestSchema> = {
   },
   'POST /jsonify': (req, reply) => {
     const { date } = req.body;
-    return reply.status(200).send({ date: date.toJSON() });
+    date.charAt; // ok for string, not ok for Date
+    return reply.status(200).send({ date: new Date(date).toJSON() });
   },
   'POST /': (req, reply) => {
     if (req.operationPath !== 'POST /') {