Fix customSuccessMessage ts definition (#270)

* fix customSuccessMessage ts definition * add tsd for customSuccessMessage
pinojs · Jan 30, 2023 · 62c8e7d · 62c8e7d
1 parent 81bb437
commit 62c8e7d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/import.test-d.ts b/import.test-d.ts
@@ -11,6 +11,11 @@ const { pinoHttp: pinoHttpCjsNamed } = require('.');
 const logger = pino();
 
 expectType<HttpLogger>(pinoHttp({ logger }));
+expectType<HttpLogger>(pinoHttp({
+  customSuccessMessage(req, res, responseTime) {
+    return `${responseTime}`
+  }
+}));
 expectType<HttpLogger>(pinoHttpNamed());
 expectType<HttpLogger>(pinoHttpStar.default());
 expectType<HttpLogger>(pinoHttpStar.pinoHttp());

diff --git a/index.d.ts b/index.d.ts
@@ -30,7 +30,7 @@ export interface Options extends pino.LoggerOptions {
     autoLogging?: boolean | AutoLoggingOptions | undefined;
     customLogLevel?: ((req: IncomingMessage, res: ServerResponse, error?: Error) => pino.LevelWithSilent) | undefined;
     customReceivedMessage?: ((req: IncomingMessage, res: ServerResponse) => string) | undefined;
-    customSuccessMessage?: ((req: IncomingMessage, res: ServerResponse) => string) | undefined;
+    customSuccessMessage?: ((req: IncomingMessage, res: ServerResponse, responseTime: number) => string) | undefined;
     customErrorMessage?: ((req: IncomingMessage, res: ServerResponse, error: Error) => string) | undefined;
     customReceivedObject?: ((req: IncomingMessage, res: ServerResponse, val?: any) => any) | undefined;
     customSuccessObject?: ((req: IncomingMessage, res: ServerResponse, val: any) => any) | undefined;