fix: make res.statusCode optional (#4471)

* fix: make res.statusCode optional * test: add tests Co-authored-by: Поляков Владислав Михайлович <VMikPolyakov@sber.ru>
fastify · Dec 19, 2022 · 35e7c1a · 35e7c1a
1 parent f0455e7
commit 35e7c1a
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/test/types/logger.test-d.ts b/test/types/logger.test-d.ts
@@ -123,6 +123,21 @@ const serverAutoInferredFileOption = fastify({
 
 expectType<FastifyBaseLogger>(serverAutoInferredFileOption.log)
 
+const serverAutoInferredSerializerResponseObjectOption = fastify({
+  logger: {
+    serializers: {
+      res (ServerResponse) {
+        expectType<FastifyReply>(ServerResponse)
+        return {
+          status: '200'
+        }
+      }
+    }
+  }
+})
+
+expectType<FastifyBaseLogger>(serverAutoInferredSerializerResponseObjectOption.log)
+
 const serverAutoInferredSerializerObjectOption = fastify({
   logger: {
     serializers: {

diff --git a/types/logger.d.ts b/types/logger.d.ts
@@ -60,7 +60,7 @@ export interface FastifyLoggerOptions<
       [key: string]: unknown;
     };
     res?: (res: RawReply) => {
-      statusCode: string | number;
+      statusCode?: string | number;
       [key: string]: unknown;
     };
   };