Mark Bergsma has uploaded a new change for review.

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


Change subject: X-Forwarded-Proto is empty for straight http
......................................................................

X-Forwarded-Proto is empty for straight http

Change-Id: I236c25f345965b6551d42778d4480c4ca0860632
---
M templates/varnish/text-frontend.inc.vcl.erb
1 file changed, 5 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/42/75642/1

diff --git a/templates/varnish/text-frontend.inc.vcl.erb 
b/templates/varnish/text-frontend.inc.vcl.erb
index 7fd467d..675ffb4 100644
--- a/templates/varnish/text-frontend.inc.vcl.erb
+++ b/templates/varnish/text-frontend.inc.vcl.erb
@@ -11,7 +11,11 @@
                && req.url ~ "^/wiki/") {
                set req.http.MobileHost = regsub(req.http.host, 
"^(?:www\.(?=mediawiki))?((?:commons|species|meta|incubator|outreach|strategy|wikimania2012|wikimania2013)\.(?=wikimedia)|\w+\.(?!wikimedia|mediawiki)|(?=mediawiki))(wikimedia|wikipedia|wiktionary|wikinews|wikisource|wikiquote|wikibooks|wikiversity|wikivoyage|mediawiki)\.",
 "\1m.\2.");
                if (req.http.Host != req.http.MobileHost) {
-                       set req.http.Location = req.http.X-Forwarded-Proto + 
"://" + req.http.MobileHost + req.url;
+                       if (req.http.X-Forwarded-Proto {
+                               set req.http.Location = 
req.http.X-Forwarded-Proto + "://" + req.http.MobileHost + req.url;
+                       } else {
+                               set req.http.Location = "http://"; + 
req.http.MobileHost + req.url;
+                       }
                        error 666 "Found";
                }
                unset req.http.MobileHost;
@@ -35,12 +39,7 @@
         * do this after vcl_recv_purge, as it operates on a full URI */
        call rewrite_proxy_urls;
 
-       if (!req.http.X-Forwarded-Proto) {
-               set req.http.X-Forwarded-Proto = "http";
-       }
-
        call mobile_redirect;
-
        call pass_requests;
 
        /* Users that just logged out, should not get a 304 for their

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I236c25f345965b6551d42778d4480c4ca0860632
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