symfony
diff --git a/‎src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
+27-15Lines changed: 27 additions & 15 deletions b/‎src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
+27-15Lines changed: 27 additions & 15 deletions
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
+33Lines changed: 33 additions & 0 deletions b/‎src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
+33Lines changed: 33 additions & 0 deletions
@@ -22,6 +22,7 @@ class EnvVarProcessor implements EnvVarProcessorInterface
 {
     private $container;
     private $loaders;
+    private $loadedLoaderIndex = 0;
     private $loadedVars = [];
 
     /**
@@ -30,8 +31,7 @@ class EnvVarProcessor implements EnvVarProcessorInterface
     public function __construct(ContainerInterface $container, \Traversable $loaders = null)
     {
         $this->container = $container;
-        $this->loaders = new \IteratorIterator($loaders ?? new \ArrayIterator());
-        $this->loaders = $this->loaders->getInnerIterator();
+        $this->loaders = $loaders ?? new \ArrayIterator();
     }
 
     /**
@@ -141,20 +141,32 @@ public function getEnv($prefix, $name, \Closure $getEnv)
                 }
             }
 
-            $loaders = $this->loaders;
-            $this->loaders = new \ArrayIterator();
-
-            try {
-                while ((false === $env || null === $env) && $loaders->valid()) {
-                    $loader = $loaders->current();
-                    $loaders->next();
-                    $this->loadedVars[] = $vars = $loader->loadEnvVars();
-                    $env = $vars[$name] ?? false;
+            if (false === $env || null === $env) {
+                $loaders = $this->loaders;
+                $this->loaders = new \ArrayIterator();
+
+                try {
+                    $index = 0;
+                    $ended = true;
+                    foreach ($loaders as $loader) {
+                        if ($index++ < $this->loadedLoaderIndex) {
+                            continue;
+                        }
+                        $this->loadedLoaderIndex = $index;
+                        $this->loadedVars[] = $vars = $loader->loadEnvVars();
+                        if (false !== $env = $vars[$name] ?? false) {
+                            $ended = false;
+                            break;
+                        }
+                    }
+                    if ($ended) {
+                        $loaders = new \ArrayIterator();
+                    }
+                } catch (ParameterCircularReferenceException $e) {
+                    // skip loaders that need an env var that is not defined
+                } finally {
+                    $this->loaders = $loaders;
                 }
-            } catch (ParameterCircularReferenceException $e) {
-                // skip loaders that need an env var that is not defined
-            } finally {
-                $this->loaders = $loaders;
             }
 
             if (false === $env || null === $env) {
 
@@ -3,10 +3,12 @@
 namespace Symfony\Component\DependencyInjection\Tests;
 
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\DependencyInjection\Argument\RewindableGenerator;
 use Symfony\Component\DependencyInjection\Container;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\EnvVarLoaderInterface;
 use Symfony\Component\DependencyInjection\EnvVarProcessor;
+use Symfony\Component\DependencyInjection\Exception\ParameterCircularReferenceException;
 
 class EnvVarProcessorTest extends TestCase
 {
@@ -553,4 +555,35 @@ public function loadEnvVars(): array
         $result = $processor->getEnv('string', 'FOO_ENV_LOADER', function () {});
         $this->assertSame('123', $result); // check twice
     }
+
+    public function testCircularEnvLoader()
+    {
+        $container = new ContainerBuilder();
+        $container->setParameter('env(FOO_CONTAINER)', 'foo');
+        $container->compile();
+
+        $index = 0;
+        $loaders = function () use (&$index) {
+            if (0 === $index++) {
+                throw new ParameterCircularReferenceException(['FOO_CONTAINER']);
+            }
+
+            yield new class() implements EnvVarLoaderInterface {
+                public function loadEnvVars(): array
+                {
+                    return [
+                        'FOO_ENV_LOADER' => '123',
+                    ];
+                }
+            };
+        };
+
+        $processor = new EnvVarProcessor($container, new RewindableGenerator($loaders, 1));
+
+        $result = $processor->getEnv('string', 'FOO_CONTAINER', function () {});
+        $this->assertSame('foo', $result); // check twice
+
+        $result = $processor->getEnv('string', 'FOO_ENV_LOADER', function () {});
+        $this->assertSame('123', $result); // check twice
+    }
 }