Faidon Liambotis has submitted this change and it was merged.

Change subject: Varnish: filter-noise on mobile-frontend as well
......................................................................


Varnish: filter-noise on mobile-frontend as well

Copy this from text-frontend. We should probably abstract this into a
common file but let's deal with the 500 spikes first.

Change-Id: I91b3b68982f9fa0fc14fe9de5876f94d7c0494ea
---
M templates/varnish/mobile-frontend.inc.vcl.erb
1 file changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Faidon Liambotis: Verified; Looks good to me, approved



diff --git a/templates/varnish/mobile-frontend.inc.vcl.erb 
b/templates/varnish/mobile-frontend.inc.vcl.erb
index 9fe4009..2c47063 100644
--- a/templates/varnish/mobile-frontend.inc.vcl.erb
+++ b/templates/varnish/mobile-frontend.inc.vcl.erb
@@ -4,7 +4,15 @@
 include "device-detection.inc.vcl";
 include "zero.inc.vcl";
 
+sub filter_noise {
+       if (req.request == "POST" && req.url ~ 
"index\.php\?option=com_jce&task=plugin&plugin=imgmanager&file=imgmanager&method=form&cid=20&6bc427c8a7981f4fe1f5ac65c1246b5f=cf6dd3cf1923c950586d0dd595c8e20b$")
 {
+               error 403 "Noise";
+       }
+}
+
 sub vcl_recv {
+       call filter_noise;
+
        if (!req.http.X-Forwarded-Proto) {
                /* This call will always set the following req.http values:
                        X-Subdomain = M or ZERO

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I91b3b68982f9fa0fc14fe9de5876f94d7c0494ea
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <[email protected]>
Gerrit-Reviewer: Faidon Liambotis <[email protected]>

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

Reply via email to