@platformatic/foundation 3.65.0 → 3.66.0

package/lib/configuration.js

@@ -513,6 +513,7 @@ export async function loadConfiguration (source, schema, options = {}) {
 
   if (shouldReplaceEnv) {
     const missingEnv = new Set()
+    const fallbackEnv = new Set()
 
     config = replaceEnv(
       config,
@@ -522,6 +523,11 @@ export async function loadConfiguration (source, schema, options = {}) {
 
         if (typeof value === 'undefined' || value === null) {
           missingEnv.add(key)
+        } else {
+          // The variable is not set: it only has a value because onMissingEnv provided a fallback.
+          // Track it separately so that strictEnv can still report it, as a fallback can silently
+          // mask a misconfiguration.
+          fallbackEnv.add(key)
         }
 
         return value
@@ -533,17 +539,31 @@ export async function loadConfiguration (source, schema, options = {}) {
     // (runtime configurations) or in the runtime property (capabilities configurations).
     const strictEnv = normalizeStrictEnv(strictEnvOption ?? config.strictEnv ?? config.runtime?.strictEnv)
 
+    function warn (message) {
+      if (logger) {
+        logger.warn(message)
+      } else {
+        process.emitWarning(message)
+      }
+    }
+
+    // Variables resolved by a fallback are always reported as a warning, never as an error: they did
+    // resolve to a value, so failing on them would change which configurations are able to boot.
+    // This is emitted before handling the missing ones so that a throw does not swallow it.
+    if (strictEnv && fallbackEnv.size > 0) {
+      const keys = Array.from(fallbackEnv).sort().join(', ')
+
+      warn(
+        'The configuration references the following environment variables which are not set ' +
+          `and have been replaced by a fallback value: ${keys}`
+      )
+    }
+
     if (strictEnv && missingEnv.size > 0) {
       const keys = Array.from(missingEnv).sort().join(', ')
 
       if (strictEnv === 'warn') {
-        const message = `The configuration references the following environment variables which are not set: ${keys}`
-
-        if (logger) {
-          logger.warn(message)
-        } else {
-          process.emitWarning(message)
-        }
+        warn(`The configuration references the following environment variables which are not set: ${keys}`)
       } else {
         throw new MissingEnvVariablesError(keys)
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/foundation",
-  "version": "3.65.0",
+  "version": "3.66.0",
   "description": "Platformatic Foundation",
   "main": "index.js",
   "types": "index.d.ts",
@@ -50,6 +50,7 @@
   "scripts": {
     "test": "npm run test:types && node --test --test-reporter=cleaner-spec-reporter --test-concurrency=1 --test-timeout=2000000 test/*.test.js test/**/*.test.js",
     "test:types": "tstyche",
-    "lint": "eslint"
+    "lint": "eslint",
+    "lint:fix": "eslint --fix"
   }
 }