Merge pull request #51 from contentstack/bugfix/ie11-node-fetch-error

uttamukkoji · web-flow · commit 88b660e0c18e · 2021-04-16T17:50:59.000+05:30
Bugfix: IE11 node fetch error
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,12 @@
 
+
+### Version: 3.13.1
+#### Date: Apr-16-2021
+##### Bug fix:
+ - IE 11 request method issue resolved
+ 
 ### Version: 3.13.0
-#### Date: 
+#### Date: Apr-05-2021
 
 ##### Update API:
  - [Query]: Added support for method includeEmbeddedItems
diff --git a/dist/nativescript/contentstack.js b/dist/nativescript/contentstack.js
diff --git a/dist/node/contentstack.js b/dist/node/contentstack.js
diff --git a/dist/react-native/contentstack.js b/dist/react-native/contentstack.js
diff --git a/dist/web/contentstack.js b/dist/web/contentstack.js
diff --git a/examples/node/contentstack-demo.js b/examples/node/contentstack-demo.js
@@ -15,7 +15,6 @@ class ContentstackDemo {
             //console.log("datattatata", Contentstack.Region.EUROPE)
 
         this.Stack = Contentstack.Stack(...config);
-        console.log(Contentstack.Utils.render());
     }
     
 
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "contentstack",
-  "version": "3.13.0",
+  "version": "3.13.1",
   "description": "Contentstack Javascript SDK",
   "homepage": "https://www.contentstack.com/",
   "author": {
@@ -10,7 +10,7 @@
   "main": "dist/node/contentstack.js",
   "browser": "dist/web/contentstack.js",
   "types": "./index.d.ts",
-  "_id": "contentstack@3.13.0",
+  "_id": "contentstack@3.13.1",
   "scripts": {
     "test": "npm run test:e2e && npm run test:typescript",
     "test:e2e": "istanbul cover tape test/index.js | tap-html --out ./tap-html.html",
@@ -22,7 +22,8 @@
     "buildnativescript": "webpack --config webpack/webpack.nativescript.js",
     "build": "NODE_ENV=production && npm run buildnode && npm run buildweb && npm run buildreactnative && npm run buildnativescript",
     "generate-docs": "node_modules/.bin/jsdoc --configure docs-config.json --verbose",
-    "prepare": "npm run build"
+    "prepare": "npm run build",
+    "pretest": "npm run build"
   },
   "repository": {
     "type": "git",
@@ -53,14 +54,14 @@
   ],
   "dist": {
     "shasum": "a328ed07240476a26b31a23261355dc929e1da63",
-    "tarball": "https://registry.npmjs.org/contentstack/-/contentstack-3.13.0.tgz"
+    "tarball": "https://registry.npmjs.org/contentstack/-/contentstack-3.13.1.tgz"
   },
   "license": "MIT",
   "directories": {},
-  "_resolved": "https://registry.npmjs.org/contentstack/-/contentstack-3.13.0.tgz",
+  "_resolved": "https://registry.npmjs.org/contentstack/-/contentstack-3.13.1.tgz",
   "_npmOperationalInternal": {
     "host": "packages-18-east.internal.npmjs.com",
-    "tmp": "tmp/contentstack-3.13.0.tgz_1477830884275_0.9869455888401717"
+    "tmp": "tmp/contentstack-3.13.1.tgz_1477830884275_0.9869455888401717"
   },
   "devDependencies": {
     "@types/jest": "^26.0.13",
@@ -99,7 +100,7 @@
   "dependencies": {
     "@contentstack/utils": "1.0.0",
     "es6-promise": "4.1.1",
-    "localStorage": "1.0.3",
-    "node-fetch": "^2.6.1"
+    "isomorphic-fetch": "^3.0.0",
+    "localStorage": "1.0.3"
   }
 }
diff --git a/src/core/lib/request.js b/src/core/lib/request.js
@@ -61,7 +61,7 @@ async function safeParseJSON(response) {
     const body = await response.text();
 }
 function fetchRetry(url, headers, retryDelay = 300, retryLimit = 5, fetchOptions, resolve, reject) {
-    var option = Object.assign({ 
+    var option = Utils.mergeDeep({ 
         method: 'GET',
         headers: headers,
         timeout: 30000,                                
diff --git a/src/core/lib/utils.js b/src/core/lib/utils.js
@@ -238,9 +238,7 @@ export function sendRequest(queryObject, options) {
     let tojson = (typeof self.tojson !== 'undefined') ? self.tojson : false;
     let isSingle = (self.entry_uid || self.singleEntry || self.asset_uid) ? true : false;
     let hashQuery = getHash(parseQueryFromParams(self, isSingle, tojson));
-    if (queryObject.requestParams.url.includes(config.urls.sync)) {
-        cachePolicy = -1;
-    }
+
     /**
     for new api v3
     */
diff --git a/src/core/modules/assets.js b/src/core/modules/assets.js
@@ -118,7 +118,7 @@ export default class Assets {
                     query: this._query
                 }
             }
-            var options = Object.assign({}, this.fetchOptions, fetchOptions);
+            var options = Utils.mergeDeep({}, this.fetchOptions, fetchOptions);
             return Utils.sendRequest(this, options);
         } else {
             console.error("Kindly provide an asset uid. e.g. .Assets('bltsomething123')");
diff --git a/src/core/modules/entry.js b/src/core/modules/entry.js
@@ -341,7 +341,7 @@ export default class Entry {
                     query: this._query
                 }
             };
-            var options = Object.assign({}, this.fetchOptions, fetchOptions);
+            var options = Utils.mergeDeep({}, this.fetchOptions, fetchOptions);
             return Utils.sendRequest(this, options);
         } else {
             console.error("Kindly provide an entry uid. e.g. .Entry('bltsomething123')");
diff --git a/src/core/modules/query.js b/src/core/modules/query.js
@@ -757,7 +757,7 @@ export default class Query extends Entry {
                 query: this._query
             }
         };
-        var options = Object.assign({}, this.fetchOptions, fetchOptions);
+        var options = Utils.mergeDeep({}, this.fetchOptions, fetchOptions);
         return Utils.sendRequest(this, options);
     }
 
@@ -790,7 +790,7 @@ export default class Query extends Entry {
                 query: this._query
             }
         };
-        var options = Object.assign({}, this.fetchOptions);
+        var options = Utils.mergeDeep({}, this.fetchOptions);
         return Utils.sendRequest(this, options);
     }
 
diff --git a/src/core/stack.js b/src/core/stack.js
@@ -57,14 +57,14 @@ export default class Stack {
                 return false
             }
          };
