Merge pull request #12364 from MrPetovan/bug/warnings

[friendica.git] / src / Model / APContact.php

diff --git a/src/Model/APContact.php b/src/Model/APContact.php
index d07d24b42b0644aecab77c81e66a2591b1c49630..fd748f1cd4e92419b77513f5cad205a3cec60fcb 100644 (file)
--- a/src/Model/APContact.php
+++ b/src/Model/APContact.php
@@ -24,10 +24,11 @@ namespace Friendica\Model;
 use Friendica\Content\Text\HTML;
 use Friendica\Core\Cache\Enum\Duration;
 use Friendica\Core\Logger;
+use Friendica\Core\Protocol;
 use Friendica\Core\System;
 use Friendica\Database\DBA;
-use Friendica\Database\DBStructure;
 use Friendica\DI;
+use Friendica\Model\Item;
 use Friendica\Network\HTTPClient\Client\HttpClientAccept;
 use Friendica\Network\HTTPException;
 use Friendica\Network\Probe;
@@ -39,6 +40,7 @@ use Friendica\Util\DateTimeFormat;
 use Friendica\Util\HTTPSignature;
 use Friendica\Util\JsonLD;
 use Friendica\Util\Network;
+use GuzzleHttp\Psr7\Uri;
 
 class APContact
 {
@@ -60,7 +62,7 @@ class APContact
                                'addr'      => $local_owner['addr'],
                                'baseurl'   => $local_owner['baseurl'],
                                'url'       => $local_owner['url'],
-                               'subscribe' => $local_owner['baseurl'] . '/follow?url={uri}'];
+                               'subscribe' => $local_owner['baseurl'] . '/contact/follow?url={uri}'];
 
                        if (!empty($local_owner['alias']) && ($local_owner['url'] != $local_owner['alias'])) {
                                $data['alias'] = $local_owner['alias'];
@@ -165,6 +167,8 @@ class APContact
                                return $fetched_contact;
                        }
                        $url = $apcontact['url'];
+               } elseif (empty(parse_url($url, PHP_URL_PATH))) {
+                       $apcontact['baseurl'] = $url;
                }
 
                // Detect multiple fast repeating request to the same address
@@ -287,28 +291,27 @@ class APContact
                        return $fetched_contact;
                }
 
