Skip to content

extract vcl into a reusable set of subroutines #229

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

Merged
merged 1 commit into from
Jun 30, 2015

Conversation

zghosts
Copy link
Contributor

@zghosts zghosts commented Jun 27, 2015

Sqashed featurebranch ready to be merged

@zghosts zghosts mentioned this pull request Jun 27, 2015
@ddeboer
Copy link
Member

ddeboer commented Jun 27, 2015

@dbu This looks good to go to me. Please have one last look.

@zghosts
Copy link
Contributor Author

zghosts commented Jun 27, 2015

Awesome

@dbu
Copy link
Contributor

dbu commented Jun 28, 2015 via email

ddeboer added a commit that referenced this pull request Jun 30, 2015
extract vcl into a reusable set of subroutines
@ddeboer ddeboer merged commit f97980c into FriendsOfSymfony:master Jun 30, 2015
@ddeboer
Copy link
Member

ddeboer commented Jun 30, 2015

Again, thanks for your work, @zghosts! 😄

@@ -40,15 +53,21 @@ To configure Varnish for `handling PURGE requests <https://www.varnish-cache.org

Purge removes a specific URL (including query strings) in all its variants (as specified by the ``Vary`` header).

.. configuration-block::
Subroutines are provided in ``resources/config/varnis-[version]/fos_purge.vcl``.
Copy link
Member

Choose a reason for hiding this comment

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

missing h here

Copy link
Contributor

@dbu dbu Jul 3, 2015 via email

Choose a reason for hiding this comment

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

dbu added a commit that referenced this pull request Jul 3, 2015
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