Fix json-prune-xhr-response as per feedback

Related commit: abe4103
gorhill · Sep 6, 2023 · 14d60ac · 14d60ac
1 parent 5c9c0a8
commit 14d60ac
Showing 1 changed file with 26 additions and 10 deletions.
diff --git a/assets/resources/scriptlets.js b/assets/resources/scriptlets.js
@@ -1365,25 +1365,41 @@ function jsonPruneXhrResponse(
             if ( xhrDetails === undefined ) {
                 return innerResponse;
             }
-            if ( typeof innerResponse !== 'object' ) {
-                xhrDetails.response = innerResponse;
+            if ( xhrDetails.response !== undefined ) {
+                return xhrDetails.response;
             }
-            let outerResponse = xhrDetails.response;
-            if ( outerResponse !== undefined ) {
-                return outerResponse;
+            let objBefore;
+            if ( typeof innerResponse === 'object' ) {
+                objBefore = innerResponse;
+            } else if ( typeof innerResponse === 'string' ) {
+                try { objBefore = safe.jsonParse(innerResponse); }
+                catch(ex) { }
+            }
+            if ( typeof objBefore !== 'object' ) {
+                return (xhrDetails.response = innerResponse);
             }
-            outerResponse = objectPrune(
-                innerResponse,
+            const objAfter = objectPrune(
+                objBefore,
                 rawPrunePaths,
                 rawNeedlePaths,
                 { matchAll: true },
                 extraArgs
             );
-            if ( typeof outerResponse !== 'object' ) {
+            let outerResponse;
+            if ( typeof objAfter === 'object' ) {
+                outerResponse = typeof innerResponse === 'string'
+                    ? safe.jsonStringify(objAfter)
+                    : objAfter;
+            } else {
                 outerResponse = innerResponse;
             }
-            xhrDetails.response = outerResponse;
-            return outerResponse;
+            return (xhrDetails.response = outerResponse);
+        }
+        get responseText() {
+            const response = this.response;
+            return typeof response !== 'string'
+                ? super.responseText
+                : response;
         }
     };
 }