@tryghost/mw-error-handler 0.1.2 → 0.1.5

package/lib/mw-error-handler.js

@@ -78,8 +78,6 @@ module.exports.prepareError = (err, req, res, next) => {
     // alternative for res.status();
     res.statusCode = err.statusCode;
 
-    prepareStackForUser(err);
-
     // never cache errors
     res.set({
         'Cache-Control': 'no-cache, private, no-store, must-revalidate, max-stale=0, post-check=0, pre-check=0'
@@ -88,6 +86,12 @@ module.exports.prepareError = (err, req, res, next) => {
     next(err);
 };
 
+module.exports.prepareStack = (err, req, res, next) => { // eslint-disable-line no-unused-vars
+    const clonedError = prepareStackForUser(err);
+
+    next(clonedError);
+};
+
 const jsonErrorRenderer = (err, req, res, next) => { // eslint-disable-line no-unused-vars
     res.json({
         errors: [{
@@ -177,6 +181,8 @@ module.exports.handleJSONResponse = sentry => [
     module.exports.prepareError,
     // Handle the error in Sentry
     sentry.errorHandler,
+    // Format the stack for the user
+    module.exports.prepareStack,
     // Render the error using JSON format
     jsonErrorRenderer
 ];
@@ -186,6 +192,8 @@ module.exports.handleJSONResponseV2 = sentry => [
     module.exports.prepareError,
     // Handle the error in Sentry
     sentry.errorHandler,
+    // Format the stack for the user
+    module.exports.prepareStack,
     // Render the error using JSON format
     jsonErrorRendererV2
 ];
@@ -194,6 +202,8 @@ module.exports.handleHTMLResponse = sentry => [
     // Make sure the error can be served
     module.exports.prepareError,
     // Handle the error in Sentry
-    sentry.errorHandler
+    sentry.errorHandler,
+    // Format the stack for the user
+    module.exports.prepareStack
 ];
 

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@tryghost/mw-error-handler",
-  "version": "0.1.2",
+  "version": "0.1.5",
   "repository": "https://github.com/TryGhost/Utils/tree/main/packages/mw-error-handler",
   "author": "Ghost Foundation",
   "license": "MIT",
   "main": "index.js",
   "scripts": {
     "dev": "echo \"Implement me!\"",
-    "test": "NODE_ENV=testing c8 --check-coverage mocha './test/**/*.test.js'",
+    "test": "NODE_ENV=testing c8 --all --check-coverage --reporter text --reporter cobertura mocha './test/**/*.test.js'",
     "lint": "eslint . --ext .js --cache",
     "posttest": "yarn lint"
   },
@@ -26,7 +26,9 @@
   },
   "dependencies": {
     "@tryghost/debug": "^0.1.9",
-    "@tryghost/tpl": "^0.1.8"
+    "@tryghost/errors": "1.2.5",
+    "@tryghost/tpl": "^0.1.8",
+    "lodash": "4.17.21"
   },
-  "gitHead": "d9374c7c0a1bd92f53b4bf330797a890cdae7e6d"
+  "gitHead": "64b09915cbfd97d139bf8360a79b36a0f45fc552"
 }