@tryghost/mw-error-handler 0.2.3 → 1.0.1

package/lib/mw-error-handler.js

@@ -9,14 +9,14 @@ const messages = {
     pageNotFound: 'Page not found',
     resourceNotFound: 'Resource not found',
     methodNotAcceptableVersionAhead: {
-        message: 'Request not acceptable for provided Accept-Version header.',
-        context: 'Provided client version {acceptVersion} is ahead of current Ghost instance version {ghostVersion}.',
-        help: 'Upgrade your Ghost instance.'
+        message: 'Request could not be served, the endpoint was not found.',
+        context: 'Provided client accept-version {acceptVersion} is ahead of current Ghost version {ghostVersion}.',
+        help: 'Try upgrading your Ghost install.'
     },
     methodNotAcceptableVersionBehind: {
-        message: 'Request not acceptable for provided Accept-Version header.',
-        context: 'Provided client version {acceptVersion} is outdated and is behind current Ghost version {ghostVersion}.',
-        help: 'Upgrade your Ghost API client.'
+        message: 'Request could not be served, the endpoint was not found.',
+        context: 'Provided client accept-version {acceptVersion} is behind current Ghost version {ghostVersion}.',
+        help: 'Try upgrading your Ghost API client.'
     },
     actions: {
         images: {
@@ -105,32 +105,19 @@ module.exports.prepareStack = (err, req, res, next) => { // eslint-disable-line
 };
 
 const jsonErrorRenderer = (err, req, res, next) => { // eslint-disable-line no-unused-vars
-    res.json({
-        errors: [{
-            message: err.message,
-            context: err.context,
-            help: err.help,
-            errorType: err.errorType,
-            errorDetails: err.errorDetails,
-            ghostErrorCode: err.ghostErrorCode
-        }]
-    });
-};
-
-const jsonErrorRendererV2 = (err, req, res, next) => { // eslint-disable-line no-unused-vars
     const userError = prepareUserMessage(err, req);
 
     res.json({
         errors: [{
-            message: userError.message || null,
+            message: userError.message,
             context: userError.context || null,
             type: err.errorType || null,
             details: err.errorDetails || null,
             property: err.property || null,
             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)
+            id: err.id || null,
+            ghostErrorCode: err.ghostErrorCode || null
         }]
     });
 };
@@ -231,9 +218,6 @@ 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,
@@ -245,21 +229,6 @@ 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,
-    // Handle the error in Sentry
-    sentry.errorHandler,
-    // Format the stack for the user
-    module.exports.prepareStack,
-    // Render the error using JSON format
-    jsonErrorRendererV2
-];
-
 module.exports.handleHTMLResponse = sentry => [
     // Make sure the error can be served
     module.exports.prepareError,

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@tryghost/mw-error-handler",
-  "version": "0.2.3",
+  "version": "1.0.1",
   "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 --all --check-coverage --reporter text --reporter cobertura mocha './test/**/*.test.js'",
+    "test": "NODE_ENV=testing c8 --all --check-coverage --100 --reporter text --reporter cobertura mocha './test/**/*.test.js'",
     "lint": "eslint . --ext .js --cache",
     "posttest": "yarn lint"
   },
@@ -22,7 +22,7 @@
     "c8": "7.11.2",
     "mocha": "10.0.0",
     "should": "13.2.3",
-    "sinon": "13.0.2"
+    "sinon": "14.0.0"
   },
   "dependencies": {
     "@tryghost/debug": "^0.1.9",
@@ -31,5 +31,5 @@
     "lodash": "^4.17.21",
     "semver": "^7.3.6"
   },
-  "gitHead": "e14f269ad87fee8f5c7c9fa6c2bc69d587f26e38"
+  "gitHead": "273e3f58cdce8874bd732ddc5a3fc0aad71e5a96"
 }