systeminformation 5.17.15 → 5.17.17

package/lib/osinfo.js

@@ -168,15 +168,21 @@ function getFQDN() {
   let fqdn = os.hostname;
   if (_linux || _darwin) {
     try {
-      const stdout = execSync('hostname -f');
-      fqdn = stdout.toString().split(os.EOL)[0];
+      const stdout = execSync('hostnamectl --json short 2>/dev/null');
+      const json = JSON.parse(stdout.toString());
+
+      fqdn = json['StaticHostname'];
     } catch (e) {
-      util.noop();
+      try {
+        const stdout = execSync('hostname -f 2>/dev/null');
+        fqdn = stdout.toString().split(os.EOL)[0];
+      } catch (e) {
+        util.noop();
+      }
     }
-  }
-  if (_freebsd || _openbsd || _netbsd) {
+  } if (_freebsd || _openbsd || _netbsd) {
     try {
-      const stdout = execSync('hostname');
+      const stdout = execSync('hostname 2>/dev/null');
       fqdn = stdout.toString().split(os.EOL)[0];
     } catch (e) {
       util.noop();
@@ -1053,7 +1059,7 @@ function getUniqueMacAdresses() {
     for (let dev in ifaces) {
       if ({}.hasOwnProperty.call(ifaces, dev)) {
         ifaces[dev].forEach(function (details) {
-          if (details?.mac && details.mac !== '00:00:00:00:00:00') {
+          if (details && details.mac && details.mac !== '00:00:00:00:00:00') {
             const mac = details.mac.toLowerCase();
             if (macs.indexOf(mac) === -1) {
               macs.push(mac);

package/lib/usb.js

@@ -125,20 +125,36 @@ function parseDarwinUsb(usb, id) {
       if (lines[i] !== '{' && lines[i] !== '}' && lines[i + 1] && lines[i + 1].trim() !== '}') {
         lines[i] = lines[i] + ',';
       }
+
+      lines[i] = lines[i].replace(':Yes,', ':"Yes",');
       lines[i] = lines[i].replace(': Yes,', ': "Yes",');
+      lines[i] = lines[i].replace(': Yes', ': "Yes"');
+      lines[i] = lines[i].replace(':No,', ':"No",');
       lines[i] = lines[i].replace(': No,', ': "No",');
+      lines[i] = lines[i].replace(': No', ': "No"');
+
+      // In this case (("com.apple.developer.driverkit.transport.usb"))
+      lines[i] = lines[i].replace('((', '').replace('))', '');
+
+      // In case we have <923c11> we need make it "<923c11>" for correct JSON parse
+      const match = /<(\w+)>/.exec(lines[i]);
+      if (match) {
+        const number = match[0];
+        lines[i] = lines[i].replace(number, `"${number}"`);
+      }
     }
     const usbObj = JSON.parse(lines.join('\n'));
-    const removableDrive = usbObj['Built-In'].toLowerCase() !== 'yes' && usbObj['non-removable'].toLowerCase() === 'no';
+    const removableDrive = (usbObj['Built-In'] ? usbObj['Built-In'].toLowerCase() !== 'yes' : true) && (usbObj['non-removable'] ? usbObj['non-removable'].toLowerCase() === 'no' : true);
 
     result.bus = null;
     result.deviceId = null;
     result.id = usbObj['USB Address'] || null;
     result.name = usbObj['kUSBProductString'] || usbObj['USB Product Name'] || null;
     result.type = getDarwinUsbType((usbObj['kUSBProductString'] || usbObj['USB Product Name'] || '').toLowerCase() + (removableDrive ? ' removable' : ''));
-    result.removable = usbObj['non-removable'].toLowerCase() === 'no';
+    result.removable = usbObj['non-removable'] ? usbObj['non-removable'].toLowerCase() || '' === 'no' : true;
     result.vendor = usbObj['kUSBVendorString'] || usbObj['USB Vendor Name'] || null;
     result.manufacturer = usbObj['kUSBVendorString'] || usbObj['USB Vendor Name'] || null;
+
     result.maxPower = null;
     result.serialNumber = usbObj['kUSBSerialNumberString'] || null;
 

package/lib/util.js

@@ -315,7 +315,7 @@ function getWmic() {
     if (!fs.existsSync(wmicPath)) {
       try {
         const wmicPathArray = execSync('WHERE WMIC', execOptsWin).toString().split('\r\n');
-        if (wmicPathArray?.length) {
+        if (wmicPathArray && wmicPathArray.length) {
           wmicPath = wmicPathArray[0];
         } else {
           wmicPath = 'wmic';
@@ -385,7 +385,7 @@ function powerShellStart() {
       encoding: 'UTF-8',
       env: util._extend({}, process.env, { LANG: 'en_US.UTF-8' })
     });
-    if (_psChild?.pid) {
+    if (_psChild && _psChild.pid) {
       _psPersistent = true;
       _psChild.stdout.on('data', function (data) {
         _psResult = _psResult + data.toString('utf8');
@@ -436,7 +436,7 @@ function powerShell(cmd) {
           start: new Date()
         });
         try {
-          if (_psChild?.pid) {
+          if (_psChild && _psChild.pid) {
             _psChild.stdin.write(_psToUTF8 + 'echo ' + _psCmdStart + id + _psIdSeperator + '; ' + os.EOL + cmd + os.EOL + 'echo ' + _psCmdSeperator + os.EOL);
           }
         } catch (e) {
@@ -464,7 +464,7 @@ function powerShell(cmd) {
               resolve(result);
             });
           }
-          if (child?.pid) {
+          if (child && child.pid) {
             child.stdout.on('data', function (data) {
               result = result + data.toString('utf8');
             });
@@ -514,7 +514,7 @@ function execSafe(cmd, args, options) {
             resolve(result);
           });
         }
-        if (child?.pid) {
+        if (child && child.pid) {
           child.stdout.on('data', function (data) {
             result += data.toString();
           });
@@ -576,7 +576,7 @@ function smartMonToolsInstalled() {
   if (_windows) {
     try {
       const pathArray = execSync('WHERE smartctl 2>nul', execOptsWin).toString().split('\r\n');
-      if (pathArray?.length) {
+      if (pathArray && pathArray.length) {
         _smartMonToolsInstalled = pathArray[0].indexOf(':\\') >= 0;
       } else {
         _smartMonToolsInstalled = false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "systeminformation",
-  "version": "5.17.15",
+  "version": "5.17.17",
   "description": "Advanced, lightweight system and OS information library",
   "license": "MIT",
   "author": "Sebastian Hildebrandt <hildebrandt@plus-innovations.com> (https://plus-innovations.com)",