fix: Pino options have a generic type (#304)

pinojs · Oct 28, 2023 · f19ed45 · f19ed45
1 parent 9bf0802
commit f19ed45
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/index.d.ts b/index.d.ts
@@ -12,13 +12,13 @@ import { IncomingMessage, ServerResponse } from 'http';
 import pino from 'pino';
 import { err, req, res, SerializedError, SerializedRequest, SerializedResponse } from 'pino-std-serializers';
 
-declare function PinoHttp<IM = IncomingMessage, SR = ServerResponse>(opts?: Options<IM, SR>, stream?: pino.DestinationStream): HttpLogger<IM, SR>;
+declare function PinoHttp<IM = IncomingMessage, SR = ServerResponse, Opts = Options<IM, SR>>(opts?: Opts, stream?: pino.DestinationStream): HttpLogger<IM, SR, Opts>;
 
 declare function PinoHttp<IM = IncomingMessage, SR = ServerResponse>(stream?: pino.DestinationStream): HttpLogger<IM, SR>;
 
-export interface HttpLogger<IM = IncomingMessage, SR = ServerResponse> {
+export interface HttpLogger<IM = IncomingMessage, SR = ServerResponse, Opts = Options<IM, SR>> {
     (req: IM, res: SR, next?: () => void): void;
-    logger: pino.Logger;
+    logger: pino.Logger<Opts>;
 }
 export type ReqId = number | string | object;
 

diff --git a/index.test-d.ts b/index.test-d.ts
@@ -205,3 +205,11 @@ const httpServerListener: RequestListener = (request, response) => {
   request.allLogs[0].info("allLogs available on response");
   response.end("Hello world");
 };
+
+// custom levels added in the options should be available
+// on the logger returned by pino-http
+pinoHttp({
+    customLevels: {
+        bark: 25,
+    }
+}).logger.bark("arf arf");