XHR details not being passed properly (#802)

* Use a string to pass the XHR detail object to solve a Chromium thing. * Fix High Low helper and other XHR related features https://stackoverflow.com/a/53914790 https://issues.chromium.org/issues/40091619 --------- Co-authored-by: Sashank999 <[email protected]>
Mephiles · Aug 10, 2024 · 29e3ff3 · 29e3ff3
1 parent 1616144
commit 29e3ff3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/extension/changelog.json b/extension/changelog.json
@@ -8,7 +8,8 @@
 			"fixes": [
 				{ "message": "Fix Disable Gym Stats when switching gyms.", "contributor": "TheFoxMan" },
 				{ "message": "Add Vanguard armor set to Auction House Filter.", "contributor": "TheFoxMan" },
-				{ "message": "Fix Employee Effectiveness Highlight after training an employee.", "contributor": "TheFoxMan" }
+				{ "message": "Fix Employee Effectiveness Highlight after training an employee.", "contributor": "TheFoxMan" },
+				{ "message": "Fix High Low helper and other XHR related features.", "contributor": "DeKleineKobini" }
 			],
 			"changes": [{ "message": "Total Portfolio Value is now live and accurate unless stocks transactions are made.", "contributor": "TheFoxMan" }],
 			"removed": []

diff --git a/extension/scripts/global/inject/xhr.inject.js b/extension/scripts/global/inject/xhr.inject.js
@@ -45,7 +45,11 @@ function interceptXHR(channel) {
 							json,
 							uri,
 							xhr: {
-								...this,
+								// We used to pass the current XHR here as "...this"
+								// but not possible due to some change in Chromium.
+								// https://stackoverflow.com/a/53914790
+								// https://issues.chromium.org/issues/40091619
+								requestBody: this.requestBody,
 								response: this.response,
 								responseURL: this.responseURL,
 							},