change setBindings to clearBindings (#23)

maou-shonen · Oct 20, 2024 · 3e9595b · 3e9595b
1 parent 6ee75f1
commit 3e9595b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/logger.test.ts b/src/logger.test.ts
@@ -50,13 +50,13 @@ describe("logger", () => {
     expect(logger.bindings()).toStrictEqual({ foo: "bar" });
   });
 
-  it("set bindings", () => {
-    logger.setBindings({ foo: "bar" });
+  it("clear bindings", () => {
+    logger.assign({ foo: "bar" });
+    logger.clearBindings();
     logger.info("foo");
     expect(logs[0]).toStrictEqual({
       msg: "foo",
       level: 30,
-      foo: "bar",
     });
   });
 

diff --git a/src/logger.ts b/src/logger.ts
@@ -30,10 +30,10 @@ export class PinoLogger {
   }
 
   /**
-   * Set bindings to http log context
+   * Clear bindings from http log context
    */
-  setBindings(bindings: pino.Bindings) {
-    this._logger = this.#rootLogger.child(bindings);
+  clearBindings() {
+    this._logger = this.#rootLogger.child({});
   }
 
   /**