Skip to content
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

Updated Cookies & Caching section #5188

Closed
wants to merge 2 commits into from
Closed

Updated Cookies & Caching section #5188

wants to merge 2 commits into from

Conversation

lukey78
Copy link
Contributor

@lukey78 lukey78 commented Apr 17, 2015

Correct code for Varnish 4 (unset instead of remove)

Correct code for Varnish 4 (unset instead of remove)
.. 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.

@javiereguiluz
Copy link
Member

This pull request looks ready to be merged.

}
}
}

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.

@xabbuh
Copy link
Member

xabbuh commented Jun 24, 2015

@lukey78 I just left you one last minor comment. Sorry for reacting so lately, but unfortunately we are not notified when someone pushes additional commits to a pull request.

@dbu Would you mind having a look at this change please? You do have much more expertise with Varnish than me. :)

@dbu
Copy link
Contributor

dbu commented Jun 24, 2015

according to the varnish changelog remove was indeed replaced by unset in varnish 4. the wiki page where this example comes from seems to not be updated...

@javiereguiluz
Copy link
Member

This PR can be labeled as finished because @dbu verified the proposed changes and the reamining blank line can be removed during the merge process. Thanks

xabbuh added a commit that referenced this pull request Sep 23, 2015
This PR was submitted for the 2.6 branch but it was merged into the 2.3 branch instead (closes #5188).

Discussion
----------

Updated Cookies & Caching section

Correct code for Varnish 4 (unset instead of remove)

Commits
-------

805a548 Updated Cookies & Caching section
@xabbuh
Copy link
Member

xabbuh commented Sep 23, 2015

Thank you for the changes @lukey78 and please apologise the slow merge. I squashed your commits so that's the reason the pull request is shown as closed. I also removed the additional blank line in ca3b830.

@dbu Thanks for reviewing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants