Skip to content

Commit f934881

Browse files
authored
Merge pull request #522 from FriendsOfSymfony/allow-parameters
Do not validate configuration when environment parameters are used
2 parents 86d539e + 03de6fe commit f934881

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

src/DependencyInjection/FOSHttpCacheExtension.php

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -375,11 +375,20 @@ private function loadProxyClient(ContainerBuilder $container, XmlFileLoader $loa
375375
private function createHttpDispatcherDefinition(ContainerBuilder $container, array $config, $serviceName)
376376
{
377377
foreach ($config['servers'] as $url) {
378-
$this->validateUrl($url, 'Not a valid Varnish server address: "%s"');
378+
$usedEnvs = [];
379+
$container->resolveEnvPlaceholders($url, null, $usedEnvs);
380+
if (0 === \count($usedEnvs)) {
381+
$this->validateUrl($url, 'Not a valid Varnish server address: "%s"');
382+
}
379383
}
380384
if (!empty($config['base_url'])) {
381-
$baseUrl = $this->prefixSchema($config['base_url']);
382-
$this->validateUrl($baseUrl, 'Not a valid base path: "%s"');
385+
$baseUrl = $config['base_url'];
386+
$usedEnvs = [];
387+
$container->resolveEnvPlaceholders($baseUrl, null, $usedEnvs);
388+
if (0 === \count($usedEnvs)) {
389+
$baseUrl = $this->prefixSchema($baseUrl);
390+
$this->validateUrl($baseUrl, 'Not a valid base path: "%s"');
391+
}
383392
} else {
384393
$baseUrl = null;
385394
}

0 commit comments

Comments
 (0)