-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Incomplete cookbook entry about Varnish + ESI ? #3881
Comments
I agree - this is incomplete as it is now - someone needs to really go through a Varnish setup with Symfony and outline all the steps and things to watch out for. We don't need to rewrite any Varnish docs, we just need a guide on how to navigate through Varnish and Symfony (including details about cookies, recommendations, and links to Varnish docs). A big +1 for this. |
The new docs should probably answer the following questions :
My answers to these questions (open for discussion, may be completely wrong) :
|
It's exactly what I do in my varnish conf. If the response has a But we can not document that. It's too specific and too important. One can mess the conf and break the security layer. People should understand how varnish + http + symfony work. And then they can tweak the config. Anyway, if there is no cookie in the response, varnish is able to cache... |
@lyrixx : so what would you recommend instead ? Links to the Varnish documentation about cookies, and nothing more, so that people figure something out by themselves ? |
Good idea ;) |
is there no more elegant solution about the cookie problem than unsetting it in varnish? can't symfony determine if it needs a session or not? (i asked the same question in #4141 - maybe you can give input there as well. with regard to #4175 i hope to review the caching documentation and propose improvements) |
This PR was merged into the 2.3 branch. Discussion ---------- Varnish cookbook session cookie handling | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | all | Fixed tickets | #3881 This builds on top of #4627 but i wanted to keep it separate as there are open questions in here. Commits ------- b294b24 cleanup from feedback 7a4dafc remove part about vary on cookie c88ad32 explain how to work with cookies and sessions when caching
I was struggling with ESI today, when I stumbled upon this SO question.
The cookbook entry about ESI doesn't talk about session cookies, and it seems that they prevent cache lookups (when in the request) and caching (when in the response) (at least when you start from the default VCL configuration and apply the changes documented in the cookbook entry).
Following the cookbook entry leads to a working html page, but with no caching. Shouldn't this be documented ?
The text was updated successfully, but these errors were encountered: