@matdata/yasr 5.20.2 → 5.20.3

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@matdata/yasr",
   "description": "Yet Another SPARQL Resultset GUI",
-  "version": "5.20.2",
+  "version": "5.20.3",
   "main": "build/yasr.min.js",
   "types": "build/ts/src/index.d.ts",
   "license": "MIT",

package/src/index.ts

@@ -551,6 +551,11 @@ export class Yasr extends EventEmitter {
   }
 
   public getResponseFromStorage() {
+    if (this.config.maxPersistentResponseSize <= 0) {
+      const storageId = this.getStorageId(this.config.persistenceLabelResponse);
+      if (storageId) this.storage.remove(storageId);
+      return;
+    }
     const storageId = this.getStorageId(this.config.persistenceLabelResponse);
     if (storageId) {
       return this.storage.get(storageId);
@@ -581,6 +586,10 @@ export class Yasr extends EventEmitter {
   }
   private storeResponse() {
     const storageId = this.getStorageId(this.config.persistenceLabelResponse);
+    if (storageId && this.config.maxPersistentResponseSize <= 0) {
+      this.storage.remove(storageId);
+      return;
+    }
     if (storageId && this.results) {
       const storeObj = this.results.getAsStoreObject(this.config.maxPersistentResponseSize);
       if (storeObj && !storeObj.error) {

package/src/parsers/index.ts

@@ -292,9 +292,21 @@ class Parser {
       };
     }
     if (summary) {
-      //if data is set, it should be less than max limit. If not set, it's probably a response error that we'd like to store as well
-      if (summary.data && summary.data.length > maxResponseSize) {
-        return undefined;
+      // If data is set, it should be less than max limit. For non-strings, compare serialized size.
+      if (summary.data !== undefined) {
+        let dataSize = 0;
+        if (typeof summary.data === "string") {
+          dataSize = summary.data.length;
+        } else {
+          try {
+            dataSize = JSON.stringify(summary.data).length;
+          } catch (_e) {
+            return undefined;
+          }
+        }
+        if (dataSize > maxResponseSize) {
+          return undefined;
+        }
       }
       return summary;
     }