@tryghost/mw-error-handler 0.2.2 → 0.2.3

package/lib/mw-error-handler.js

@@ -130,6 +130,7 @@ const jsonErrorRendererV2 = (err, req, res, next) => { // eslint-disable-line no
             help: err.help || null,
             code: err.code || null,
             id: err.id || null
+            // @TODO: add ghostErrorCode here in a major (I thought V2 was for the V2 api, but it's actually the newer/correct handler)
         }]
     });
 };
@@ -193,9 +194,9 @@ module.exports.resourceNotFound = (req, res, next) => {
             semver.coerce(res.locals.safeVersion)
         );
 
-        let errorOptions;
+        let notAcceptableError;
         if (versionComparison === 1) {
-            errorOptions = {
+            notAcceptableError = new errors.RequestNotAcceptableError({
                 message: tpl(
                     messages.methodNotAcceptableVersionAhead.message
                 ),
@@ -205,9 +206,9 @@ module.exports.resourceNotFound = (req, res, next) => {
                 }),
                 help: tpl(messages.methodNotAcceptableVersionAhead.help),
                 code: 'UPDATE_GHOST'
-            };
+            });
         } else {
-            errorOptions = {
+            notAcceptableError = new errors.RequestNotAcceptableError({
                 message: tpl(
                     messages.methodNotAcceptableVersionBehind.message
                 ),
@@ -217,10 +218,10 @@ module.exports.resourceNotFound = (req, res, next) => {
                 }),
                 help: tpl(messages.methodNotAcceptableVersionBehind.help),
                 code: 'UPDATE_CLIENT'
-            };
+            });
         }
 
-        next(new errors.RequestNotAcceptableError(errorOptions));
+        next(notAcceptableError);
     } else {
         next(new errors.NotFoundError({message: tpl(messages.resourceNotFound)}));
     }
@@ -230,6 +231,9 @@ module.exports.pageNotFound = (req, res, next) => {
     next(new errors.NotFoundError({message: tpl(messages.pageNotFound)}));
 };
 
+/**
+ * @deprecated: not used as of Ghost v5 and should be removed in a major bump
+ */
 module.exports.handleJSONResponse = sentry => [
     // Make sure the error can be served
     module.exports.prepareError,
@@ -241,6 +245,10 @@ module.exports.handleJSONResponse = sentry => [
     jsonErrorRenderer
 ];
 
+/**
+ * @deprecated with above
+ * @TODO: rename this without the v2 in a major bump
+ */
 module.exports.handleJSONResponseV2 = sentry => [
     // Make sure the error can be served
     module.exports.prepareError,
@@ -260,4 +268,3 @@ module.exports.handleHTMLResponse = sentry => [
     // Format the stack for the user
     module.exports.prepareStack
 ];
-

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tryghost/mw-error-handler",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "repository": "https://github.com/TryGhost/Utils/tree/main/packages/mw-error-handler",
   "author": "Ghost Foundation",
   "license": "MIT",
@@ -20,9 +20,9 @@
   },
   "devDependencies": {
     "c8": "7.11.2",
-    "mocha": "9.1.3",
+    "mocha": "10.0.0",
     "should": "13.2.3",
-    "sinon": "12.0.1"
+    "sinon": "13.0.2"
   },
   "dependencies": {
     "@tryghost/debug": "^0.1.9",
@@ -31,5 +31,5 @@
     "lodash": "^4.17.21",
     "semver": "^7.3.6"
   },
-  "gitHead": "9a90e404200df76c6e4e3056f89d73eb69c9013e"
+  "gitHead": "e14f269ad87fee8f5c7c9fa6c2bc69d587f26e38"
 }