only populate the body on RequestInit options

ServiceStack · Apr 13, 2018 · 5ffb22c · 5ffb22c · olibanjoli · Apr 13, 2018
1 parent 3e33d4b
commit 5ffb22c
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 19 deletions.
diff --git a/src/index.js b/src/index.js
diff --git a/src/index.js.map b/src/index.js.map
diff --git a/src/index.ts b/src/index.ts
@@ -852,23 +852,27 @@ export class JsonServiceClient {
                 this.headers.delete("Cookie");
         }
 
-        // Set `compress` false due to common error
-        // https://github.com/bitinn/node-fetch/issues/93#issuecomment-200791658
-        var reqOptions = {
+        var hasRequestBody = HttpMethods.hasRequestBody(method);
+        var reqOptions:RequestInit = {
             method: method,
             mode: this.mode,
             credentials: this.credentials,
             headers: this.headers,
-            compress: false
         };
-        const req = new Request(url, reqOptions);
 
-        if (HttpMethods.hasRequestBody(method)) {
-            (req as any).body = body || JSON.stringify(request);
+        // Set `compress` false due to common error
+        // https://github.com/bitinn/node-fetch/issues/93#issuecomment-200791658
+        try {
+            (reqOptions as any).compress = false;
+        } catch(e){}
+
+        if (hasRequestBody) {
+            reqOptions.body = body || JSON.stringify(request);
+        }
+        const req = new Request(url, reqOptions);
 
-            if (typeof window != "undefined" && body instanceof FormData) {
-                req.headers.delete('Content-Type'); //set by FormData
-            }
+        if (hasRequestBody && typeof window != "undefined" && body instanceof FormData) {
+            req.headers.delete('Content-Type'); //set by FormData
         }
 
         var opt:IRequestFilterOptions = { url };