Merge pull request #60 from commercetools/serialize-parse-error

Log JSON received from the CTP
Adyen · May 3, 2019 · 28599d6 · 28599d6
2 parents 88086ec + 39dede3
commit 28599d6
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 3 deletions.
diff --git a/extension/package-lock.json b/extension/package-lock.json
diff --git a/extension/package.json b/extension/package.json
@@ -66,6 +66,7 @@
     "bluebird": "^3.5.3",
     "bunyan": "^1.8.12",
     "lodash": "^4.17.11",
-    "node-fetch": "^2.3.0"
+    "node-fetch": "^2.3.0",
+    "serialize-error": "^3.0.0"
   }
 }
diff --git a/extension/src/api/payment/payment.controller.js b/extension/src/api/payment/payment.controller.js
@@ -1,3 +1,4 @@
+const serializeError = require('serialize-error')
 const httpUtils = require('../../utils')
 const creditCardHandler = require('../../paymentHandler/creditCard/credit-card.handler')
 const paypalHandler = require('../../paymentHandler/paypal/paypal.handler')
@@ -47,8 +48,14 @@ async function processRequest (request, response) {
 }
 
 async function _getPaymentObject (request) {
-  const requestBody = JSON.parse(await httpUtils.collectRequestData(request))
-  return requestBody.resource.obj
+  const body = await httpUtils.collectRequestData(request)
+  try {
+    const requestBody = JSON.parse(body)
+    return requestBody.resource.obj
+  } catch (err) {
+    throw new Error(`Error during parsing CTP request: '${body}'. Ending the process. `
+      + `Error: ${JSON.stringify(serializeError(err))}`)
+  }
 }
 
 function _getPaymentHandler (paymentObject) {