Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Varnish configuration to make Symfony routing behave #2857

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 6 commits into from

Conversation

TomiS
Copy link
Contributor

@TomiS TomiS commented Jul 25, 2013

Relates to discussion in this issue


.. note::

Remember to set framework.trust_proxy_headers: true in Symfony configuration for this to work.
Copy link
Member

Choose a reason for hiding this comment

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

in the Symfony configuration

------------------

To ensure that the Symfony Router generates urls correctly with Varnish, proper ```X-Forwarded``` headers must be added. Headers depend on how you have configured hosts and ports for the web server and Varnish but this example should work if the web server is using the same IP as Varnish but a different port (e.g. 8080).

Copy link
Member

Choose a reason for hiding this comment

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

you should wrap your lines after the first word that crosses the 72th character


.. note::

Remember to set framework.trust_proxy_headers: true in the Symfony
Copy link
Member

Choose a reason for hiding this comment

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

You should set framework.trust_proxy_headers: true into double backticks: framework.trust_proxy_headers: true

weaverryan added a commit that referenced this pull request Jul 29, 2013
… non-deprecated trusted_proxies config in forwarded headers docs
@weaverryan
Copy link
Member

Thanks Tomi! I've patched your commits into the 2.2 branch at the following shas:

with some tweaks at sha: bd63f26

Thanks for your addition!

@weaverryan weaverryan closed this Jul 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants