rookout - npm Package Compare versions

Comparing version 0.1.98 to 0.1.99

com_ws/information.js

@@ -181,4 +181,4 @@ "use strict";
     return {
-      "version": "0.1.98",
-      "commit": "845ba3f4064c7c760f2a24040a1c63e34f3356d3"
+      "version": "0.1.99",
+      "commit": "0d6787bd2231516a0ab7d2f1fb385f93d23fadb7"
     };
@@ -185,0 +185,0 @@ }

config.js

@@ -18,4 +18,4 @@ "use strict";
     this.VersionConfiguration = {
-      VERSION: "0.1.98",
-      COMMIT: "845ba3f4064c7c760f2a24040a1c63e34f3356d3"
+      VERSION: "0.1.99",
+      COMMIT: "0d6787bd2231516a0ab7d2f1fb385f93d23fadb7"
     };
@@ -22,0 +22,0 @@ this.ControllerAddress = {

package.json

 {
   "name": "rookout",
-  "commit": "845ba3f4064c7c760f2a24040a1c63e34f3356d3",
-  "version": "0.1.98",
+  "commit": "0d6787bd2231516a0ab7d2f1fb385f93d23fadb7",
+  "version": "0.1.99",
   "description": "Rook is a package for on the fly debugging and data extraction for applications in production",
@@ -6,0 +6,0 @@ "main": "index.js",

processor/NamespaceSerializer.js

@@ -215,5 +215,4 @@ "use strict";
       return;
-    } // FUTURE - consider treating objects with a LOT of attributes as collections
+    }
 
-
     for (let key of Object.keys(obj)) {
@@ -230,14 +229,19 @@ // Protect against primitives pointing to themselves
 
-      let value = new variant_pb.Variant();
+      if (currentDepth + 1 < dumpConfig.maxDepth) {
+        let value = new variant_pb.Variant();
 
-      if ('get' in Object.getOwnPropertyDescriptor(obj, key)) {
-        value.setVariantType(variant_pb.Variant.Type.VARIANT_DYNAMIC);
+        if ('get' in Object.getOwnPropertyDescriptor(obj, key)) {
+          value.setVariantType(variant_pb.Variant.Type.VARIANT_DYNAMIC);
+        } else {
+          this.dumpJSObject(obj[key], value, currentDepth + 1, dumpConfig);
+        }
+
+        let attribute = new variant_pb.Variant.NamedValue();
+        attribute.setName(key);
+        attribute.setValue(value);
+        variant.addAttributes(attribute);
       } else {
-        this.dumpJSObject(obj[key], value, currentDepth + 1, dumpConfig);
+        variant.setMaxDepth(true);
+        continue;
       }
-
-      let attribute = new variant_pb.Variant.NamedValue();
-      attribute.setName(key);
-      attribute.setValue(value);
-      variant.addAttributes(attribute);
     }
@@ -247,11 +251,15 @@
 
-    for (let index in symbols) {
-      let symbol = symbols[index];
-      let symbolName = symbol.toString();
-      let value = new variant_pb.Variant();
-      this.dumpJSObject(obj[symbol], value, currentDepth + 1, dumpConfig);
-      let attribute = new variant_pb.Variant.NamedValue();
-      attribute.setName(symbolName);
-      attribute.setValue(value);
-      variant.addAttributes(attribute);
+    if (currentDepth + 1 < dumpConfig.maxDepth) {
+      for (let index in symbols) {
+        let symbol = symbols[index];
+        let symbolName = symbol.toString();
+        let value = new variant_pb.Variant();
+        this.dumpJSObject(obj[symbol], value, currentDepth + 1, dumpConfig);
+        let attribute = new variant_pb.Variant.NamedValue();
+        attribute.setName(symbolName);
+        attribute.setValue(value);
+        variant.addAttributes(attribute);
+      }
+    } else if (symbols && symbols.length) {
+      variant.setMaxDepth(true);
     }
@@ -366,35 +374,39 @@ }
 
-    for (let property of namespace.properties) {
-      // If it's an Array, ignore it's index attributes and the length method
-      if (namespace.obj.subtype === 'array' && (parseInt(property.name).toString() === property.name || property.name === 'length')) {
-        continue;
-      } // We don't dump proto automatically as it leads to very large dumps
+    if (currentDepth + 1 < dumpConfig.maxDepth) {
+      for (let property of namespace.properties) {
+        // If it's an Array, ignore it's index attributes and the length method
+        if (namespace.obj.subtype === 'array' && (parseInt(property.name).toString() === property.name || property.name === 'length')) {
+          continue;
+        } // We don't dump proto automatically as it leads to very large dumps
 
 
-      if (property.name === '__proto__') {
-        continue;
-      }
+        if (property.name === '__proto__') {
+          continue;
+        }
 
-      let item = new variant_pb.Variant();
+        let item = new variant_pb.Variant();
 
-      if (property.value === null || property.value === undefined) {
-        if ('get' in property) {
-          this.dumpDynamicInspectorObjectNamespace(item);
+        if (property.value === null || property.value === undefined) {
+          if ('get' in property) {
+            this.dumpDynamicInspectorObjectNamespace(item);
+          } else {
+            throw new RookInspectorUnknownProperty(property);
+          }
         } else {
-          throw new RookInspectorUnknownProperty(property);
+          let obj = _InspectorObjectNamespace.default.getObjectInternal(namespace.inspector, property.value);
+
+          if (obj instanceof _InspectorObjectNamespace.default) {
+            this.dumpInspectorObjectNamespace(obj, item, currentDepth + 1, dumpConfig, logErrors);
+          } else {
+            this.dumpJSObject(obj, item, currentDepth + 1, dumpConfig);
+          }
         }
-      } else {
-        let obj = _InspectorObjectNamespace.default.getObjectInternal(namespace.inspector, property.value);
 
-        if (obj instanceof _InspectorObjectNamespace.default) {
-          this.dumpInspectorObjectNamespace(obj, item, currentDepth + 1, dumpConfig, logErrors);
-        } else {
-          this.dumpJSObject(obj, item, currentDepth + 1, dumpConfig);
-        }
+        let attribute = new variant_pb.Variant.NamedValue();
+        attribute.setName(property.name);
+        attribute.setValue(item);
+        variant.addAttributes(attribute);
       }
-
-      let attribute = new variant_pb.Variant.NamedValue();
-      attribute.setName(property.name);
-      attribute.setValue(item);
-      variant.addAttributes(attribute);
+    } else if (namespace.properties && namespace.properties.length) {
+      variant.setMaxDepth(true);
     }
@@ -401,0 +413,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

2311996

increased by0.34%

Lines of code

20370

increased by0.35%

No dependency changes