fix incorrect type definition of censor function (#1389)

* fix incorrect type definition of censor function `censor` takes two arguments: value and path * add assertions for censor function censor function should take two arguments: value and path
pinojs · Apr 8, 2022 · 47e97f7 · 47e97f7
1 parent 75f0dc7
commit 47e97f7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/pino.d.ts b/pino.d.ts
@@ -38,7 +38,7 @@ type CustomLevelLogger<Options> = Options extends { customLevels: Record<string,
 
 interface redactOptions {
     paths: string[];
-    censor?: string | ((v: any) => any);
+    censor?: string | ((value: any, path: string[]) => any);
     remove?: boolean;
 }
 

diff --git a/test/types/pino.test-d.ts b/test/types/pino.test-d.ts
@@ -63,6 +63,14 @@ pino({
     redact: { paths: [], censor: () => "SECRET" },
 });
 
+pino({
+    redact: { paths: [], censor: (value) => value },
+});
+
+pino({
+    redact: { paths: [], censor: (value, path) => path.join() },
+});
+
 pino({
     depthLimit: 1
 });