Mark Bergsma has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/59401


Change subject: Simplify and cleanup the mobile host rewrites
......................................................................

Simplify and cleanup the mobile host rewrites

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


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/01/59401/1

diff --git a/templates/varnish/mobile-frontend.inc.vcl.erb 
b/templates/varnish/mobile-frontend.inc.vcl.erb
index da1b627..7d6263c 100644
--- a/templates/varnish/mobile-frontend.inc.vcl.erb
+++ b/templates/varnish/mobile-frontend.inc.vcl.erb
@@ -574,19 +574,17 @@
        set req.http.host = regsub(req.http.host, 
"^([a-zA-Z0-9-]+)\.(m|zero)\.", "\1.");
 
        /* Rewrite <language>.wap.wikipedia.org to new mobile site */
-       if ( req.http.host ~ "^([a-zA-Z0-9-]+)\.wap\.([a-zA-Z0-9-]+)\.org" ) {
-               set req.http.host = regsub( req.http.host, 
"^([a-zA-Z0-9-]+)\.wap\.([a-zA-Z0-9-]+)\.org", "\1.mobile.\2.org");
-       }
+       set req.http.host = regsub(req.http.host, "^([a-zA-Z0-9-]+)\.wap\.", 
"\1.mobile.");
 
        /* Rewrite m.mediawiki.org to mediawiki.org etc */
-       if ( req.http.host ~ "^m\.mediawiki\.org" ) {
-               set req.http.host = regsub( req.http.host, 
"^m\.mediawiki\.org", "www.mediawiki.org");
+       if (req.http.host == "m.mediawiki.org") {
+               set req.http.host = "www.mediawiki.org";
        }
 
        /* Support the old mobile.wikipedia.org wap gateway */
-       else if ( req.http.host ~ 
"^([a-zA-Z0-9-]+)\.mobile\.([a-zA-Z0-9-]+)\.org" ) {
-               set req.http.host = regsub( req.http.host, 
"^([a-zA-Z0-9-]+)\.mobile\.([a-zA-Z0-9-]+)\.org", "\1.\2.org" );
-               set req.url = regsub( req.url, "\S+[\?&]go=([^&]+).*", 
"/wiki/\1" );
+       if (req.http.host ~ "^([a-zA-Z0-9-]+)\.mobile\.([a-zA-Z0-9-]+)\.org$") {
+               set req.http.host = regsub(req.http.host, 
"^([a-zA-Z0-9-]+)\.mobile\.", "\1.");
+               set req.url = regsub(req.url, "\S+[\?&]go=([^&]+).*", 
"/wiki/\1");
        }
 
        /* FIXME: deploy X-Vary-Options support */

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iab9633b265fa0f7a7f05080ed46c217bedd75bd8
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mark Bergsma <[email protected]>

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

Reply via email to