Merge pull request #13932 from annando/oembed-cleanup

Unused OEmbed functionality is removed
This commit is contained in:
Hypolite Petovan 2024-02-24 11:03:48 -05:00 committed by GitHub
commit f74d6f9ebb
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
7 changed files with 168 additions and 395 deletions

View File

@ -22,10 +22,9 @@
namespace Friendica\Content; namespace Friendica\Content;
use DOMDocument; use DOMDocument;
use DOMNode;
use DOMText;
use DOMXPath; use DOMXPath;
use Exception; use Exception;
use Friendica\Content\Text\BBCode;
use Friendica\Core\Cache\Enum\Duration; use Friendica\Core\Cache\Enum\Duration;
use Friendica\Core\Hook; use Friendica\Core\Hook;
use Friendica\Core\Renderer; use Friendica\Core\Renderer;
@ -49,32 +48,15 @@ use Friendica\Util\Strings;
*/ */
class OEmbed class OEmbed
{ {
/**
* Callback for fetching URL, checking allowance and returning formatted HTML
*
* @param array $matches
* @return string Formatted HTML
*/
public static function replaceCallback(array $matches): string
{
$embedurl = $matches[1];
$j = self::fetchURL($embedurl, !self::isAllowedURL($embedurl));
$s = self::formatObject($j);
return $s;
}
/** /**
* Get data from an URL to embed its content. * Get data from an URL to embed its content.
* *
* @param string $embedurl The URL from which the data should be fetched. * @param string $embedurl The URL from which the data should be fetched.
* @param bool $no_rich_type If set to true rich type content won't be fetched.
* @param bool $use_parseurl Use the "ParseUrl" functionality to add additional data
* *
* @return \Friendica\Object\OEmbed * @return \Friendica\Object\OEmbed
* @throws \Friendica\Network\HTTPException\InternalServerErrorException * @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/ */
public static function fetchURL(string $embedurl, bool $no_rich_type = false, bool $use_parseurl = true): \Friendica\Object\OEmbed private static function fetchURL(string $embedurl): \Friendica\Object\OEmbed
{ {
$embedurl = trim($embedurl, '\'"'); $embedurl = trim($embedurl, '\'"');
@ -119,7 +101,7 @@ class OEmbed
$href = str_replace(['http://www.youtube.com/', 'http://player.vimeo.com/'], $href = str_replace(['http://www.youtube.com/', 'http://player.vimeo.com/'],
['https://www.youtube.com/', 'https://player.vimeo.com/'], $href); ['https://www.youtube.com/', 'https://player.vimeo.com/'], $href);
$result = DI::httpClient()->fetchFull($href . '&maxwidth=' . $a->getThemeInfoValue('videowidth')); $result = DI::httpClient()->fetchFull($href . '&maxwidth=' . $a->getThemeInfoValue('videowidth'));
if ($result->getReturnCode() === 200) { if ($result->isSuccess()) {
$json_string = $result->getBodyString(); $json_string = $result->getBodyString();
break; break;
} }
@ -157,57 +139,55 @@ class OEmbed
} }
// Improve the OEmbed data with data from OpenGraph, Twitter cards and other sources // Improve the OEmbed data with data from OpenGraph, Twitter cards and other sources
if ($use_parseurl) { $data = ParseUrl::getSiteinfoCached($embedurl);
$data = ParseUrl::getSiteinfoCached($embedurl, false);
if (($oembed->type == 'error') && empty($data['title']) && empty($data['text'])) { if (($oembed->type == 'error') && empty($data['title']) && empty($data['text'])) {
return $oembed; return $oembed;
} }
if ($no_rich_type || ($oembed->type == 'error')) { if (!self::isAllowedURL($embedurl) || ($oembed->type == 'error')) {
$oembed->html = ''; $oembed->html = '';
$oembed->type = $data['type']; $oembed->type = $data['type'];
if ($oembed->type == 'photo') { if ($oembed->type == 'photo') {
if (!empty($data['images'])) { if (!empty($data['images'])) {
$oembed->url = $data['images'][0]['src']; $oembed->url = $data['images'][0]['src'];
$oembed->width = $data['images'][0]['width']; $oembed->width = $data['images'][0]['width'];
$oembed->height = $data['images'][0]['height']; $oembed->height = $data['images'][0]['height'];
} else { } else {
$oembed->type = 'link'; $oembed->type = 'link';
}
} }
} }
}
if (!empty($data['title'])) { if (!empty($data['title'])) {
$oembed->title = $data['title']; $oembed->title = $data['title'];
} }
if (!empty($data['text'])) { if (!empty($data['text'])) {
$oembed->description = $data['text']; $oembed->description = $data['text'];
} }
if (!empty($data['publisher_name'])) { if (!empty($data['publisher_name'])) {
$oembed->provider_name = $data['publisher_name']; $oembed->provider_name = $data['publisher_name'];
} }
if (!empty($data['publisher_url'])) { if (!empty($data['publisher_url'])) {
$oembed->provider_url = $data['publisher_url']; $oembed->provider_url = $data['publisher_url'];
} }
if (!empty($data['author_name'])) { if (!empty($data['author_name'])) {
$oembed->author_name = $data['author_name']; $oembed->author_name = $data['author_name'];
} }
if (!empty($data['author_url'])) { if (!empty($data['author_url'])) {
$oembed->author_url = $data['author_url']; $oembed->author_url = $data['author_url'];
} }
if (!empty($data['images']) && ($oembed->type != 'photo')) { if (!empty($data['images']) && ($oembed->type != 'photo')) {
$oembed->thumbnail_url = $data['images'][0]['src']; $oembed->thumbnail_url = $data['images'][0]['src'];
$oembed->thumbnail_width = $data['images'][0]['width']; $oembed->thumbnail_width = $data['images'][0]['width'];
$oembed->thumbnail_height = $data['images'][0]['height']; $oembed->thumbnail_height = $data['images'][0]['height'];
}
} }
Hook::callAll('oembed_fetch_url', $embedurl, $oembed); Hook::callAll('oembed_fetch_url', $embedurl, $oembed);
@ -219,9 +199,10 @@ class OEmbed
* Returns a formatted string from OEmbed object * Returns a formatted string from OEmbed object
* *
* @param \Friendica\Object\OEmbed $oembed * @param \Friendica\Object\OEmbed $oembed
* @param int $uriid
* @return string * @return string
*/ */
private static function formatObject(\Friendica\Object\OEmbed $oembed): string private static function formatObject(\Friendica\Object\OEmbed $oembed, int $uriid): string
{ {
$ret = '<div class="oembed ' . $oembed->type . '">'; $ret = '<div class="oembed ' . $oembed->type . '">';
@ -241,7 +222,7 @@ class OEmbed
'$escapedhtml' => base64_encode($oembed->html), '$escapedhtml' => base64_encode($oembed->html),
'$tw' => $tw, '$tw' => $tw,
'$th' => $th, '$th' => $th,
'$turl' => $oembed->thumbnail_url, '$turl' => BBCode::proxyUrl($oembed->thumbnail_url, BBCode::INTERNAL, $uriid, Proxy::SIZE_SMALL),
]); ]);
} else { } else {
$ret = $oembed->html; $ret = $oembed->html;
@ -249,14 +230,14 @@ class OEmbed
break; break;
case 'photo': case 'photo':
$ret .= '<img width="' . $oembed->width . '" src="' . Proxy::proxifyUrl($oembed->url) . '">'; $ret .= '<img width="' . $oembed->width . '" src="' . BBCode::proxyUrl($oembed->url, BBCode::INTERNAL, $uriid, Proxy::SIZE_MEDIUM) . '">';
break; break;
case 'link': case 'link':
break; break;
case 'rich': case 'rich':
$ret .= Proxy::proxifyHtml($oembed->html); $ret .= Proxy::proxifyHtml($oembed->html, $uriid);
break; break;
} }
@ -294,12 +275,21 @@ class OEmbed
$ret .= '<a href="' . $oembed->embed_url . '" rel="oembed">' . $oembed->embed_url . '</a>'; $ret .= '<a href="' . $oembed->embed_url . '" rel="oembed">' . $oembed->embed_url . '</a>';
} }
$ret .= "</h4>"; $ret .= "</h4>";
if ($oembed->type == 'link') {
if (!empty($oembed->thumbnail_url)) {
$ret .= '<img width="' . $oembed->width . '" src="' . BBCode::proxyUrl($oembed->thumbnail_url, BBCode::INTERNAL, $uriid, Proxy::SIZE_MEDIUM) . '">';
}
if (!empty($oembed->description)) {
$ret .= '<p>' . $oembed->description . '</p>';
}
}
} elseif (!strpos($oembed->html, $oembed->embed_url)) { } elseif (!strpos($oembed->html, $oembed->embed_url)) {
// add <a> for html2bbcode conversion // add <a> for html2bbcode conversion
$ret .= '<a href="' . $oembed->embed_url . '" rel="oembed">' . $oembed->title . '</a>'; $ret .= '<a href="' . $oembed->embed_url . '" rel="oembed">' . $oembed->title . '</a>';
} }
$ret .= '</div>'; $ret .= '</div>';
$test = Proxy::proxifyHtml($ret, $uriid);
return str_replace("\n", "", $ret); return str_replace("\n", "", $ret);
} }
@ -308,51 +298,19 @@ class OEmbed
* Converts BBCode to HTML code * Converts BBCode to HTML code
* *
* @param string $text * @param string $text
* @param int $uriid
* @return string * @return string
*/ */
public static function BBCode2HTML(string $text): string public static function BBCode2HTML(string $text, int $uriid): string
{ {
if (DI::config()->get('system', 'no_oembed')) { if (!preg_match_all("/\[embed\](.+?)\[\/embed\]/is", $text, $matches, PREG_SET_ORDER)) {
return preg_replace("/\[embed\](.+?)\[\/embed\]/is", "<!-- oembed $1 --><i>" . DI::l10n()->t('Embedding disabled') . " : $1</i><!-- /oembed $1 -->", $text);
}
return preg_replace_callback("/\[embed\](.+?)\[\/embed\]/is", [self::class, 'replaceCallback'], $text);
}
/**
* Find <span class='oembed'>..<a href='url' rel='oembed'>..</a></span>
* and replace it with [embed]url[/embed]
*
* @param string $text
* @return string
*/
public static function HTML2BBCode(string $text): string
{
// start parser only if 'oembed' is in text
if (strpos($text, 'oembed')) {
// convert non ascii chars to html entities
$html_text = mb_convert_encoding($text, 'HTML-ENTITIES', mb_detect_encoding($text));
// If it doesn't parse at all, just return the text.
$dom = new DOMDocument();
if (!@$dom->loadHTML($html_text)) {
return $text;
}
$xpath = new DOMXPath($dom);
$xattr = self::buildXPath('class', 'oembed');
$entries = $xpath->query("//div[$xattr]");
$xattr = "@rel='oembed'"; //oe_build_xpath("rel","oembed");
foreach ($entries as $e) {
$href = $xpath->evaluate("a[$xattr]/@href", $e)->item(0)->nodeValue;
if (!is_null($href)) {
$e->parentNode->replaceChild(new DOMText('[embed]' . $href . '[/embed]'), $e);
}
}
return self::getInnerHTML($dom->getElementsByTagName('body')->item(0));
} else {
return $text; return $text;
} }
foreach ($matches as $match) {
$data = self::fetchURL($match[1]);
$text = str_replace($match[0], self::formatObject($data, $uriid), $text);
}
return $text;
} }
/** /**
@ -373,26 +331,25 @@ class OEmbed
return false; return false;
} }
$str_allowed = DI::config()->get('system', 'allowed_oembed', ''); $allowed = DI::config()->get('system', 'allowed_oembed', '');
if (empty($str_allowed)) { if (empty($allowed)) {
return false; return false;
} }
$allowed = explode(',', $str_allowed); return Network::isDomainMatch($domain, explode(',', $allowed));
return Network::isDomainMatch($domain, $allowed);
} }
/** /**
* Returns a formatted HTML code from given URL and sets optional title * Returns a formatted HTML code from given URL and sets optional title
* *
* @param string $url URL to fetch * @param string $url URL to fetch
* @param string $title Optional title (default: what comes from OEmbed object) * @param string $title title (default: what comes from OEmbed object)
* @param int $uriid
* @return string Formatted HTML * @return string Formatted HTML
*/ */
public static function getHTML(string $url, string $title = ''): string public static function getHTML(string $url, string $title, int $uriid): string
{ {
$o = self::fetchURL($url, !self::isAllowedURL($url)); $o = self::fetchURL($url);
if (!is_object($o) || property_exists($o, 'type') && $o->type == 'error') { if (!is_object($o) || property_exists($o, 'type') && $o->type == 'error') {
throw new Exception('OEmbed failed for URL: ' . $url); throw new Exception('OEmbed failed for URL: ' . $url);
@ -402,74 +359,8 @@ class OEmbed
$o->title = $title; $o->title = $title;
} }
$html = self::formatObject($o); $html = self::formatObject($o, $uriid);
return $html; return $html;
} }
/**
* Generates the iframe HTML for an oembed attachment.
*
* Width and height are given by the remote, and are regularly too small for
* the generated iframe.
*
* The width is entirely discarded for the actual width of the post, while fixed
* height is used as a starting point before the inevitable resizing.
*
* Since the iframe is automatically resized on load, there are no need for ugly
* and impractical scrollbars.
*
* @todo This function is currently unused until someone™ adds support for a separate OEmbed domain
*
* @param string $src Original remote URL to embed
* @param string $width
* @param string $height
* @return string Formatted HTML
*
* @throws \Friendica\Network\HTTPException\InternalServerErrorException
* @see oembed_format_object()
*/
private static function iframe(string $src, string $width, string $height): string
{
if (!$height || strstr($height, '%')) {
$height = '200';
}
$width = '100%';
$src = DI::baseUrl() . '/oembed/' . Strings::base64UrlEncode($src);
return '<iframe onload="resizeIframe(this);" class="embed_rich" height="' . $height . '" width="' . $width . '" src="' . $src . '" allowfullscreen scrolling="no" frameborder="no">' . DI::l10n()->t('Embedded content') . '</iframe>';
}
/**
* Generates attribute search XPath string
*
* Generates an XPath query to select elements whose provided attribute contains
* the provided value in a space-separated list.
*
* @param string $attr Name of the attribute to search
* @param string $value Value to search in a space-separated list
* @return string
*/
private static function buildXPath(string $attr, $value): string
{
// https://www.westhoffswelt.de/blog/2009/6/9/select-html-elements-with-more-than-one-css-class-using-xpath
return "contains(normalize-space(@$attr), ' $value ') or substring(normalize-space(@$attr), 1, string-length('$value') + 1) = '$value ' or substring(normalize-space(@$attr), string-length(@$attr) - string-length('$value')) = ' $value' or @$attr = '$value'";
}
/**
* Returns the inner XML string of a provided DOMNode
*
* @param DOMNode $node
* @return string
*/
private static function getInnerHTML(DOMNode $node): string
{
$innerHTML = '';
$children = $node->childNodes;
foreach ($children as $child) {
$innerHTML .= $child->ownerDocument->saveXML($child);
}
return $innerHTML;
}
} }

