@vscode/extension-telemetry - npm Package Compare versions

Comparing version 0.8.3 to 0.8.4

dist/browser/common/baseTelemetryReporter.js

@@ -79,6 +79,7 @@ /*---------------------------------------------------------
         const modifiedProperties = { ...properties };
-        for (const property of Object.keys(modifiedProperties ?? {})) {
-            if (typeof property === "string") {
+        for (const propertyKey of Object.keys(modifiedProperties ?? {})) {
+            const propertyValue = modifiedProperties[propertyKey];
+            if (typeof propertyKey === "string" && propertyValue !== undefined) {
                 // Trusted values are not sanitized, which is what we want for raw telemetry
-                modifiedProperties[property] = new this.vscodeAPI.TelemetryTrustedValue(property);
+                modifiedProperties[propertyKey] = new this.vscodeAPI.TelemetryTrustedValue(typeof propertyValue === 'string' ? propertyValue : propertyValue.value);
             }
@@ -85,0 +86,0 @@ }

dist/browser/common/util.js

@@ -53,3 +53,3 @@ /*---------------------------------------------------------
             // Do not change this string as it gets found and replaced upon packaging
-            "common.telemetryclientversion": "0.8.3"
+            "common.telemetryclientversion": "0.8.4"
         };
@@ -56,0 +56,0 @@ }

dist/node/common/baseTelemetryReporter.js

@@ -82,6 +82,7 @@ "use strict";
         const modifiedProperties = { ...properties };
-        for (const property of Object.keys(modifiedProperties ?? {})) {
-            if (typeof property === "string") {
+        for (const propertyKey of Object.keys(modifiedProperties ?? {})) {
+            const propertyValue = modifiedProperties[propertyKey];
+            if (typeof propertyKey === "string" && propertyValue !== undefined) {
                 // Trusted values are not sanitized, which is what we want for raw telemetry
-                modifiedProperties[property] = new this.vscodeAPI.TelemetryTrustedValue(property);
+                modifiedProperties[propertyKey] = new this.vscodeAPI.TelemetryTrustedValue(typeof propertyValue === 'string' ? propertyValue : propertyValue.value);
             }
@@ -88,0 +89,0 @@ }

dist/node/common/util.js

@@ -56,3 +56,3 @@ "use strict";
             // Do not change this string as it gets found and replaced upon packaging
-            "common.telemetryclientversion": "0.8.3"
+            "common.telemetryclientversion": "0.8.4"
         };
@@ -59,0 +59,0 @@ }

dist/node/node/telemetryReporter.js

@@ -162,3 +162,7 @@ "use strict";
                 });
-                req.write(payload.data);
+                req.write(payload.data, (err) => {
+                    if (err) {
+                        oncomplete(0, {});
+                    }
+                });
                 req.end();
@@ -165,0 +169,0 @@ }

package.json

 {
   "name": "@vscode/extension-telemetry",
   "description": "A module for Visual Studio Code extensions to report consistent telemetry.",
-  "version": "0.8.3",
+  "version": "0.8.4",
   "author": {
@@ -32,3 +32,3 @@ "name": "Microsoft Corporation"
     "@types/sinon": "^10.0.16",
-    "@types/vscode": "^1.81.0",
+    "@types/vscode": "^1.75.0",
     "@typescript-eslint/eslint-plugin": "^6.3.0",
@@ -35,0 +35,0 @@ "@typescript-eslint/parser": "^6.3.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

67065

increased by0.7%

Lines of code

1241

increased by0.49%

No dependency changes