Fix: JSONResponse helper to preserve original onResponse behavior (Fixes #5042)

myrepo · myrepo · commit aa987ae28e9e · 2025-08-24T09:59:23.000+05:30
diff --git a/lib/helper/JSONResponse.js b/lib/helper/JSONResponse.js
@@ -72,10 +72,11 @@ class JSONResponse extends Helper {
     if (!this.helpers[this.options.requestHelper]) {
       throw new Error(`Error setting JSONResponse, helper ${this.options.requestHelper} is not enabled in config, helpers: ${Object.keys(this.helpers)}`)
     }
-    // connect to REST helper
+    const origOnResponse = this.helpers[this.options.requestHelper].config.onResponse;
     this.helpers[this.options.requestHelper].config.onResponse = response => {
-      this.response = response
-    }
+      this.response = response;
+      if (typeof origOnResponse === 'function') origOnResponse(response);
+    };
   }
 
   _before() {

Original file line number	Diff line number	Diff line change
`@@ -72,10 +72,11 @@ class JSONResponse extends Helper {`
`72`	`72`	`if (!this.helpers[this.options.requestHelper]) {`
`73`	`73`	throw new Error(`Error setting JSONResponse, helper ${this.options.requestHelper} is not enabled in config, helpers: ${Object.keys(this.helpers)}`)
`74`	`74`	`}`
`75`		`- // connect to REST helper`
	`75`	`+ const origOnResponse = this.helpers[this.options.requestHelper].config.onResponse;`
`76`	`76`	`this.helpers[this.options.requestHelper].config.onResponse = response => {`
`77`		`- this.response = response`
`78`		`- }`
	`77`	`+ this.response = response;`
	`78`	`+ if (typeof origOnResponse === 'function') origOnResponse(response);`
	`79`	`+ };`
`79`	`80`	`}`
`80`	`81`
`81`	`82`	`_before() {`