Minor bug fix and workaround a potential GNU Social issue

This commit is contained in:
Diogo Cordeiro 2018-07-06 21:31:21 +01:00
parent 4cb1ed2c9d
commit 4d3321ac21

View File

@ -33,12 +33,25 @@ if (!defined('GNUSOCIAL')) { exit(1); }
class Activitypub_Discovery { class Activitypub_Discovery {
public function lookup ($url) public function lookup ($url)
{ {
// First check if we already have it locally // First check if we already have it locally and, if so, return it
if (($actor_profile = Profile::getKV("profileurl", $url)) != false) { if (($actor_profile = Profile::getKV("profileurl", $url)) != false) {
return $actor_profile; return $actor_profile;
} else {
// Sometimes it is not true that the user is not locally available,
// mostly when it is a local user and URLs slightly changed
// (not sure if this is necessary, but anyway)
// Iff we really are in the same instance
$root_url_len = strlen (common_root_url ());
if (substr ($url, 0, $root_url_len) == common_root_url ()) {
// Grab the nickname and try to get the user
if (($actor_profile = Profile::getKV("nickname", substr ($url, $root_url_len))) != false) {
return $actor_profile;
}
}
} }
// If that's not the case, grab it // If the local fetch fails: grab it remotely, store locally and return
$client = new HTTPClient (); $client = new HTTPClient ();
$headers = array(); $headers = array();
$headers[] = 'Accept: application/ld+json; profile="https://www.w3.org/ns/activitystreams"'; $headers[] = 'Accept: application/ld+json; profile="https://www.w3.org/ns/activitystreams"';
@ -56,7 +69,7 @@ class Activitypub_Discovery {
$profile = new Profile; $profile = new Profile;
$profile->profileurl = $res["url"]; $profile->profileurl = $res["url"];
$profile->nickname = $res["nickname"]; $profile->nickname = $res["nickname"];
$profile->fulname = $res["display_name"]; $profile->fullname = $res["display_name"];
$profile->bio = str_limit($res["summary"], 1000); $profile->bio = str_limit($res["summary"], 1000);
$profile->insert (); $profile->insert ();