keith/friendica
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Simplify the probing process

Browse source
This commit is contained in:
Michael 2020-06-10 19:19:10 +00:00
parent 59bd6fd908
commit e49de92cd4
1 changed files with 17 additions and 63 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

80
src/Network/Probe.php
View file

@ -536,50 +536,6 @@ class Probe
return false; return false;
} }
/**
* Checks if a profile url should be OStatus but only provides partial information
*
* @param array $webfinger Webfinger data
* @param string $lrdd Path template for webfinger request
* @param string $type type
*
* @return array fixed webfinger data
* @throws HTTPException\InternalServerErrorException
*/
private static function fixOStatus($webfinger, $lrdd, $type)
{
if (empty($webfinger['links']) || empty($webfinger['subject'])) {
return $webfinger;
}
$is_ostatus = false;
$has_key = false;
foreach ($webfinger['links'] as $link) {
if ($link['rel'] == ActivityNamespace::OSTATUSSUB) {
$is_ostatus = true;
}
if ($link['rel'] == 'magic-public-key') {
$has_key = true;
}
}
if (!$is_ostatus || $has_key) {
return $webfinger;
}
$url = Network::switchScheme($webfinger['subject']);
$path = str_replace('{uri}', urlencode($url), $lrdd);
$webfinger2 = self::webfinger($path, $type);
// Is the new webfinger detectable as OStatus?
if (self::ostatus($webfinger2, true)) {
$webfinger = $webfinger2;
}
return $webfinger;
}
/** /**
* Fetch the "subscribe" and add it to the result * Fetch the "subscribe" and add it to the result
* *
@ -706,29 +662,28 @@ class Probe
continue; continue;
} }
// At first try it with the given uri // Try the URI first
$path = str_replace('{uri}', urlencode($uri), $template); if ($uri != $addr) {
$webfinger = self::webfinger($path, $type); $path = str_replace('{uri}', urlencode($uri), $template);
$webfinger = self::webfinger($path, $type);
// Fix possible problems with GNU Social probing to wrong scheme
$webfinger = self::fixOStatus($webfinger, $template, $type);
// We cannot be sure that the detected address was correct, so we don't use the values
if ($webfinger && ($uri != $addr)) {
$nick = "";
$addr = "";
} }
// Try webfinger with the address (user@domain.tld) // Then try the address
if (!$webfinger) {
$path = str_replace('{uri}', urlencode("acct:" . $addr), $template);
$webfinger = self::webfinger($path, $type);
}
// Finally try without the "acct"
if (!$webfinger) { if (!$webfinger) {
$path = str_replace('{uri}', urlencode($addr), $template); $path = str_replace('{uri}', urlencode($addr), $template);
$webfinger = self::webfinger($path, $type); $webfinger = self::webfinger($path, $type);
} }
// Mastodon needs to have it with "acct:" // We cannot be sure that the detected address was correct, so we don't use the values
if (!$webfinger) { if ($webfinger && ($uri != $addr)) {
$path = str_replace('{uri}', urlencode("acct:".$addr), $template); $nick = "";
$webfinger = self::webfinger($path, $type); $addr = "";
} }
} }
@ -1511,7 +1466,7 @@ class Probe
&& (($link["type"] ?? "") == "text/html") && (($link["type"] ?? "") == "text/html")
&& ($link["href"] != "") && ($link["href"] != "")
) { ) {
$data["url"] = $link["href"]; $data["url"] = $data["alias"] = $link["href"];
} elseif (($link["rel"] == "salmon") && !empty($link["href"])) { } elseif (($link["rel"] == "salmon") && !empty($link["href"])) {
$data["notify"] = $link["href"]; $data["notify"] = $link["href"];
} elseif (($link["rel"] == ActivityNamespace::FEED) && !empty($link["href"])) { } elseif (($link["rel"] == ActivityNamespace::FEED) && !empty($link["href"])) {
@ -1594,8 +1549,7 @@ class Probe
$data["url"] = $feed_data["header"]["author-link"]; $data["url"] = $feed_data["header"]["author-link"];
} }
if (($data['poll'] == $data['url']) && ($data["alias"] != '')) { if ($data["url"] == $data["alias"]) {
$data['url'] = $data["alias"];
$data["alias"] = ''; $data["alias"] = '';
} }