-        this.config = Object.assign({}, config)
+        this.config = Utils.mergeDeep({}, config)
 
         if(stack_arguments[0].region && stack_arguments[0].region !== undefined && stack_arguments[0].region !== "us") {
             this.config['host'] = stack_arguments[0].region+"-"+"cdn.contentstack.com";
         } 
 
         if (stack_arguments[0].fetchOptions && stack_arguments[0].fetchOptions !== undefined) {
-            this.fetchOptions =  Object.assign(this.fetchOptions, stack_arguments[0].fetchOptions);
+            this.fetchOptions =  Utils.mergeDeep(this.fetchOptions, stack_arguments[0].fetchOptions);
         }
         
         this.cachePolicy = CacheProvider.policies.IGNORE_CACHE;
@@ -107,7 +107,7 @@ export default class Stack {
                     if(typeof stack_arguments[3] === "string" && stack_arguments[3] !== undefined && stack_arguments[3] !== "us") {
                         this.config['host'] = stack_arguments[3]+"-"+"cdn.contentstack.com";
                     } else if (typeof stack_arguments[3] === 'object') {
-                        this.fetchOptions = Object.assign(this.fetchOptions, stack_arguments[3]);
+                        this.fetchOptions = Utils.mergeDeep(this.fetchOptions, stack_arguments[3]);
                     }
                 }
                 return this;
