X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;ds=sidebyside;f=src%2FUtil%2FProxy.php;h=dcc46115588af775e53d1feb16861afa93b615e9;hb=fca93793480839660870e899bea88f3637859fff;hp=839442797ef8ed3b7a29bb3df63874266952fe7e;hpb=e519b782fd2180d326ffbd9fc8f1d6fca111ce98;p=friendica.git diff --git a/src/Util/Proxy.php b/src/Util/Proxy.php index 839442797e..dcc4611558 100644 --- a/src/Util/Proxy.php +++ b/src/Util/Proxy.php @@ -21,6 +21,8 @@ namespace Friendica\Util; +use Friendica\Core\Logger; +use Friendica\Core\System; use Friendica\DI; /** @@ -28,12 +30,6 @@ use Friendica\DI; */ class Proxy { - - /** - * Default time to keep images in proxy storage - */ - const DEFAULT_TIME = 86400; // 1 Day - /** * Sizes constants */ @@ -76,37 +72,25 @@ 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 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, $size = '') { - // 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; } @@ -132,6 +116,8 @@ class Proxy $size = ':' . $size; } + Logger::info('Created proxy link', ['url' => $url, 'callstack' => System::callstack(20)]); + // Too long files aren't supported by Apache if (strlen($proxypath) > 250) { return DI::baseUrl() . '/proxy/' . $shortpath . '?url=' . urlencode($url); @@ -175,11 +161,7 @@ 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); } /**