Merge pull request #4231 from snyk/chore/HMMR-614_analytics_exception…

…_handling chore: wrap osName() in try-catch
snyk · Dec 5, 2022 · f6d7eb2 · f6d7eb2
2 parents d9151b6 + ead1824
commit f6d7eb2
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/lib/analytics/getStandardData.ts b/src/lib/analytics/getStandardData.ts
@@ -39,8 +39,15 @@ export async function getStandardData(
   const durationMs = Date.now() - START_TIME;
   const metrics = getMetrics(durationMs);
 
+  let osNameString;
+  try {
+    osNameString = osName(os.platform(), os.release());
+  } catch (e) {
+    osNameString = 'unknown';
+  }
+
   const data = {
-    os: osName(os.platform(), os.release()),
+    os: osNameString,
     osPlatform: os.platform(),
     osRelease: os.release(),
     osArch: os.arch(),