[DependencyInjection] fix "url" env var processor

nicolas-grekas · nicolas-grekas · commit 5a75f4821077 · 2021-10-17T09:04:24.000+02:00
diff --git a/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php b/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
@@ -255,10 +255,8 @@ public function getEnv($prefix, $name, \Closure $getEnv)
                 'fragment' => null,
             ];
 
-            if (null !== $parsedEnv['path']) {
-                // remove the '/' separator
-                $parsedEnv['path'] = '/' === $parsedEnv['path'] ? null : substr($parsedEnv['path'], 1);
-            }
+            // remove the '/' separator
+            $parsedEnv['path'] = '/' === ($parsedEnv['path'] ?? '/') ? '' : substr($parsedEnv['path'], 1);
 
             return $parsedEnv;
         }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php b/src/Symfony/Component/DependencyInjection/Tests/EnvVarProcessorTest.php
@@ -624,8 +624,8 @@ public function testGetEnvUrlPath(?string $expected, string $url)
     public function provideGetEnvUrlPath()
     {
         return [
-            [null, 'https://symfony.com'],
-            [null, 'https://symfony.com/'],
+            ['', 'https://symfony.com'],
+            ['', 'https://symfony.com/'],
             ['/', 'https://symfony.com//'],
             ['blog', 'https://symfony.com/blog'],
             ['blog/', 'https://symfony.com/blog/'],
