]> git.mxchange.org Git - friendica.git/blobdiff - src/Util/Proxy.php
spelling: occurred
[friendica.git] / src / Util / Proxy.php
index 437b52e1e6245488c6f178a5a6b5133956a3f8fa..7f3c94672273dacb5341e6d98d6ef4080fd73c4e 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2010-2021, the Friendica project
+ * @copyright Copyright (C) 2010-2023, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
 
 namespace Friendica\Util;
 
+use Friendica\Core\Logger;
+use Friendica\Core\System;
 use Friendica\DI;
+use GuzzleHttp\Psr7\Uri;
 
 /**
  * Proxy utilities class
  */
 class Proxy
 {
-
-       /**
-        * Default time to keep images in proxy storage
-        */
-       const DEFAULT_TIME = 86400; // 1 Day
-
        /**
         * Sizes constants
         */
@@ -76,55 +73,33 @@ class Proxy
         * Transform a remote URL into a local one.
         *
         * This function only performs the URL replacement on http URL and if the
-        * provided URL isn't local, "the isn't deactivated" (sic) and if the config
-        * system.proxy_disabled is set to false.
+        * provided URL isn't local
         *
         * @param string $url       The URL to proxyfy
-        * @param bool   $writemode Returns a local path the remote URL should be saved to
-        * @param string $size      One of the ProxyUtils::SIZE_* constants
-        *
+        * @param string $size      One of the Proxy::SIZE_* constants
         * @return string The proxyfied URL or relative path
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
-       public static function proxifyUrl($url, $writemode = false, $size = '')
+       public static function proxifyUrl(string $url, string $size = ''): string
        {
-               // Get application instance
-               $a = DI::app();
-
-               // Trim URL first
-               $url = trim($url);
-
-               // Is no http in front of it?
-               /// @TODO To weak test for being a valid URL
-               if (substr($url, 0, 4) !== 'http') {
+               if (!DI::config()->get('system', 'proxify_content')) {
                        return $url;
                }
 
-               // Only continue if it isn't a local image and the isn't deactivated
-               if (self::isLocalImage($url)) {
-                       $url = str_replace(Strings::normaliseLink(DI::baseUrl()) . '/', DI::baseUrl() . '/', $url);
-                       return $url;
-               }
+               // Trim URL first
+               $url = trim($url);
 
-               // Is the proxy disabled?
-               if (DI::config()->get('system', 'proxy_disabled')) {
+               // Quit if not an HTTP/HTTPS link or if local
+               if (!in_array(parse_url($url, PHP_URL_SCHEME), ['http', 'https']) || self::isLocalImage($url)) {
                        return $url;
                }
 
                // Image URL may have encoded ampersands for display which aren't desirable for proxy
                $url = html_entity_decode($url, ENT_NOQUOTES, 'utf-8');
 
-               // Creating a sub directory to reduce the amount of files in the cache directory
-               $basepath = $a->getBasePath() . '/proxy';
-
                $shortpath = hash('md5', $url);
                $longpath = substr($shortpath, 0, 2);
 
-               if (is_dir($basepath) && $writemode && !is_dir($basepath . '/' . $longpath)) {
-                       mkdir($basepath . '/' . $longpath);
-                       chmod($basepath . '/' . $longpath, 0777);
-               }
-
                $longpath .= '/' . strtr(base64_encode($url), '+/', '-_');
 
                // Extract the URL extension
@@ -141,14 +116,11 @@ class Proxy
                        $size = ':' . $size;
                }
 
+               Logger::info('Created proxy link', ['url' => $url, 'callstack' => System::callstack(20)]);
+
                // Too long files aren't supported by Apache
-               // Writemode in combination with long files shouldn't be possible
-               if ((strlen($proxypath) > 250) && $writemode) {
-                       return $shortpath;
-               } elseif (strlen($proxypath) > 250) {
+               if (strlen($proxypath) > 250) {
                        return DI::baseUrl() . '/proxy/' . $shortpath . '?url=' . urlencode($url);
-               } elseif ($writemode) {
-                       return $longpath;
                } else {
                        return $proxypath . $size;
                }
@@ -165,21 +137,22 @@ class Proxy
         * @return string Proxified HTML code
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
-       public static function proxifyHtml($html)
+       public static function proxifyHtml(string $html): string
        {
                $html = str_replace(Strings::normaliseLink(DI::baseUrl()) . '/', DI::baseUrl() . '/', $html);
 
-               return preg_replace_callback('/(<img [^>]*src *= *["\'])([^"\']+)(["\'][^>]*>)/siU', 'self::replaceUrl', $html);
+               return preg_replace_callback('/(<img [^>]*src *= *["\'])([^"\']+)(["\'][^>]*>)/siU', [self::class, 'replaceUrl'], $html);
        }
 
        /**
         * Checks if the URL is a local URL.
         *
         * @param string $url
+        *
         * @return boolean
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
-       public static function isLocalImage($url)
+       public static function isLocalImage(string $url): bool
        {
                if (substr($url, 0, 1) == '/') {
                        return true;
@@ -189,37 +162,38 @@ class Proxy
                        return true;
                }
 
-               // links normalised - bug #431
-               $baseurl = Strings::normaliseLink(DI::baseUrl());
-               $url = Strings::normaliseLink($url);
-
-               return (substr($url, 0, strlen($baseurl)) == $baseurl);
+               return Network::isLocalLink($url);
        }
 
        /**
         * Return the array of query string parameters from a URL
         *
         * @param string $url URL to parse
+        *
         * @return array Associative array of query string parameters
         */
-       private static function parseQuery($url)
+       private static function parseQuery(string $url): array
        {
-               $query = parse_url($url, PHP_URL_QUERY);
-               $query = html_entity_decode($query);
+               try {
+                       $uri = new Uri($url);
 
-               parse_str($query, $arr);
+                       parse_str($uri->getQuery(), $arr);
 
-               return $arr;
+                       return $arr;
+               } catch (\Throwable $e) {
+                       return [];
+               }
        }
 
        /**
         * Call-back method to replace the UR
         *
         * @param array $matches Matches from preg_replace_callback()
+        *
         * @return string Proxified HTML image tag
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
-       private static function replaceUrl(array $matches)
+       private static function replaceUrl(array $matches): string
        {
                // if the picture seems to be from another picture cache then take the original source
                $queryvar = self::parseQuery($matches[2]);