-               $parts = parse_url($apcontact['url']);
-               unset($parts['scheme']);
-               unset($parts['path']);
-
                if (empty($apcontact['addr'])) {
-                       if (!empty($apcontact['nick']) && is_array($parts)) {
-                               $apcontact['addr'] = $apcontact['nick'] . '@' . str_replace('//', '', Network::unparseURL($parts));
-                       } else {
+                       try {
+                               $apcontact['addr'] = $apcontact['nick'] . '@' . (new Uri($apcontact['url']))->getAuthority();
+                       } catch (\Throwable $e) {
+                               Logger::warning('Unable to coerce APContact URL into a UriInterface object', ['url' => $apcontact['url'], 'error' => $e->getMessage()]);
                                $apcontact['addr'] = '';
                        }
                }
 
                $apcontact['pubkey'] = null;
                if (!empty($compacted['w3id:publicKey'])) {
-                       $apcontact['pubkey'] = trim(JsonLD::fetchElement($compacted['w3id:publicKey'], 'w3id:publicKeyPem', '@value'));
-                       if (strstr($apcontact['pubkey'], 'RSA ')) {
+                       $apcontact['pubkey'] = trim(JsonLD::fetchElement($compacted['w3id:publicKey'], 'w3id:publicKeyPem', '@value') ?? '');
+                       if (strpos($apcontact['pubkey'], 'RSA ') !== false) {
                                $apcontact['pubkey'] = Crypto::rsaToPem($apcontact['pubkey']);
                        }
                }
 
                $apcontact['manually-approve'] = (int)JsonLD::fetchElement($compacted, 'as:manuallyApprovesFollowers');
 
+               $apcontact['suspended'] = (int)JsonLD::fetchElement($compacted, 'toot:suspended');
+
                if (!empty($compacted['as:generator'])) {
                        $apcontact['baseurl'] = JsonLD::fetchElement($compacted['as:generator'], 'as:url', '@id');
                        $apcontact['generator'] = JsonLD::fetchElement($compacted['as:generator'], 'as:name', '@value');
@@ -348,17 +351,18 @@ class APContact
 
                if (!empty($apcontact['outbox'])) {
                        if (!empty($local_owner)) {
-                               $outbox = ActivityPub\Transmitter::getOutbox($local_owner);
+                               $statuses_count = self::getStatusesCount($local_owner);
                        } else {
                                $outbox = ActivityPub::fetchContent($apcontact['outbox']);
+                               $statuses_count = $outbox['totalItems'] ?? 0;
                        }
-                       if (!empty($outbox['totalItems'])) {
+                       if (!empty($statuses_count)) {
                                // Mastodon seriously allows for this condition?
                                // Jul 20 2021 - See https://chaos.social/@m11 for a negative posts count
-                               if ($outbox['totalItems'] < 0) {
-                                       $outbox['totalItems'] = 0;
+                               if ($statuses_count < 0) {
+                                       $statuses_count = 0;
                                }
-                               $apcontact['statuses_count'] = $outbox['totalItems'];
+                               $apcontact['statuses_count'] = $statuses_count;
                        }
                }
 
@@ -376,14 +380,14 @@ class APContact
                // kroeg:blocks, updated
 
                // When the photo is too large, try to shorten it by removing parts
-               if (strlen($apcontact['photo']) > 255) {
+               if (strlen($apcontact['photo'] ?? '') > 255) {
                        $parts = parse_url($apcontact['photo']);
                        unset($parts['fragment']);
-                       $apcontact['photo'] = Network::unparseURL($parts);
+                       $apcontact['photo'] = (string)Uri::fromParts($parts);
 
                        if (strlen($apcontact['photo']) > 255) {
                                unset($parts['query']);
-                               $apcontact['photo'] = Network::unparseURL($parts);
+                               $apcontact['photo'] = (string)Uri::fromParts($parts);
                        }
 
                        if (strlen($apcontact['photo']) > 255) {
@@ -479,6 +483,30 @@ class APContact
                return DBA::selectFirst('apcontact', [], ['url' => $apcontact['url']]) ?: [];
        }
 
+       /**
+        * Fetch the number of statuses for the given owner
+        *
+        * @param array $owner
+        *
+        * @return integer
+        */
+       private static function getStatusesCount(array $owner): int
+       {
+               $condition = [
+                       'private'        => [Item::PUBLIC, Item::UNLISTED],
+                       'author-id'      => Contact::getIdForURL($owner['url'], 0, false),
+                       'gravity'        => [Item::GRAVITY_PARENT, Item::GRAVITY_COMMENT],
+                       'network'        => Protocol::DFRN,
+                       'parent-network' => Protocol::FEDERATED,
+                       'deleted'        => false,
+                       'visible'        => true,
+               ];
+
+               $count = Post::countPosts($condition);
+
+               return $count;
+       }
+
        /**
         * Mark the given AP Contact as "to archive"
         *
@@ -537,4 +565,28 @@ class APContact
 
                HTTPSignature::setInboxStatus($url, true, $shared);
        }
+
+       /**
+        * Check if the apcontact is a relay account
+        *
+        * @param array $apcontact
+        *
+        * @return bool
+        */
+       public static function isRelay(array $apcontact): bool
+       {
+               if (empty($apcontact['nick']) || $apcontact['nick'] != 'relay') {
+                       return false;
+               }
+
+               if (!empty($apcontact['type']) && $apcontact['type'] == 'Application') {
+                       return true;
+               }
+
+               if (!empty($apcontact['type']) && in_array($apcontact['type'], ['Group', 'Service']) && is_null($apcontact['outbox'])) {
+                       return true;
+               }
+
+               return false;
+       }
 }