@bymax-one/nest-logger 1.4.7 → 1.4.8

package/CHANGELOG.md

@@ -11,6 +11,33 @@ heading here.
 
 ## [Unreleased]
 
+## [1.4.8] - 2026-11-01
+
+### Fixed
+
+- **A logging call no longer throws because of what it was handed.** `error`, `warn`, `log`, `fatal`
+  and the other NestJS variadic methods threw out of the call when an `Error` had a `name` or `message`
+  getter that throws, and when the message had no text form (a null-prototype object, a throwing
+  `toString`). In `catch (e) { logger.error(e) }` that second throw replaced the original error and
+  nothing was logged. The call now returns and the entry is written: an unreadable `Error` arrives as
+  the same `err` node `errorStructured` produces, with `err.type: 'SanitizeFailed'`, and a message with
+  no text form is logged as `Unreadable log message`.
+
+#### Apply to a derived backend
+
+Nothing to change in code. What can differ at run time:
+
+- Entries that were never written now appear. A host that handled the throw (a `try` around the logging
+  call, or an error handler that reacted to it) no longer sees it; one that relied on the throw to detect
+  a hostile input should look for `err.type: 'SanitizeFailed'` (an `Error`) or the message
+  `Unreadable log message` (a message) instead.
+
+### Tests
+
+- Each variadic position with an unreadable `name` or `message`, and each message with no text form on
+  `error`, `warn` and `log`, has a failing-first spec through the real Pino pipeline, with a control that
+  a readable error is logged as before.
+
 ## [1.4.7] - 2026-11-01
 
 ### Fixed
@@ -2311,7 +2338,8 @@ published `dist/` is identical — no runtime behaviour changes for consumers.
 - Professional CI suite: `ci.yml`, `bench.yml`, `codeql.yml`, `scorecard.yml`,
   `release.yml`, Dependabot, and issue templates
 
-[Unreleased]: https://github.com/bymaxone/nest-logger/compare/v1.4.7...HEAD
+[Unreleased]: https://github.com/bymaxone/nest-logger/compare/v1.4.8...HEAD
+[1.4.8]: https://github.com/bymaxone/nest-logger/compare/v1.4.7...v1.4.8
 [1.4.7]: https://github.com/bymaxone/nest-logger/compare/v1.4.6...v1.4.7
 [1.4.6]: https://github.com/bymaxone/nest-logger/compare/v1.4.5...v1.4.6
 [1.4.5]: https://github.com/bymaxone/nest-logger/compare/v1.4.4...v1.4.5

package/dist/server/index.cjs

@@ -386,15 +386,29 @@ function sanitizeError(err, options) {
   return toSanitizedError(err, /* @__PURE__ */ new WeakSet(), 0, maxCauseDepth);
 }
 function isErrorLike(value) {
+  try {
+    return readsAsError(value);
+  } catch {
+    return isErrorInstance(value);
+  }
+}
+function readsAsError(value) {
   if (typeof value !== "object" || value === null) {
     return false;
   }
   const candidate = value;
   return typeof candidate.name === "string" && typeof candidate.message === "string";
 }
