Skip to content

Updated Cookies & Caching section #5188

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.

Sign up for GitHub

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 2 commits into from
Closed

Updated Cookies & Caching section #5188

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
59a27de
Updated Cookies & Caching section
lukey78 Apr 17, 2015
dd51621
Changed code block indentation
lukey78 Apr 20, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Updated Cookies & Caching section 
Correct code for Varnish 4 (unset instead of remove)
  • Loading branch information
@lukey78
lukey78 committed Apr 17, 2015
commit 59a27de1e0e5d8d7a5aedd67b04ea81fade9063a
25 changes: 23 additions & 2 deletions cookbook/cache/varnish.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,9 @@ session cookie, if there is one, and get rid of all other cookies so that pages
are cached if there is no active session. Unless you changed the default
configuration of PHP, your session cookie has the name ``PHPSESSID``:

.. code-block:: varnish4
.. configuration-block::

.. code-block:: varnish4

sub vcl_recv {
Copy link
Member

Choose a reason for hiding this comment

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

The code block itself now needs to be indented by four spaces.

// Remove all cookies except the session ID.
Expand All @@ -89,11 +91,30 @@ configuration of PHP, your session cookie has the name ``PHPSESSID``:

if (req.http.Cookie == "") {
// If there are no more cookies, remove the header to get page cached.
remove req.http.Cookie;
unset req.http.Cookie;
}
}
}

.. code-block:: varnish3

sub vcl_recv {
// Remove all cookies except the session ID.
if (req.http.Cookie) {
set req.http.Cookie = ";" + req.http.Cookie;
set req.http.Cookie = regsuball(req.http.Cookie, "; +", ";");
set req.http.Cookie = regsuball(req.http.Cookie, ";(PHPSESSID)=", "; \1=");
set req.http.Cookie = regsuball(req.http.Cookie, ";[^ ][^;]*", "");
set req.http.Cookie = regsuball(req.http.Cookie, "^[; ]+|[; ]+$", "");

if (req.http.Cookie == "") {
// If there are no more cookies, remove the header to get page cached.
remove req.http.Cookie;
}
}
}

Copy link
Member

Choose a reason for hiding this comment

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

This blank line should be removed.


.. tip::

If content is not different for every user, but depends on the roles of a
Expand Down