Skip to content

Commit

Permalink
renamed proxy setting to trust_proxy_headers
Browse files Browse the repository at this point in the history
  • Loading branch information
fabpot committed Jul 6, 2011
1 parent e943fde commit 722ad12
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion UPDATE.md
Expand Up @@ -18,7 +18,7 @@ RC4 to RC5
following configuration:

framework:
proxy: true
trust_proxy_headers: true

* To avoid hidden naming collisions, the AbstractType does not try to define
the name of form types magically. You now need to implement the `getName()`
Expand Down
Expand Up @@ -47,7 +47,7 @@ public function getConfigTreeBuilder()
$rootNode
->children()
->scalarNode('charset')->end()
->scalarNode('proxy')->defaultFalse()->end()
->scalarNode('trust_proxy_headers')->defaultFalse()->end()
->scalarNode('secret')->isRequired()->end()
->scalarNode('exception_controller')->defaultValue('Symfony\\Bundle\\FrameworkBundle\\Controller\\ExceptionController::showAction')->end()
->scalarNode('ide')->defaultNull()->end()
Expand Down
Expand Up @@ -66,7 +66,7 @@ public function load(array $configs, ContainerBuilder $container)
$container->setParameter('kernel.secret', $config['secret']);
$container->setParameter('exception_listener.controller', $config['exception_controller']);

$container->setParameter('kernel.proxy', $config['proxy']);
$container->setParameter('kernel.trust_proxy_headers', $config['trust_proxy_headers']);

if (!empty($config['test'])) {
$loader->load('test.xml');
Expand Down
2 changes: 1 addition & 1 deletion src/Symfony/Bundle/FrameworkBundle/FrameworkBundle.php
Expand Up @@ -37,7 +37,7 @@ class FrameworkBundle extends Bundle
{
public function boot()
{
if ($this->container->getParameter('kernel.proxy')) {
if ($this->container->getParameter('kernel.trust_proxy_headers')) {
Request::trustProxyData();
}
}
Expand Down

1 comment on commit 722ad12

@Seldaek
Copy link
Member

@Seldaek Seldaek commented on 722ad12 Jul 6, 2011

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Please sign in to comment.