From bd9df7e630877b66fe3274d2732bb4c8799d45de Mon Sep 17 00:00:00 2001
From: Michael Schiemer <kontakt@michaelschiemer.de>
Date: Sun, 2 Nov 2025 21:58:40 +0100
Subject: [PATCH] fix: Don't store empty values when Docker Secret resolution
 fails in all()

- Update all() first pass to skip storing empty values when secret resolution fails
- Ensures second pass can still resolve secrets from *_FILE variables
- Prevents empty values from blocking secret resolution in second pass
- Should fix issue where REDIS_PASSWORD was empty even though REDIS_PASSWORD_FILE existed
---
 src/Framework/Config/Environment.php | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/Framework/Config/Environment.php b/src/Framework/Config/Environment.php
index 48c4c1fe..809f8c83 100644
--- a/src/Framework/Config/Environment.php
+++ b/src/Framework/Config/Environment.php
@@ -192,6 +192,9 @@ final readonly class Environment
                     $resolved[$key] = $secretValue;
                     continue;
                 }
+                // If secret resolution failed, don't add empty value yet
+                // We'll check again in second pass with *_FILE pattern
+                continue;
             }
             // Include non-empty values
             $resolved[$key] = $value;