-
Notifications
You must be signed in to change notification settings - Fork 32
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
Remove /etc/varnish from our packages #106
Comments
I'm a bit on the fence about this, I clearly follow the desire to clean things up, but:
|
I don't think this is a good idea. Significant part of the value you get from Varnish is VCL. I can't think of a single dynamic site that won't end up in return(pass) because of cookies everywhere, if someone tried to use -b today. This will create another step anyone new to Varnish must accomplish to get it going/caching, and that is just wasted effort instead of doing it at the packaging level. |
This came up again during a discussion and I still think we should ship an empty We already ship a default At the very least, moving from |
From #105:
The only reason why we ship things in
/etc/varnish
and especially adefault.vcl
is tradition. There's nothing in varnish-cache itself that warrants such a default configuration. I suggest we make this change to the weeklies after the Sep 2018 release.The text was updated successfully, but these errors were encountered: