Mark Bergsma has submitted this change and it was merged.

Change subject: Fix mobile redirect
......................................................................


Fix mobile redirect

req.http.Host is not a full URL, and doesn't start with http://

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

Approvals:
  Mark Bergsma: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/templates/varnish/text-frontend.inc.vcl.erb 
b/templates/varnish/text-frontend.inc.vcl.erb
index c50262d..95f742a 100644
--- a/templates/varnish/text-frontend.inc.vcl.erb
+++ b/templates/varnish/text-frontend.inc.vcl.erb
@@ -9,7 +9,7 @@
                && req.http.User-Agent !~ 
"(iPad|Android.3|(?i)tablet|(?i)(?!google)bot|PlayBook|Wii)"
                && req.http.Cookie !~ 
"(stopMobileRedirect=true|mf_useformat=desktop)"
                && req.url ~ "^/wiki/") {
-               set req.http.MobileHost = regsub(req.http.host, 
"^http://(?:www\.(?=mediawiki))?((?:commons|species|meta|incubator|outreach|strategy|wikimania2012|wikimania2013)\.(?=wikimedia)|\w+\.(?!wikimedia|mediawiki)|(?=mediawiki|wikimediafoundation))(wikimedia|wikipedia|wiktionary|wikinews|wikisource|wikiquote|wikibooks|wikiversity|wikivoyage|mediawiki|wikimediafoundation)\.",
 "\1m.\2.");
+               set req.http.MobileHost = regsub(req.http.Host, 
"^(?:www\.(?=mediawiki))?((?:commons|species|meta|incubator|outreach|strategy|wikimania2012|wikimania2013)\.(?=wikimedia)|\w+\.(?!wikimedia|mediawiki)|(?=mediawiki|wikimediafoundation))(wikimedia|wikipedia|wiktionary|wikinews|wikisource|wikiquote|wikibooks|wikiversity|wikivoyage|mediawiki|wikimediafoundation)\.",
 "\1m.\2.");
                if (req.http.Host != req.http.MobileHost) {
                        if (req.http.X-Forwarded-Proto) {
                                set req.http.Location = 
req.http.X-Forwarded-Proto + "://" + req.http.MobileHost + req.url;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2cf9286b8b6e0df40d7391837097b8815086f74e
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Mark Bergsma <[email protected]>
Gerrit-Reviewer: Mark Bergsma <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to