From: Mikael Nordfeldth <mmn@hethane.se>
Date: Mon, 12 Jan 2015 17:56:19 +0000 (+0100)
Subject: When called in offline queue, File::url was HTTP
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=cbb7ec07a58e44ffa4f98328511869eee2d6cd43;p=quix0rs-gnu-social.git

When called in offline queue, File::url was HTTP

Despite having the site configured "always" for HTTPS, File generated
thumbnails and such with HTTP urls.
---

diff --git a/classes/File.php b/classes/File.php
index 8eff7d6182..1edc89d5d7 100644
--- a/classes/File.php
+++ b/classes/File.php
@@ -303,7 +303,7 @@ class File extends Managed_DataObject
 
         }
 
-        if (StatusNet::isHTTPS()) {
+        if (StatusNet::useHTTPS()) {
 
             $sslserver = common_config('attachments', 'sslserver');
 
diff --git a/lib/statusnet.php b/lib/statusnet.php
index a9b05388c3..844a15c339 100644
--- a/lib/statusnet.php
+++ b/lib/statusnet.php
@@ -423,11 +423,20 @@ class StatusNet
     static function isHTTPS()
     {
         // There are some exceptions to this; add them here!
-        if(empty($_SERVER['HTTPS'])) {
+        if (empty($_SERVER['HTTPS'])) {
             return false;
-        } else {
-            return $_SERVER['HTTPS'] !== 'off';
         }
+
+        // If it is _not_ "off", it is on, so "true".
+        return strtolower($_SERVER['HTTPS']) !== 'off';
+    }
+
+    /**
+     * Can we use HTTPS? Then do! Only return false if it's not configured ("never").
+     */
+    static function useHTTPS()
+    {
+        return self::isHTTPS() || common_config('site', 'ssl') != 'never';
     }
 }