sap-ux-sap-systems-ext@1.0.49
Enables you to store connection information for remote SAP systems.
AI Security Review
scanned 6h ago · by lpm-firewall-aiThe package’s SAP connection management behavior and keyring use fit its stated purpose, but runtime telemetry is activated by default without clear disclosure in the package manifest. This warrants a warning; the reviewed telemetry payloads contain usage fields rather than credentials or connection contents.
Static reason
One or more suspicious static signals were detected.
Trigger
When the user opens the extension view or invokes one of its commands, activation initializes telemetry; relevant system actions emit events.
Impact
Usage and platform information is reported without an evident opt-in. The source provides telemetry settings, but defaulting to enabled makes this an undisclosed runtime telemetry risk.
Mechanism
The extension sends action/status telemetry through its Azure Application Insights client. No credential or SAP connection data in the telemetry payload was identified.
AI rationale
The package’s SAP connection management behavior and keyring use fit its stated purpose, but runtime telemetry is activated by default without clear disclosure in the package manifest. This warrants a warning; the reviewed telemetry payloads contain usage fields rather than credentials or connection contents.
Evidence
dist/extension.jspackage.json
Network endpoints2
westus-0.in.applicationinsights.azure.comwesteurope-5.in.applicationinsights.azure.com
Decision evidence
public snapshotAI called this Suspicious at 88.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- Telemetry initializes on extension activation; when no stored setting exists, the code enables it by default.
- The extension sends system telemetry events containing action and status fields to its telemetry client.
- The bundled telemetry client defines Azure Application Insights ingestion endpoints for the US and EU.
Evidence against
- The package is a VS Code SAP connection manager, and its activation is tied to its view and commands.
- Credential operations use a keyring API for storing, retrieving, and deleting passwords; no credential transfer to telemetry was identified.
- The manifest has no npm install lifecycle scripts.
Behavioral surface
DynamicRequireEnvironmentVarsEvalFilesystemNetwork
HighEntropyStringsMinifiedPossibleObfuscationUrlStrings
Source & flagged code
4 flaggeddist/webapp/store.jsView file
68patternName = generic_password
severity = medium
line = 68
Medium
dist/zowe-keyring-wrapper.jsView file
5L6: const { join } = require('path');
L7: const { existsSync } = require('fs');
Medium
Dynamic Require
Package source references dynamic require/import behavior.
dist/zowe-keyring-wrapper.jsView on unpkg · L5prebuilds/keyring.linux-x64-musl.nodeView file
•path = prebuilds/keyring.linux-x64-musl.node
kind = native_binary
sizeBytes = 393832
magicHex = [redacted]
Findings
7 Medium6 Low
MediumSecret Patterndist/webapp/store.js
MediumDynamic Requiredist/zowe-keyring-wrapper.js
MediumNetwork
MediumEnvironment Vars
MediumShips Native Binaryprebuilds/keyring.linux-x64-musl.node
MediumOversized Source Filedist/extension.js
MediumStructural Risk Force Deep Review
LowScripts Present
LowEval
LowFilesystem
LowHigh Entropy Strings
LowUrl Strings
LowNo License
Affected versions and remediation
This report applies to sap-ux-sap-systems-ext@1.0.49.
See all versions of sap-ux-sap-systems-ext for other recorded verdicts.
- Review the evidence and your use of sap-ux-sap-systems-ext@1.0.49 before allowing it. Restrict the permissions described in this report.
- Choose an independently verified alternative or release. This report does not establish that other versions are safe.
Evidence last updated: .
Think this verdict is wrong? Dispute it on GitHub or read how reports are produced.