Hopefully better clarification why ESI includes need to be http:// URLs

Author: vvuksan

Block/GeoIp/GetAction.php

@@ -108,7 +108,9 @@ protected function _toHtml() // @codingStandardsIgnoreLine - required by parent
         if (empty($header)) {
             $this->response->setHeader("x-esi", "1");
         }
-        // HTTPS ESIs are not supported so we need to turn them into HTTP
+        // Due to Varnish parser limitations HTTPS ESIs are not supported so we need to turn them into HTTP URLs
+        // This does not mean that request will go over HTTP. ESI subrequest will go out to the backend that is 
+        // currently specified so if it's HTTPS it will go over HTTPS
         return sprintf(
             '<esi:include src=\'%s\' />',
             preg_replace(