fix: cache local environment values

lib/current-env.js

@@ -1,5 +1,6 @@
 const process = require('node:process')
 const nodeOs = require('node:os')
+const fs = require('node:fs')
 
 function isMusl (file) {
   return file.includes('libc.musl-') || file.includes('ld-musl-')
@@ -13,20 +14,42 @@ function cpu () {
   return process.arch
 }
 
+const LDD_PATH = '/usr/bin/ldd'
+function getFamilyFromFilesystem () {
+  try {
+    const content = fs.readFileSync(LDD_PATH, 'utf-8')
+    if (content.includes('musl')) {
+      return 'musl'
+    }
+    if (content.includes('GNU C Library')) {
+      return 'glibc'
+    }
+    return null
+  } catch {
+    return undefined
+  }
+}
+
+let family
 function libc (osName) {
-  // this is to make it faster on non linux machines
   if (osName !== 'linux') {
     return undefined
   }
-  let family
-  const originalExclude = process.report.excludeNetwork
-  process.report.excludeNetwork = true
-  const report = process.report.getReport()
-  process.report.excludeNetwork = originalExclude
-  if (report.header?.glibcVersionRuntime) {
-    family = 'glibc'
-  } else if (Array.isArray(report.sharedObjects) && report.sharedObjects.some(isMusl)) {
-    family = 'musl'
+  if (family === undefined) {
+    family = getFamilyFromFilesystem()
+    if (family === undefined) {
+      const originalExclude = process.report.excludeNetwork
+      process.report.excludeNetwork = true
+      const report = process.report.getReport()
+      process.report.excludeNetwork = originalExclude
+      if (report.header?.glibcVersionRuntime) {
+        family = 'glibc'
+      } else if (Array.isArray(report.sharedObjects) && report.sharedObjects.some(isMusl)) {
+        family = 'musl'
+      } else {
+        family = null
+      }
+    }
   }
   return family
 }

test/check-platform.js

@@ -94,48 +94,68 @@ t.test('wrong libc with overridden libc', async t =>
   }), { code: 'EBADPLATFORM' }))
 
 t.test('libc', (t) => {
-  let PLATFORM = ''
-
-  const _processPlatform = Object.getOwnPropertyDescriptor(process, 'platform')
-  Object.defineProperty(process, 'platform', {
-    enumerable: true,
-    configurable: true,
-    get: () => PLATFORM,
-  })
-
+  let noCacheChckPtfm
+  let PLATFORM = 'linux'
   let REPORT = {}
-  const _processReport = process.report.getReport
-  process.report.getReport = () => REPORT
-
-  t.teardown(() => {
-    Object.defineProperty(process, 'platform', _processPlatform)
-    process.report.getReport = _processReport
-  })
+  let readFileSync
+
+  function withoutLibcCache () {
+    readFileSync = () => {
+      throw new Error('File not found')
+    }
+    noCacheChckPtfm = (...args) => {
+      const original = t.mock('..', {
+        '../lib/current-env': t.mock('../lib/current-env', {
+          'node:fs': {
+            readFileSync,
+          },
+          'node:process': {
+            platform: PLATFORM,
+            report: {
+              getReport: () => REPORT,
+            },
+          },
+        }),
+      }).checkPlatform
+      withoutLibcCache()
+      return original(...args)
+    }
+  }
+
+  withoutLibcCache()
 
   t.test('fails when not in linux', (t) => {
     PLATFORM = 'darwin'
 
-    t.throws(() => checkPlatform({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
+    t.throws(() => noCacheChckPtfm({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
       'fails for glibc when not in linux')
-    t.throws(() => checkPlatform({ libc: 'musl' }), { code: 'EBADPLATFORM' },
+    t.throws(() => noCacheChckPtfm({ libc: 'musl' }), { code: 'EBADPLATFORM' },
       'fails for musl when not in linux')
     t.end()
   })
 
   t.test('glibc', (t) => {
     PLATFORM = 'linux'
 
+    readFileSync = () => 'this ldd file contains GNU C Library'
+    t.doesNotThrow(() => noCacheChckPtfm({ libc: 'glibc' }), 'allows glibc on glibc from ldd file')
+
     REPORT = {}
-    t.throws(() => checkPlatform({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
+    t.throws(() => noCacheChckPtfm({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
       'fails when report is missing header property')
 
     REPORT = { header: {} }
-    t.throws(() => checkPlatform({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
+    t.throws(() => noCacheChckPtfm({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
       'fails when header is missing glibcVersionRuntime property')
 
     REPORT = { header: { glibcVersionRuntime: '1' } }
-    t.doesNotThrow(() => checkPlatform({ libc: 'glibc' }), 'allows glibc on glibc')
-    t.throws(() => checkPlatform({ libc: 'musl' }), { code: 'EBADPLATFORM' },
+    t.doesNotThrow(() => noCacheChckPtfm({ libc: 'glibc' }), 'allows glibc on glibc')
+
+    readFileSync = () => 'this ldd file is unsupported'
+    t.throws(() => noCacheChckPtfm({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
+      'fails when ldd file exists but is not something known')
+
+    t.throws(() => noCacheChckPtfm({ libc: 'musl' }), { code: 'EBADPLATFORM' },
       'does not allow musl on glibc')
 
     t.end()
@@ -144,25 +164,28 @@ t.test('libc', (t) => {
   t.test('musl', (t) => {
     PLATFORM = 'linux'
 
+    readFileSync = () => 'this ldd file contains musl'
+    t.doesNotThrow(() => noCacheChckPtfm({ libc: 'musl' }), 'allows musl on musl from ldd file')
+
     REPORT = {}
-    t.throws(() => checkPlatform({ libc: 'musl' }), { code: 'EBADPLATFORM' },
+    t.throws(() => noCacheChckPtfm({ libc: 'musl' }), { code: 'EBADPLATFORM' },
       'fails when report is missing sharedObjects property')
 
     REPORT = { sharedObjects: {} }
-    t.throws(() => checkPlatform({ libc: 'musl' }), { code: 'EBADPLATFORM' },
+    t.throws(() => noCacheChckPtfm({ libc: 'musl' }), { code: 'EBADPLATFORM' },
       'fails when sharedObjects property is not an array')
 
     REPORT = { sharedObjects: [] }
-    t.throws(() => checkPlatform({ libc: 'musl' }), { code: 'EBADPLATFORM' },
+    t.throws(() => noCacheChckPtfm({ libc: 'musl' }), { code: 'EBADPLATFORM' },
       'fails when sharedObjects does not contain musl')
 
     REPORT = { sharedObjects: ['ld-musl-foo'] }
-    t.doesNotThrow(() => checkPlatform({ libc: 'musl' }), 'allows musl on musl as ld-musl-')
+    t.doesNotThrow(() => noCacheChckPtfm({ libc: 'musl' }), 'allows musl on musl as ld-musl-')
 
     REPORT = { sharedObjects: ['libc.musl-'] }
-    t.doesNotThrow(() => checkPlatform({ libc: 'musl' }), 'allows musl on musl as libc.musl-')
+    t.doesNotThrow(() => noCacheChckPtfm({ libc: 'musl' }), 'allows musl on musl as libc.musl-')
 
-    t.throws(() => checkPlatform({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
+    t.throws(() => noCacheChckPtfm({ libc: 'glibc' }), { code: 'EBADPLATFORM' },
       'does not allow glibc on musl')
 
     t.end()