BBlack has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/391025 )

Change subject: cache_upload: do not apply 256K hfp to CL-less requests
......................................................................


cache_upload: do not apply 256K hfp to CL-less requests

Change-Id: Ib3e14c94fcf9b078fff388b04f27f23b7d9117ea
---
M modules/varnish/templates/upload-frontend.inc.vcl.erb
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Ema: Looks good to me, but someone else must approve
  BBlack: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/varnish/templates/upload-frontend.inc.vcl.erb 
b/modules/varnish/templates/upload-frontend.inc.vcl.erb
index 5c10bc4..a00a667 100644
--- a/modules/varnish/templates/upload-frontend.inc.vcl.erb
+++ b/modules/varnish/templates/upload-frontend.inc.vcl.erb
@@ -115,7 +115,7 @@
        }
 
        // hit-for-pass objects >= 256K size
-       if (std.integer(beresp.http.Content-Length, 262144) >= 262144 || 
beresp.http.Content-Length ~ "^[0-9]{9}") {
+       if (std.integer(beresp.http.Content-Length, 0) >= 262144 || 
beresp.http.Content-Length ~ "^[0-9]{9}") {
                set beresp.http.X-CDIS = "pass";
                set beresp.uncacheable = true;
                return (deliver);

-- 
To view, visit https://gerrit.wikimedia.org/r/391025
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib3e14c94fcf9b078fff388b04f27f23b7d9117ea
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: BBlack <[email protected]>
Gerrit-Reviewer: BBlack <[email protected]>
Gerrit-Reviewer: Ema <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to