From 4c0e5674eb42c71dc74d1a7d1afb6c1668d64bee Mon Sep 17 00:00:00 2001
From: Phu Hoang <hmphu.it@gmail.com>
Date: Thu, 17 Nov 2016 20:03:05 +0700
Subject: [PATCH 1/2] Correct varnish v3 template

---
 .../Model/Varnish/Configurator/Abstract.php         | 13 ++++++++++++-
 .../Nexcessnet/Turpentine/misc/version-3.vcl        |  8 ++++----
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php
index 77e4ec675..f83c1bfd4 100644
--- a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php
+++ b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php
@@ -910,11 +910,22 @@ protected function _vcl_sub_https_redirect_fix() {
         $baseUrl = str_replace(array('http://','https://'), '', $baseUrl);
         $baseUrl = rtrim($baseUrl,'/');
         
-        $tpl = <<<EOS
+        switch (Mage::getStoreConfig('turpentine_varnish/servers/version')) {
+            case 4.0:
+                $tpl = <<<EOS
 if ( (req.http.host ~ "^(?i)www.$baseUrl" || req.http.host ~ "^(?i)$baseUrl") && req.http.X-Forwarded-Proto !~ "(?i)https") {
         return (synth(750, ""));
     }
 EOS;
+                break;
+            default:
+                $tpl = <<<EOS
+if ( (req.http.host ~ "^(?i)www.$baseUrl" || req.http.host ~ "^(?i)$baseUrl") && req.http.X-Forwarded-Proto !~ "(?i)https") {
+        error 750 "https://" + req.http.host + req.url;
+    }
+EOS;
+        }
+
         return $tpl;
     }
 
diff --git a/app/code/community/Nexcessnet/Turpentine/misc/version-3.vcl b/app/code/community/Nexcessnet/Turpentine/misc/version-3.vcl
index b960df79a..a391f2d8d 100644
--- a/app/code/community/Nexcessnet/Turpentine/misc/version-3.vcl
+++ b/app/code/community/Nexcessnet/Turpentine/misc/version-3.vcl
@@ -96,10 +96,10 @@ sub generate_session_expires {
 {{generate_session_end}}
 ## Varnish Subroutines
 
-sub vcl_synth {
-    if (resp.status == 750) {
-        set resp.status = 301;
-        set resp.http.Location = "https://" + req.http.host + req.url;
+sub vcl_error {
+    if (obj.status == 750) {
+        set obj.http.Location = obj.response;
+        set obj.status = 301;
         return(deliver);
     }
 }

From 0d5287849a87412399b9066e1eecb856e9e12ff1 Mon Sep 17 00:00:00 2001
From: Phu Hoang <hmphu.it@gmail.com>
Date: Thu, 17 Nov 2016 20:49:10 +0700
Subject: [PATCH 2/2] Migrate vcl_synth subroutine for Varnish v4

---
 .../Model/Varnish/Configurator/Abstract.php   | 37 +++++++++++++++++++
 .../Nexcessnet/Turpentine/misc/version-4.vcl  |  8 ----
 2 files changed, 37 insertions(+), 8 deletions(-)

diff --git a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php
index f83c1bfd4..f539e2b41 100644
--- a/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php
+++ b/app/code/community/Nexcessnet/Turpentine/Model/Varnish/Configurator/Abstract.php
@@ -969,6 +969,40 @@ protected function _vcl_sub_synth()
             'vcl_synth_content' => Mage::getStoreConfig('turpentine_vcl/maintenance/custom_vcl_synth')));
     }
 
+    /**
+     * vcl_synth for fixing https
+     *
+     * @return string
+     */
+    protected function _vcl_sub_synth_https_fix()
+    {
+        $tpl = $this->_vcl_sub_synth();
+
+        if(!$tpl){
+            $tpl = <<<EOS
+sub vcl_synth {
+    if (resp.status == 750) {
+        set resp.status = 301;
+        set resp.http.Location = "https://" + req.http.host + req.url;
+        return(deliver);
+    }
+}
+EOS;
+        }else{
+            $tpl_750 = '
+sub vcl_synth {
+    if (resp.status == 750) {
+        set resp.status = 301;
+        set resp.http.Location = "https://" + req.http.host + req.url;
+        return(deliver);
+    }';
+
+        $tpl = str_ireplace('sub vcl_synth {', $tpl_750, $tpl);
+        }
+
+        return $tpl;
+    }
+
 
 
     /**
@@ -1051,6 +1085,9 @@ protected function _getTemplateVars() {
         
         if (Mage::getStoreConfig('turpentine_varnish/general/https_redirect_fix')) {
             $vars['https_redirect'] = $this->_vcl_sub_https_redirect_fix();
+            if(Mage::getStoreConfig('turpentine_varnish/servers/version') == '4.0'){
+                $vars['vcl_synth'] = $this->_vcl_sub_synth_https_fix();
+            }
         }
 
         foreach (array('','top') as $position) {
diff --git a/app/code/community/Nexcessnet/Turpentine/misc/version-4.vcl b/app/code/community/Nexcessnet/Turpentine/misc/version-4.vcl
index efef43060..5b5758b8d 100644
--- a/app/code/community/Nexcessnet/Turpentine/misc/version-4.vcl
+++ b/app/code/community/Nexcessnet/Turpentine/misc/version-4.vcl
@@ -102,14 +102,6 @@ sub generate_session_expires {
 {{generate_session_end}}
 ## Varnish Subroutines
 
-sub vcl_synth {
-    if (resp.status == 750) {
-        set resp.status = 301;
-        set resp.http.Location = "https://" + req.http.host + req.url;
-        return(deliver);
-    }
-}
-
 sub vcl_init {
     {{directors}}
 }