+function isErrorInstance(value) {
+  try {
+    return value instanceof Error;
+  } catch {
+    return false;
+  }
+}
 function toSanitizedError(value, seen, depth2, maxDepth) {
   try {
-    if (!isErrorLike(value)) {
+    if (!readsAsError(value)) {
       return { name: "UnknownError", message: String(value) };
     }
     seen.add(value);
@@ -500,6 +514,13 @@ function safeErrorMessage(error) {
     return "Unreadable error message";
   }
 }
+function safeLogMessage(message) {
+  try {
+    return toSingleLineMessage(typeof message === "string" ? message : String(message));
+  } catch {
+    return "Unreadable log message";
+  }
+}
 function findErrorParam(optionalParams) {
   return optionalParams.find((param) => isErrorLike(param));
 }
@@ -571,10 +592,7 @@ exports.PinoLoggerService = class PinoLoggerService {
     assignIfDefined(payload, "context", context);
     assignIfDefined(payload, "stack", stack);
     assignIfDefined(payload, "err", findErrorParam(optionalParams));
-    this.pino.error(
-      payload,
-      toSingleLineMessage(typeof message === "string" ? message : String(message))
-    );
+    this.pino.error(payload, safeLogMessage(message));
   }
   /**
    * NestJS `warn` (warn level).
@@ -731,7 +749,7 @@ exports.PinoLoggerService = class PinoLoggerService {
     assignIfDefined(payload, "context", this.resolveContext(optionalParams));
     const leading = isErrorLike(message) ? message : void 0;
     assignIfDefined(payload, "err", leading ?? findErrorParam(optionalParams));
-    const msg = leading === void 0 ? toSingleLineMessage(typeof message === "string" ? message : String(message)) : safeErrorMessage(leading);
+    const msg = leading === void 0 ? safeLogMessage(message) : safeErrorMessage(leading);
     switch (level) {
       case "info":
         this.pino.info(payload, msg);
@@ -3218,11 +3236,7 @@ exports.PinoUnexpectedErrorSink = __decorateClass([
   __decorateParam(0, common.Inject(exports.PinoLoggerService))
 ], exports.PinoUnexpectedErrorSink);
 function toLoggableError(value) {
-  try {
-    return isErrorLike(value) ? value : sanitizeError(value);
-  } catch {
-    return sanitizeError(value);
-  }
+  return isErrorLike(value) ? value : sanitizeError(value);
 }
 var INJECT_LOGGER_CONTEXT_METADATA_KEY = "bymax_logger:context";
 function InjectLogger(context) {

package/dist/server/index.mjs

@@ -383,15 +383,29 @@ function sanitizeError(err, options) {
   return toSanitizedError(err, /* @__PURE__ */ new WeakSet(), 0, maxCauseDepth);
 }
 function isErrorLike(value) {
+  try {
+    return readsAsError(value);
+  } catch {
+    return isErrorInstance(value);
+  }
+}
+function readsAsError(value) {
   if (typeof value !== "object" || value === null) {
     return false;
   }
   const candidate = value;
   return typeof candidate.name === "string" && typeof candidate.message === "string";
 }
+function isErrorInstance(value) {
+  try {
+    return value instanceof Error;
+  } catch {
+    return false;
+  }
+}
 function toSanitizedError(value, seen, depth2, maxDepth) {
   try {
-    if (!isErrorLike(value)) {
+    if (!readsAsError(value)) {
       return { name: "UnknownError", message: String(value) };
     }
     seen.add(value);
@@ -497,6 +511,13 @@ function safeErrorMessage(error) {
     return "Unreadable error message";
   }
 }
+function safeLogMessage(message) {
+  try {
+    return toSingleLineMessage(typeof message === "string" ? message : String(message));
+  } catch {
+    return "Unreadable log message";
+  }
+}
 function findErrorParam(optionalParams) {
   return optionalParams.find((param) => isErrorLike(param));
 }
@@ -568,10 +589,7 @@ var PinoLoggerService = class {
     assignIfDefined(payload, "context", context);
     assignIfDefined(payload, "stack", stack);
     assignIfDefined(payload, "err", findErrorParam(optionalParams));
-    this.pino.error(
-      payload,
-      toSingleLineMessage(typeof message === "string" ? message : String(message))
-    );
+    this.pino.error(payload, safeLogMessage(message));
   }
   /**
    * NestJS `warn` (warn level).
@@ -728,7 +746,7 @@ var PinoLoggerService = class {
     assignIfDefined(payload, "context", this.resolveContext(optionalParams));
     const leading = isErrorLike(message) ? message : void 0;
     assignIfDefined(payload, "err", leading ?? findErrorParam(optionalParams));
-    const msg = leading === void 0 ? toSingleLineMessage(typeof message === "string" ? message : String(message)) : safeErrorMessage(leading);
+    const msg = leading === void 0 ? safeLogMessage(message) : safeErrorMessage(leading);
     switch (level) {
       case "info":
         this.pino.info(payload, msg);
@@ -3215,11 +3233,7 @@ PinoUnexpectedErrorSink = __decorateClass([
   __decorateParam(0, Inject(PinoLoggerService))
 ], PinoUnexpectedErrorSink);
 function toLoggableError(value) {
-  try {
-    return isErrorLike(value) ? value : sanitizeError(value);
-  } catch {
-    return sanitizeError(value);
-  }
+  return isErrorLike(value) ? value : sanitizeError(value);
 }
 var INJECT_LOGGER_CONTEXT_METADATA_KEY = "bymax_logger:context";
 function InjectLogger(context) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bymax-one/nest-logger",
-  "version": "1.4.7",
+  "version": "1.4.8",
   "description": "Structured JSON logging for NestJS based on Pino 10, with optional OpenTelemetry correlation.",
   "author": "Bymax One <support@bymax.one>",
   "license": "MIT",