@naturalcycles/backend-lib 4.14.0 → 4.14.2

package/dist/sentry/sentry.shared.service.js

@@ -81,14 +81,11 @@ class SentrySharedService {
         // It's a Breadcrumb, not a console.log, because console.log are NOT automatically attached as Breadcrumbs in cron-job environments (outside of Express)
         this.sentry().addBreadcrumb({
             message: (0, nodejs_lib_1.inspectAny)(err, {
-                includeErrorData: true,
                 colors: false,
             }),
             // data: (err as AppError).data, // included in message
         });
-        return this.sentry().captureException((0, js_lib_1._anyToError)(err, Error, {}, {
-            stringifyFn: nodejs_lib_1.inspectAnyStringifyFn,
-        }));
+        return this.sentry().captureException((0, js_lib_1._anyToError)(err));
     }
     /**
      * Returns "eventId"
@@ -114,16 +111,13 @@ class SentrySharedService {
             error: (...args) => {
                 const message = args
                     .map(arg => (0, nodejs_lib_1.inspectAny)(arg, {
-                    includeErrorData: true,
                     colors: false,
                 }))
                     .join(' ');
                 this.sentry().addBreadcrumb({
                     message,
                 });
-                this.sentry().captureException((0, js_lib_1._anyToError)(args.length === 1 ? args[0] : args, Error, {}, {
-                    stringifyFn: nodejs_lib_1.inspectAnyStringifyFn,
-                }));
+                this.sentry().captureException((0, js_lib_1._anyToError)(args.length === 1 ? args[0] : args));
             },
         };
     }

package/dist/server/genericErrorMiddleware.js

@@ -2,7 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.respondWithError = exports.genericErrorMiddleware = void 0;
 const js_lib_1 = require("@naturalcycles/js-lib");
-const nodejs_lib_1 = require("@naturalcycles/nodejs-lib");
 const { APP_ENV } = process.env;
 const includeErrorStack = APP_ENV !== 'prod' && APP_ENV !== 'test';
 // Hacky way to store the sentryService, so it's available to `respondWithError` function
@@ -42,16 +41,16 @@ function respondWithError(req, res, err) {
     else {
         req.error(err);
     }
-    const originalError = (0, js_lib_1._anyToError)(err, Error, {}, {
-        stringifyFn: nodejs_lib_1.inspectAnyStringifyFn,
-    });
+    const originalError = (0, js_lib_1._anyToError)(err);
     let errorId;
     if (sentryService && shouldReportToSentry(originalError)) {
         errorId = sentryService.captureException(originalError, false);
     }
     if (res.headersSent)
         return;
-    const httpError = (0, js_lib_1._errorToErrorObject)(originalError, includeErrorStack);
+    const httpError = (0, js_lib_1._errorToErrorObject)(originalError);
+    if (!includeErrorStack)
+        delete httpError.stack;
     httpError.data.errorId = errorId;
     httpError.data.httpStatusCode ||= 500; // default to 500
     httpError.data.headersSent = headersSent || undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@naturalcycles/backend-lib",
-  "version": "4.14.0",
+  "version": "4.14.2",
   "scripts": {
     "prepare": "husky install && patch-package",
     "serve": "APP_ENV=dev nodemon",

package/src/sentry/sentry.shared.service.ts

@@ -1,5 +1,5 @@
 import { _anyToError, _Memo, CommonLogger, CommonLogLevel } from '@naturalcycles/js-lib'
-import { inspectAny, inspectAnyStringifyFn } from '@naturalcycles/nodejs-lib'
+import { inspectAny } from '@naturalcycles/nodejs-lib'
 import type { Breadcrumb, NodeOptions, SeverityLevel } from '@sentry/node'
 import type * as SentryLib from '@sentry/node'
 import { BackendErrorRequestHandler, BackendRequestHandler, getRequestLogger } from '../index'
@@ -93,22 +93,12 @@ export class SentrySharedService {
     // It's a Breadcrumb, not a console.log, because console.log are NOT automatically attached as Breadcrumbs in cron-job environments (outside of Express)
     this.sentry().addBreadcrumb({
       message: inspectAny(err, {
-        includeErrorData: true,
         colors: false,
       }),
       // data: (err as AppError).data, // included in message
     })
 
-    return this.sentry().captureException(
-      _anyToError(
-        err,
-        Error,
-        {},
-        {
-          stringifyFn: inspectAnyStringifyFn,
-        },
-      ),
-    )
+    return this.sentry().captureException(_anyToError(err))
   }
 
   /**
@@ -138,7 +128,6 @@ export class SentrySharedService {
         const message = args
           .map(arg =>
             inspectAny(arg, {
-              includeErrorData: true,
               colors: false,
             }),
           )
@@ -148,16 +137,7 @@ export class SentrySharedService {
           message,
         })
 
-        this.sentry().captureException(
-          _anyToError(
-            args.length === 1 ? args[0] : args,
-            Error,
-            {},
-            {
-              stringifyFn: inspectAnyStringifyFn,
-            },
-          ),
-        )
+        this.sentry().captureException(_anyToError(args.length === 1 ? args[0] : args))
       },
     }
   }

package/src/server/genericErrorMiddleware.ts

@@ -7,7 +7,6 @@ import {
   HttpErrorData,
   HttpErrorResponse,
 } from '@naturalcycles/js-lib'
-import { inspectAnyStringifyFn } from '@naturalcycles/nodejs-lib'
 import { SentrySharedService } from '../sentry/sentry.shared.service'
 import { BackendErrorRequestHandler, BackendRequest, BackendResponse } from './server.model'
 
@@ -73,14 +72,7 @@ export function respondWithError(req: BackendRequest, res: BackendResponse, err:
     req.error(err)
   }
 
-  const originalError = _anyToError(
-    err,
-    Error,
-    {},
-    {
-      stringifyFn: inspectAnyStringifyFn,
-    },
-  )
+  const originalError = _anyToError(err)
 
   let errorId: string | undefined
 
@@ -90,7 +82,8 @@ export function respondWithError(req: BackendRequest, res: BackendResponse, err:
 
   if (res.headersSent) return
 
-  const httpError = _errorToErrorObject<HttpErrorData>(originalError, includeErrorStack)
+  const httpError = _errorToErrorObject<HttpErrorData>(originalError)
+  if (!includeErrorStack) delete httpError.stack
 
   httpError.data.errorId = errorId
   httpError.data.httpStatusCode ||= 500 // default to 500