View File

@ -310,7 +310,7 @@ class BBCode
return trim($text); return trim($text);
} }
private static function proxyUrl(string $image, int $simplehtml = self::INTERNAL, int $uriid = 0, string $size = ''): string public static function proxyUrl(string $image, int $simplehtml = self::INTERNAL, int $uriid = 0, string $size = ''): string
{ {
// Only send proxied pictures to API and for internal display // Only send proxied pictures to API and for internal display
if (!in_array($simplehtml, [self::INTERNAL, self::MASTODON_API, self::TWITTER_API])) { if (!in_array($simplehtml, [self::INTERNAL, self::MASTODON_API, self::TWITTER_API])) {
@ -453,7 +453,7 @@ class BBCode
$return = ''; $return = '';
try { try {
if ($tryoembed && OEmbed::isAllowedURL($data['url'])) { if ($tryoembed && OEmbed::isAllowedURL($data['url'])) {
$return = OEmbed::getHTML($data['url'], $data['title']); $return = OEmbed::getHTML($data['url'], $data['title'], $uriid);
} else { } else {
throw new Exception('OEmbed is disabled for this attachment.'); throw new Exception('OEmbed is disabled for this attachment.');
} }
@ -1358,12 +1358,12 @@ class BBCode
* $match[1] = $url * $match[1] = $url
* $match[2] = $title or absent * $match[2] = $title or absent
*/ */
$try_oembed_callback = function (array $match) { $try_oembed_callback = function (array $match) use ($uriid) {
$url = $match[1]; $url = $match[1];
$title = $match[2] ?? ''; $title = $match[2] ?? '';
try { try {
$return = OEmbed::getHTML($url, $title); $return = OEmbed::getHTML($url, $title, $uriid);
} catch (Exception $ex) { } catch (Exception $ex) {
$return = $match[0]; $return = $match[0];
} }
@ -1810,7 +1810,7 @@ class BBCode
} }
// oembed tag // oembed tag
$text = OEmbed::BBCode2HTML($text); $text = OEmbed::BBCode2HTML($text, $uriid);
// Avoid triple linefeeds through oembed // Avoid triple linefeeds through oembed
$text = str_replace("<br style='clear:left'></span><br><br>", "<br style='clear:left'></span><br>", $text); $text = str_replace("<br style='clear:left'></span><br><br>", "<br style='clear:left'></span><br>", $text);
@ -2058,9 +2058,6 @@ class BBCode
// Default iframe allowed domains/path // Default iframe allowed domains/path
$allowedIframeDomains = [ $allowedIframeDomains = [
DI::baseUrl()->getHost()
. (DI::baseUrl()->getPath() ? '/' . DI::baseUrl()->getPath() : '')
. '/oembed/', # The path part has to change with the source in Content\Oembed::iframe
'www.youtube.com/embed/', 'www.youtube.com/embed/',
'player.vimeo.com/video/', 'player.vimeo.com/video/',
]; ];

View File

@ -1,74 +0,0 @@
<?php
/**
* @copyright Copyright (C) 2010-2024, the Friendica project
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*
*/
namespace Friendica\Module;
use Friendica\BaseModule;
use Friendica\Content;
use Friendica\Core\System;
use Friendica\DI;
use Friendica\Util\Strings;
/**
* Oembed module
*
* Displays stored embed content based on a base64 hash of a remote URL
*
* Example: /oembed/aHR0cHM6Ly9...
*
* @author Hypolite Petovan <hypolite@mrpetovan.com>
*/
class Oembed extends BaseModule
{
protected function content(array $request = []): string
{
// Unused form: /oembed/b2h?url=...
if (DI::args()->getArgv()[1] == 'b2h') {
$url = ["", trim(hex2bin($_REQUEST['url']))];
echo Content\OEmbed::replaceCallback($url);
System::exit();
}
// Unused form: /oembed/h2b?text=...
if (DI::args()->getArgv()[1] == 'h2b') {
$text = trim(hex2bin($_REQUEST['text']));
echo Content\OEmbed::HTML2BBCode($text);
System::exit();
}
// @TODO: Replace with parameter from router
if (DI::args()->getArgc() == 2) {
echo '<html><body>';
$url = Strings::base64UrlDecode(DI::args()->getArgv()[1]);
$j = Content\OEmbed::fetchURL($url);
// workaround for media.ccc.de (and any other endpoint that return size 0)
if (substr($j->html, 0, 7) == "<iframe" && strstr($j->html, 'width="0"')) {
$j->html = '<style>html,body{margin:0;padding:0;} iframe{width:100%;height:100%;}</style>' . $j->html;
$j->html = str_replace('width="0"', '', $j->html);
$j->html = str_replace('height="0"', '', $j->html);
}
echo $j->html;
echo '</body></html>';
}
System::exit();
}
}

View File

@ -99,8 +99,6 @@ class ParseUrl
* Search for cached embeddable data of an url otherwise fetch it * Search for cached embeddable data of an url otherwise fetch it
* *
* @param string $url The url of the page which should be scraped * @param string $url The url of the page which should be scraped
* @param bool $do_oembed The false option is used by the function fetch_oembed()
* to avoid endless loops
* *
* @return array which contains needed data for embedding * @return array which contains needed data for embedding
* string 'url' => The url of the parsed page * string 'url' => The url of the parsed page
@ -115,7 +113,7 @@ class ParseUrl
* @see ParseUrl::getSiteinfo() for more information about scraping * @see ParseUrl::getSiteinfo() for more information about scraping
* embeddable content * embeddable content
*/ */
public static function getSiteinfoCached(string $url, bool $do_oembed = true): array public static function getSiteinfoCached(string $url): array
{ {
if (empty($url)) { if (empty($url)) {
return [ return [
@ -127,14 +125,14 @@ class ParseUrl
$urlHash = hash('sha256', $url); $urlHash = hash('sha256', $url);
$parsed_url = DBA::selectFirst('parsed_url', ['content'], $parsed_url = DBA::selectFirst('parsed_url', ['content'],
['url_hash' => $urlHash, 'oembed' => $do_oembed] ['url_hash' => $urlHash, 'oembed' => false]
); );
if (!empty($parsed_url['content'])) { if (!empty($parsed_url['content'])) {
$data = unserialize($parsed_url['content']); $data = unserialize($parsed_url['content']);
return $data; return $data;
} }
$data = self::getSiteinfo($url, $do_oembed); $data = self::getSiteinfo($url);
$expires = $data['expires']; $expires = $data['expires'];
@ -144,7 +142,7 @@ class ParseUrl
'parsed_url', 'parsed_url',
[ [
'url_hash' => $urlHash, 'url_hash' => $urlHash,
'oembed' => $do_oembed, 'oembed' => false,
'url' => $url, 'url' => $url,
'content' => serialize($data), 'content' => serialize($data),
'created' => DateTimeFormat::utcNow(), 'created' => DateTimeFormat::utcNow(),
@ -194,7 +192,7 @@ class ParseUrl
* </body> * </body>
* @endverbatim * @endverbatim
*/ */
public static function getSiteinfo(string $url, bool $do_oembed = true, int $count = 1): array public static function getSiteinfo(string $url, int $count = 1): array
{ {
if (empty($url)) { if (empty($url)) {
return [ return [
@ -254,41 +252,6 @@ class ParseUrl
$body = $curlResult->getBodyString(); $body = $curlResult->getBodyString();
if ($do_oembed) {
$oembed_data = OEmbed::fetchURL($url, false, false);
if (!empty($oembed_data->type)) {
if (!in_array($oembed_data->type, ['error', 'rich', 'image', 'video', 'audio', ''])) {
$siteinfo['type'] = $oembed_data->type;
}
// See https://github.com/friendica/friendica/pull/5763#discussion_r217913178
if ($siteinfo['type'] != 'photo') {
if (!empty($oembed_data->title)) {
$siteinfo['title'] = trim($oembed_data->title);
}
if (!empty($oembed_data->description)) {
$siteinfo['text'] = trim($oembed_data->description);
}
if (!empty($oembed_data->author_name)) {
$siteinfo['author_name'] = trim($oembed_data->author_name);
}
if (!empty($oembed_data->author_url)) {
$siteinfo['author_url'] = Network::sanitizeUrl($oembed_data->author_url);
}
if (!empty($oembed_data->provider_name)) {
$siteinfo['publisher_name'] = trim($oembed_data->provider_name);
}
if (!empty($oembed_data->provider_url)) {
$siteinfo['publisher_url'] = Network::sanitizeUrl($oembed_data->provider_url);
}
if (!empty($oembed_data->thumbnail_url)) {
$siteinfo['image'] = $oembed_data->thumbnail_url;
}
}
}
}
$charset = ''; $charset = '';
try { try {
// Look for a charset, first in headers // Look for a charset, first in headers
@ -351,7 +314,7 @@ class ParseUrl
} }
} }
if ($content != '') { if ($content != '') {
$siteinfo = self::getSiteinfo($content, $do_oembed, ++$count); $siteinfo = self::getSiteinfo($content, ++$count);
return $siteinfo; return $siteinfo;
} }
} }

View File

@ -21,8 +21,8 @@
namespace Friendica\Util; namespace Friendica\Util;
use Friendica\Content\Text\BBCode;
use Friendica\Core\Logger; use Friendica\Core\Logger;
use Friendica\Core\System;
use Friendica\DI; use Friendica\DI;
use GuzzleHttp\Psr7\Uri; use GuzzleHttp\Psr7\Uri;
@ -133,15 +133,24 @@ class Proxy
* proxy storage directory. * proxy storage directory.
* *
* @param string $html Un-proxified HTML code * @param string $html Un-proxified HTML code
* @param int $uriid
* *
* @return string Proxified HTML code * @return string Proxified HTML code
* @throws \Friendica\Network\HTTPException\InternalServerErrorException * @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/ */
public static function proxifyHtml(string $html): string public static function proxifyHtml(string $html, int $uriid): string
{ {
$html = str_replace(Strings::normaliseLink(DI::baseUrl()) . '/', DI::baseUrl() . '/', $html); $html = str_replace(Strings::normaliseLink(DI::baseUrl()) . '/', DI::baseUrl() . '/', $html);
return preg_replace_callback('/(<img [^>]*src *= *["\'])([^"\']+)(["\'][^>]*>)/siU', [self::class, 'replaceUrl'], $html); if (!preg_match_all('/(<img [^>]*src *= *["\'])([^"\']+)(["\'][^>]*>)/siU', $html, $matches, PREG_SET_ORDER)) {
return $html;
}
foreach ($matches as $match) {
$html = str_replace($match[0], self::replaceUrl($match, $uriid), $html);
}
return $html;
} }
/** /**
@ -193,7 +202,7 @@ class Proxy
* @return string Proxified HTML image tag * @return string Proxified HTML image tag
* @throws \Friendica\Network\HTTPException\InternalServerErrorException * @throws \Friendica\Network\HTTPException\InternalServerErrorException
*/ */
private static function replaceUrl(array $matches): string private static function replaceUrl(array $matches, int $uriid): string
{ {
// if the picture seems to be from another picture cache then take the original source // if the picture seems to be from another picture cache then take the original source
$queryvar = self::parseQuery($matches[2]); $queryvar = self::parseQuery($matches[2]);
@ -208,7 +217,7 @@ class Proxy
} }
// Return proxified HTML // Return proxified HTML
return $matches[1] . self::proxifyUrl(htmlspecialchars_decode($matches[2])) . $matches[3]; return $matches[1] . BBCode::proxyUrl(htmlspecialchars_decode($matches[2]), BBCode::INTERNAL, $uriid, Proxy::SIZE_MEDIUM) . $matches[3];
} }
public static function getPixelsFromSize(string $size): int public static function getPixelsFromSize(string $size): int

View File

@ -557,11 +557,6 @@ return [
'/objects/{guid}[/{activity}]' => [Module\ActivityPub\Objects::class, [R::GET]], '/objects/{guid}[/{activity}]' => [Module\ActivityPub\Objects::class, [R::GET]],
'/oembed' => [
'/b2h' => [Module\Oembed::class, [R::GET]],
'/h2b' => [Module\Oembed::class, [R::GET]],
'/{hash}' => [Module\Oembed::class, [R::GET]],
],
'/outbox/{nickname}' => [Module\ActivityPub\Outbox::class, [R::GET, R::POST]], '/outbox/{nickname}' => [Module\ActivityPub\Outbox::class, [R::GET, R::POST]],
'/owa' => [Module\Owa::class, [R::GET]], '/owa' => [Module\Owa::class, [R::GET]],
'/openid' => [Module\Security\OpenID::class, [R::GET]], '/openid' => [Module\Security\OpenID::class, [R::GET]],

View File

@ -8,7 +8,7 @@ msgid ""
msgstr "" msgstr ""
"Project-Id-Version: 2024.03-rc\n" "Project-Id-Version: 2024.03-rc\n"
"Report-Msgid-Bugs-To: \n" "Report-Msgid-Bugs-To: \n"
"POT-Creation-Date: 2024-02-19 05:46+0000\n" "POT-Creation-Date: 2024-02-24 15:29+0000\n"
"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n" "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n" "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
"Language-Team: LANGUAGE <LL@li.org>\n" "Language-Team: LANGUAGE <LL@li.org>\n"
@ -71,11 +71,11 @@ msgstr ""
#: src/Module/Settings/Display.php:90 src/Module/Settings/Display.php:199 #: src/Module/Settings/Display.php:90 src/Module/Settings/Display.php:199
#: src/Module/Settings/Profile/Photo/Crop.php:165 #: src/Module/Settings/Profile/Photo/Crop.php:165
#: src/Module/Settings/Profile/Photo/Index.php:110 #: src/Module/Settings/Profile/Photo/Index.php:110
#: src/Module/Settings/RemoveMe.php:119 src/Module/Settings/UserExport.php:80 #: src/Module/Settings/RemoveMe.php:119 src/Module/Settings/UserExport.php:78
#: src/Module/Settings/UserExport.php:114 #: src/Module/Settings/UserExport.php:114
#: src/Module/Settings/UserExport.php:215 #: src/Module/Settings/UserExport.php:213
#: src/Module/Settings/UserExport.php:235 #: src/Module/Settings/UserExport.php:233
#: src/Module/Settings/UserExport.php:300 src/Module/User/Delegation.php:154 #: src/Module/Settings/UserExport.php:298 src/Module/User/Delegation.php:154
#: src/Module/User/Import.php:84 src/Module/User/Import.php:91 #: src/Module/User/Import.php:84 src/Module/User/Import.php:91
msgid "Permission denied." msgid "Permission denied."
msgstr "" msgstr ""
@ -302,7 +302,7 @@ msgstr ""
#: src/Module/Calendar/Event/Form.php:250 src/Module/Contact/Advanced.php:132 #: src/Module/Calendar/Event/Form.php:250 src/Module/Contact/Advanced.php:132
#: src/Module/Contact/Profile.php:364 #: src/Module/Contact/Profile.php:364
#: src/Module/Debug/ActivityPubConversion.php:140 #: src/Module/Debug/ActivityPubConversion.php:140
#: src/Module/Debug/Babel.php:313 src/Module/Debug/Localtime.php:64 #: src/Module/Debug/Babel.php:315 src/Module/Debug/Localtime.php:64
#: src/Module/Debug/Probe.php:54 src/Module/Debug/WebFinger.php:51 #: src/Module/Debug/Probe.php:54 src/Module/Debug/WebFinger.php:51
#: src/Module/FriendSuggest.php:145 src/Module/Install.php:234 #: src/Module/FriendSuggest.php:145 src/Module/Install.php:234
#: src/Module/Install.php:274 src/Module/Install.php:309 #: src/Module/Install.php:274 src/Module/Install.php:309
@ -1080,7 +1080,7 @@ msgstr ""
msgid "Email" msgid "Email"
msgstr "" msgstr ""
#: src/Content/ContactSelector.php:130 src/Module/Debug/Babel.php:307 #: src/Content/ContactSelector.php:130 src/Module/Debug/Babel.php:309
msgid "Diaspora" msgid "Diaspora"
msgstr "" msgstr ""
@ -2174,14 +2174,6 @@ msgstr ""
msgid "Site map" msgid "Site map"
msgstr "" msgstr ""
#: src/Content/OEmbed.php:316
msgid "Embedding disabled"
msgstr ""
#: src/Content/OEmbed.php:440
msgid "Embedded content"
msgstr ""
#: src/Content/Pager.php:216 #: src/Content/Pager.php:216
msgid "first" msgid "first"
msgstr "" msgstr ""
@ -2467,7 +2459,7 @@ msgid "Matrix:"
msgstr "" msgstr ""
#: src/Content/Widget/VCard.php:121 src/Model/Event.php:82 #: src/Content/Widget/VCard.php:121 src/Model/Event.php:82
#: src/Model/Event.php:109 src/Model/Event.php:471 src/Model/Event.php:963 #: src/Model/Event.php:109 src/Model/Event.php:471 src/Model/Event.php:960
#: src/Model/Profile.php:375 src/Module/Contact/Profile.php:406 #: src/Model/Profile.php:375 src/Module/Contact/Profile.php:406
#: src/Module/Directory.php:147 src/Module/Notifications/Introductions.php:187 #: src/Module/Directory.php:147 src/Module/Notifications/Introductions.php:187
#: src/Module/Profile/Profile.php:221 #: src/Module/Profile/Profile.php:221
@ -3343,12 +3335,12 @@ msgid "l F d, Y \\@ g:i A \\G\\M\\TP (e)"
msgstr "" msgstr ""
#: src/Model/Event.php:75 src/Model/Event.php:92 src/Model/Event.php:469 #: src/Model/Event.php:75 src/Model/Event.php:92 src/Model/Event.php:469
#: src/Model/Event.php:945 #: src/Model/Event.php:942
msgid "Starts:" msgid "Starts:"
msgstr "" msgstr ""
#: src/Model/Event.php:78 src/Model/Event.php:98 src/Model/Event.php:470 #: src/Model/Event.php:78 src/Model/Event.php:98 src/Model/Event.php:470
#: src/Model/Event.php:949 #: src/Model/Event.php:946
msgid "Finishes:" msgid "Finishes:"
msgstr "" msgstr ""
@ -3421,20 +3413,20 @@ msgstr ""
msgid "g:i A" msgid "g:i A"
msgstr "" msgstr ""
#: src/Model/Event.php:964 src/Model/Event.php:966 #: src/Model/Event.php:961 src/Model/Event.php:963
msgid "Show map" msgid "Show map"
msgstr "" msgstr ""
#: src/Model/Event.php:965 #: src/Model/Event.php:962
msgid "Hide map" msgid "Hide map"
msgstr "" msgstr ""
#: src/Model/Event.php:1058 #: src/Model/Event.php:1055
#, php-format #, php-format
msgid "%s's birthday" msgid "%s's birthday"
msgstr "" msgstr ""
#: src/Model/Event.php:1059 #: src/Model/Event.php:1056
#, php-format #, php-format
msgid "Happy Birthday %s" msgid "Happy Birthday %s"
msgstr "" msgstr ""
@ -6157,11 +6149,11 @@ msgstr ""
msgid "Failed to remove event" msgid "Failed to remove event"
msgstr "" msgstr ""
#: src/Module/Calendar/Event/API.php:186 src/Module/Calendar/Event/API.php:188 #: src/Module/Calendar/Event/API.php:187 src/Module/Calendar/Event/API.php:189
msgid "Event can not end before it has started." msgid "Event can not end before it has started."
msgstr "" msgstr ""
#: src/Module/Calendar/Event/API.php:195 src/Module/Calendar/Event/API.php:197 #: src/Module/Calendar/Event/API.php:196 src/Module/Calendar/Event/API.php:198
msgid "Event title and start time are required." msgid "Event title and start time are required."
msgstr "" msgstr ""
@ -7072,33 +7064,33 @@ msgstr ""
msgid "Not available." msgid "Not available."
msgstr "" msgstr ""
#: src/Module/Conversation/Network.php:202 #: src/Module/Conversation/Network.php:200
msgid "No such circle" msgid "No such circle"
msgstr "" msgstr ""
#: src/Module/Conversation/Network.php:206 #: src/Module/Conversation/Network.php:204
#, php-format #, php-format
msgid "Circle: %s" msgid "Circle: %s"
msgstr "" msgstr ""
#: src/Module/Conversation/Network.php:225 #: src/Module/Conversation/Network.php:223
#, php-format #, php-format
msgid "Error %d (%s) while fetching the timeline." msgid "Error %d (%s) while fetching the timeline."
msgstr "" msgstr ""
#: src/Module/Conversation/Network.php:302 #: src/Module/Conversation/Network.php:300
msgid "Network feed not available." msgid "Network feed not available."
msgstr "" msgstr ""
#: src/Module/Conversation/Timeline.php:194 #: src/Module/Conversation/Timeline.php:196
msgid "Own Contacts" msgid "Own Contacts"
msgstr "" msgstr ""
#: src/Module/Conversation/Timeline.php:198 #: src/Module/Conversation/Timeline.php:200
msgid "Include" msgid "Include"
msgstr "" msgstr ""
#: src/Module/Conversation/Timeline.php:199 #: src/Module/Conversation/Timeline.php:201
msgid "Hide" msgid "Hide"
msgstr "" msgstr ""
@ -7130,7 +7122,7 @@ msgid "Result Item"
msgstr "" msgstr ""
#: src/Module/Debug/ActivityPubConversion.php:129 #: src/Module/Debug/ActivityPubConversion.php:129
#: src/Module/Debug/Babel.php:293 src/Module/Moderation/Item/Source.php:87 #: src/Module/Debug/Babel.php:294 src/Module/Moderation/Item/Source.php:87
#: src/Module/Security/TwoFactor/Verify.php:98 #: src/Module/Security/TwoFactor/Verify.php:98
msgid "Error" msgid "Error"
msgid_plural "Errors" msgid_plural "Errors"
@ -7141,179 +7133,179 @@ msgstr[1] ""
msgid "Source activity" msgid "Source activity"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:51 #: src/Module/Debug/Babel.php:52
msgid "Source input" msgid "Source input"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:57 #: src/Module/Debug/Babel.php:58
msgid "BBCode::toPlaintext" msgid "BBCode::toPlaintext"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:63 #: src/Module/Debug/Babel.php:64
msgid "BBCode::convert (raw HTML)" msgid "BBCode::convert (raw HTML)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:68 #: src/Module/Debug/Babel.php:69
msgid "BBCode::convert (hex)" msgid "BBCode::convert (hex)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:73 #: src/Module/Debug/Babel.php:74
msgid "BBCode::convert" msgid "BBCode::convert"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:79 #: src/Module/Debug/Babel.php:80
msgid "BBCode::convert => HTML::toBBCode" msgid "BBCode::convert => HTML::toBBCode"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:85 #: src/Module/Debug/Babel.php:86
msgid "BBCode::toMarkdown" msgid "BBCode::toMarkdown"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:91 #: src/Module/Debug/Babel.php:92
msgid "BBCode::toMarkdown => Markdown::convert (raw HTML)" msgid "BBCode::toMarkdown => Markdown::convert (raw HTML)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:95 #: src/Module/Debug/Babel.php:96
msgid "BBCode::toMarkdown => Markdown::convert" msgid "BBCode::toMarkdown => Markdown::convert"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:101 #: src/Module/Debug/Babel.php:102
msgid "BBCode::toMarkdown => Markdown::toBBCode" msgid "BBCode::toMarkdown => Markdown::toBBCode"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:107 #: src/Module/Debug/Babel.php:108
msgid "BBCode::toMarkdown => Markdown::convert => HTML::toBBCode" msgid "BBCode::toMarkdown => Markdown::convert => HTML::toBBCode"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:115 #: src/Module/Debug/Babel.php:116
msgid "Item Body" msgid "Item Body"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:119 #: src/Module/Debug/Babel.php:120
msgid "Item Tags" msgid "Item Tags"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:125 #: src/Module/Debug/Babel.php:126
msgid "PageInfo::appendToBody" msgid "PageInfo::appendToBody"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:130 #: src/Module/Debug/Babel.php:131
msgid "PageInfo::appendToBody => BBCode::convert (raw HTML)" msgid "PageInfo::appendToBody => BBCode::convert (raw HTML)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:134 #: src/Module/Debug/Babel.php:135
msgid "PageInfo::appendToBody => BBCode::convert" msgid "PageInfo::appendToBody => BBCode::convert"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:141 #: src/Module/Debug/Babel.php:142
msgid "Source input (Diaspora format)" msgid "Source input (Diaspora format)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:150 #: src/Module/Debug/Babel.php:151
msgid "Source input (Markdown)" msgid "Source input (Markdown)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:156 #: src/Module/Debug/Babel.php:157
msgid "Markdown::convert (raw HTML)" msgid "Markdown::convert (raw HTML)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:161 #: src/Module/Debug/Babel.php:162
msgid "Markdown::convert" msgid "Markdown::convert"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:167 #: src/Module/Debug/Babel.php:168
msgid "Markdown::toBBCode" msgid "Markdown::toBBCode"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:174 #: src/Module/Debug/Babel.php:175
msgid "Raw HTML input" msgid "Raw HTML input"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:179 #: src/Module/Debug/Babel.php:180
msgid "HTML Input" msgid "HTML Input"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:186 #: src/Module/Debug/Babel.php:187
msgid "HTML Purified (raw)" msgid "HTML Purified (raw)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:191 #: src/Module/Debug/Babel.php:192
msgid "HTML Purified (hex)" msgid "HTML Purified (hex)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:196 #: src/Module/Debug/Babel.php:197
msgid "HTML Purified" msgid "HTML Purified"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:202 #: src/Module/Debug/Babel.php:203
msgid "HTML::toBBCode" msgid "HTML::toBBCode"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:208 #: src/Module/Debug/Babel.php:209
msgid "HTML::toBBCode => BBCode::convert" msgid "HTML::toBBCode => BBCode::convert"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:213 #: src/Module/Debug/Babel.php:214
msgid "HTML::toBBCode => BBCode::convert (raw HTML)" msgid "HTML::toBBCode => BBCode::convert (raw HTML)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:219 #: src/Module/Debug/Babel.php:220
msgid "HTML::toBBCode => BBCode::toPlaintext" msgid "HTML::toBBCode => BBCode::toPlaintext"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:225 #: src/Module/Debug/Babel.php:226
msgid "HTML::toMarkdown" msgid "HTML::toMarkdown"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:231 #: src/Module/Debug/Babel.php:232
msgid "HTML::toPlaintext" msgid "HTML::toPlaintext"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:237 #: src/Module/Debug/Babel.php:238
msgid "HTML::toPlaintext (compact)" msgid "HTML::toPlaintext (compact)"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:255 #: src/Module/Debug/Babel.php:256
msgid "Decoded post" msgid "Decoded post"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:276 #: src/Module/Debug/Babel.php:277
msgid "Post array before expand entities" msgid "Post array before expand entities"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:283 #: src/Module/Debug/Babel.php:284
msgid "Post converted" msgid "Post converted"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:288 #: src/Module/Debug/Babel.php:289
msgid "Converted body" msgid "Converted body"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:294 #: src/Module/Debug/Babel.php:295
msgid "Twitter addon is absent from the addon/ folder." msgid "Twitter addon is absent from the addon/ folder."
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:304 #: src/Module/Debug/Babel.php:305
msgid "Babel Diagnostic" msgid "Babel Diagnostic"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:305 #: src/Module/Debug/Babel.php:307
msgid "Source text" msgid "Source text"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:306 #: src/Module/Debug/Babel.php:308
msgid "BBCode" msgid "BBCode"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:308 #: src/Module/Debug/Babel.php:310
msgid "Markdown" msgid "Markdown"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:309 #: src/Module/Debug/Babel.php:311
msgid "HTML" msgid "HTML"
msgstr "" msgstr ""
#: src/Module/Debug/Babel.php:311 #: src/Module/Debug/Babel.php:313
msgid "Twitter Source / Tweet URL (requires API key)" msgid "Twitter Source / Tweet URL (requires API key)"
msgstr "" msgstr ""
@ -9023,21 +9015,21 @@ msgstr ""
msgid "<b>Attributed To:</b> %s<br>" msgid "<b>Attributed To:</b> %s<br>"
msgstr "" msgstr ""
#: src/Module/Photo.php:130 #: src/Module/Photo.php:123
msgid "The Photo is not available." msgid "The Photo is not available."
msgstr "" msgstr ""
#: src/Module/Photo.php:155 #: src/Module/Photo.php:148
#, php-format #, php-format
msgid "The Photo with id %s is not available." msgid "The Photo with id %s is not available."
msgstr "" msgstr ""
#: src/Module/Photo.php:196 #: src/Module/Photo.php:189
#, php-format #, php-format
msgid "Invalid external resource with url %s." msgid "Invalid external resource with url %s."
msgstr "" msgstr ""
#: src/Module/Photo.php:198 #: src/Module/Photo.php:191
#, php-format #, php-format
msgid "Invalid photo with id %s." msgid "Invalid photo with id %s."
msgstr "" msgstr ""