Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Crash when capturing JSON from invalid response

See original GitHub issue

When running a test that captures data from the response and the response does not match the expected shape it crashes

AssertionError [ERR_ASSERTION]: obj needs to be an object
    at JSONPath.query (/home/kenneth/load-testing/node_modules/jsonpath/lib/index.js:91:10)
    at extractJSONPath (/home/kenneth/load-testing/node_modules/artillery/core/lib/engine_util.js:481:26)
    at /home/kenneth/load-testing/node_modules/artillery/core/lib/engine_util.js:355:30
    at parseJSON (/home/kenneth/load-testing/node_modules/artillery/core/lib/engine_util.js:468:10)
    at /home/kenneth/load-testing/node_modules/artillery/core/lib/engine_util.js:347:7
    at /home/kenneth/load-testing/node_modules/async/lib/async.js:181:20
    at iterate (/home/kenneth/load-testing/node_modules/async/lib/async.js:262:13)
    at Object.async.forEachOfSeries.async.eachOfSeries (/home/kenneth/load-testing/node_modules/async/lib/async.js:281:9)
    at Object.async.forEachSeries.async.eachSeries (/home/kenneth/load-testing/node_modules/async/lib/async.js:214:22)
    at Object.captureOrMatch (/home/kenneth/load-testing/node_modules/artillery/core/lib/engine_util.js:333:9)

using 1.6.0-24

Issue Analytics

State:
Created 5 years ago
Comments:5 (2 by maintainers)

Top GitHub Comments

2reactions

kennethlynnecommented, Sep 26, 2018

This patch solved it it seems

patch-package
--- a/node_modules/artillery/core/lib/engine_util.js
+++ b/node_modules/artillery/core/lib/engine_util.js
@@ -478,7 +478,12 @@ function extractJSONPath(doc, expr) {
     return '';
   }
 
-  let results = jsonpath.query(doc, expr);
+  let results;
+  try {
+    results = jsonpath.query(doc, expr);
+  } catch (err) {
+    return '';
+  }
 
   if (!results) {
     return '';

0reactions

hassycommented, Oct 19, 2018

New release with a fix will be out in the next couple of days. Thanks @kennethlynne!