@@ -126,11 +126,11 @@ export default class Stack {
                     if(typeof stack_arguments[3] === "string" && stack_arguments[3] !== undefined && stack_arguments[3] !== "us") {
                         this.config['host'] = stack_arguments[3]+"-"+"cdn.contentstack.com";
                     } else if (typeof stack_arguments[3] === 'object') {
-                        this.fetchOptions = Object.assign(this.fetchOptions, stack_arguments[3]);
+                        this.fetchOptions = Utils.mergeDeep(this.fetchOptions, stack_arguments[3]);
                     }
                 }
                 if (stack_arguments[4] && typeof stack_arguments[4] === 'object') {
-                    this.fetchOptions = Object.assign(this.fetchOptions, stack_arguments[4]);
+                    this.fetchOptions = Utils.mergeDeep(this.fetchOptions, stack_arguments[4]);
                 }
                 return this;
             default:
@@ -353,7 +353,7 @@ export default class Stack {
                 environment: this.environment
             }
         };
-        var options = Object.assign({}, this.fetchOptions, fetchOptions);
+        var options = Utils.mergeDeep({}, this.fetchOptions, fetchOptions);
         return Request(result, options);
     }
 
@@ -497,7 +497,7 @@ export default class Stack {
 
     sync(params, fetchOptions) {
         this._query = {};
-        this._query = Object.assign(this._query, params);
+        this._query = Utils.mergeDeep(this._query, params);
         this.requestParams = {
             method: 'POST',
             headers: this.headers,
@@ -507,7 +507,7 @@ export default class Stack {
                 query: this._query
             }
         }
-        var options = Object.assign({}, this.fetchOptions, fetchOptions);
+        var options = Utils.mergeDeep({}, this.fetchOptions, fetchOptions);
         return Utils.sendRequest(this, options);
     }
 
diff --git a/src/runtime/web/http.js b/src/runtime/web/http.js
@@ -1,6 +1,5 @@
 import ES6Promise from 'es6-promise';
-import fetch from 'node-fetch';
-
+import fetch from 'isomorphic-fetch';
 ES6Promise.polyfill();
 
 export default fetch;
diff --git a/src/runtime/web/localstorage.js b/src/runtime/web/localstorage.js
@@ -1,16 +1,16 @@
-// export default window.localStorage;
+export default window.localStorage;
 
-let webLocalStoreage = function (){
-    try {
-        var storage = window.localStorage
-        // x = '__storage_test__';
-        // storage.setItem(x, x);
-        // storage.removeItem(x);
-        return storage;
-    } catch(e) {
-        return null
-    }
-};
+// let webLocalStoreage = function (){
+//     try {
+//         var storage = window.localStorage
+//         // x = '__storage_test__';
+//         // storage.setItem(x, x);
+//         // storage.removeItem(x);
+//         return storage;
+//     } catch(e) {
+//         return null
+//     }
+// };
 
-export default webLocalStoreage();
+// export default webLocalStoreage();
     

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,6 @@ class ContentstackDemo {`
`15`	`15`	`//console.log("datattatata", Contentstack.Region.EUROPE)`
`16`	`16`
`17`	`17`	`this.Stack = Contentstack.Stack(...config);`
`18`		`- console.log(Contentstack.Utils.render());`
`19`	`18`	`}`
`20`	`19`
`21`	`20`
Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ async function safeParseJSON(response) {`
`61`	`61`	`const body = await response.text();`
`62`	`62`	`}`
`63`	`63`	`function fetchRetry(url, headers, retryDelay = 300, retryLimit = 5, fetchOptions, resolve, reject) {`
`64`		`- var option = Object.assign({`
	`64`	`+ var option = Utils.mergeDeep({`
`65`	`65`	`method: 'GET',`
`66`	`66`	`headers: headers,`
`67`	`67`	`timeout: 30000,`