]> git.mxchange.org Git - friendica.git/blobdiff - src/Protocol/ActivityPub/Receiver.php
Improved relay post processing
[friendica.git] / src / Protocol / ActivityPub / Receiver.php
index 204e631c9bf01196f81542d6cc7d986edfcf623d..bb671003da58d292bbd0b0a76ed49032d8572848 100644 (file)
@@ -80,6 +80,7 @@ class Receiver
        const COMPLETION_RELAY    = 2;
        const COMPLETION_MANUAL   = 3;
        const COMPLETION_AUTO     = 4;
+       const COMPLETION_ASYNC    = 5;
 
        /**
         * Checks incoming message from the inbox
@@ -107,7 +108,7 @@ class Receiver
                if (empty($apcontact)) {
                        Logger::notice('Unable to retrieve AP contact for actor - message is discarded', ['actor' => $actor]);
                        return;
-               } elseif (APContact::isRelay($apcontact)) {
+               } elseif (APContact::isRelay($apcontact) && self::isRelayPost($ldactivity)) {
                        self::processRelayPost($ldactivity, $actor);
                        return;
                } else {
@@ -115,7 +116,7 @@ class Receiver
                }
 
                $sig_contact = HTTPSignature::getKeyIdContact($header);
-               if (APContact::isRelay($sig_contact)) {
+               if (APContact::isRelay($sig_contact) && self::isRelayPost($ldactivity)) {
                        Logger::info('Message from a relay', ['url' => $sig_contact['url']]);
                        self::processRelayPost($ldactivity, $sig_contact['url']);
                        return;
@@ -169,26 +170,23 @@ class Receiver
        }
 
        /**
-        * Process incoming posts from relays
+        * Check if the activity is a post rhat can be send via a relay
         *
-        * @param array  $activity
-        * @param string $actor
-        * @return void
+        * @param array $activity
+        * @return boolean
         */
-       private static function processRelayPost(array $activity, string $actor)
+       private static function isRelayPost(array $activity): bool
        {
                $type = JsonLD::fetchElement($activity, '@type');
                if (!$type) {
-                       Logger::notice('Empty type', ['activity' => $activity, 'actor' => $actor]);
-                       return;
+                       return false;
                }
 
                $object_type = JsonLD::fetchElement($activity, 'as:object', '@type') ?? '';
 
                $object_id = JsonLD::fetchElement($activity, 'as:object', '@id');
                if (empty($object_id)) {
-                       Logger::notice('No object id found', ['type' => $type, 'object_type' => $object_type, 'actor' => $actor, 'activity' => $activity]);
-                       return;
+                       return false;
                }
 
                $handle = ($type == 'as:Announce');
@@ -196,26 +194,29 @@ class Receiver
                if (!$handle && in_array($type, ['as:Create', 'as:Update'])) {
                        $handle = in_array($object_type, self::CONTENT_TYPES);
                }
+               return $handle;
+       }
 
-               if (!$handle) {
-                       $trust_source = false;
-                       $object_data = self::prepareObjectData($activity, 0, false, $trust_source);
-
-                       if (!$trust_source) {
-                               Logger::notice('Activity trust could not be achieved.',  ['type' => $type, 'object_type' => $object_type, 'object_id' => $object_id, 'actor' => $actor, 'activity' => $activity]);
-                               return;
-                       }
+       /**
+        * Process incoming posts from relays
+        *
+        * @param array  $activity
+        * @param string $actor
+        * @return void
+        */
+       private static function processRelayPost(array $activity, string $actor)
+       {
+               $type = JsonLD::fetchElement($activity, '@type');
+               if (!$type) {
+                       Logger::notice('Empty type', ['activity' => $activity, 'actor' => $actor]);
+                       return;
+               }
 
-                       if (empty($object_data)) {
-                               Logger::notice('No object data found', ['type' => $type, 'object_type' => $object_type, 'object_id' => $object_id, 'actor' => $actor, 'activity' => $activity]);
-                               return;
-                       }
+               $object_type = JsonLD::fetchElement($activity, 'as:object', '@type') ?? '';
 
-                       if (self::routeActivities($object_data, $type, true)) {
-                               Logger::debug('Handled activity', ['type' => $type, 'object_type' => $object_type, 'object_id' => $object_id, 'actor' => $actor]);
-                       } else {
-                               Logger::info('Unhandled activity', ['type' => $type, 'object_type' => $object_type, 'object_id' => $object_id, 'actor' => $actor, 'activity' => $activity]);
-                       }
+               $object_id = JsonLD::fetchElement($activity, 'as:object', '@id');
+               if (empty($object_id)) {
+                       Logger::notice('No object id found', ['type' => $type, 'object_type' => $object_type, 'actor' => $actor, 'activity' => $activity]);
                        return;
                }
 
@@ -230,7 +231,7 @@ class Receiver
                        return;
                }
 
-               Logger::debug('Got relayed message id', ['id' => $object_id, 'actor' => $actor]);
+               Logger::debug('Process post from relay server', ['type' => $type, 'object_type' => $object_type, 'object_id' => $object_id, 'actor' => $actor]);
 
                $item_id = Item::searchByLink($object_id);
                if ($item_id) {
@@ -238,10 +239,21 @@ class Receiver
                        return;
                }
 
-               $id = Processor::fetchMissingActivity($object_id, [], $actor, self::COMPLETION_RELAY);
-               if (empty($id)) {
-                       Logger::notice('Relayed message had not been fetched', ['id' => $object_id, 'actor' => $actor]);
-                       return;
+               if (!DI::config()->get('system', 'decoupled_receiver')) {
+                       $id = Processor::fetchMissingActivity($object_id, [], $actor, self::COMPLETION_RELAY);
+                       if (!empty($id)) {
+                               Logger::notice('Relayed message is fetched', ['result' => $id, 'id' => $object_id, 'actor' => $actor]);
+                       } else {
+                               Logger::notice('Relayed message had not been fetched', ['id' => $object_id, 'actor' => $actor, 'activity' => $activity]);
+                       }
+               } elseif (!Fetch::hasWorker($object_id)) {
+                       Logger::notice('Fetching is done by worker.', ['id' => $object_id]);
+                       Fetch::add($object_id);
+                       $activity['recursion-depth'] = 0;
+                       $wid = Worker::add(Worker::PRIORITY_HIGH, 'FetchMissingActivity', $object_id, [], $actor, self::COMPLETION_RELAY);
+                       Fetch::setWorkerId($object_id, $wid);
+               } else {
+                       Logger::debug('Activity will already be fetched via a worker.', ['url' => $object_id]);
                }
        }
 
@@ -291,16 +303,17 @@ class Receiver
        /**
         * Prepare the object array
         *
-        * @param array   $activity     Array with activity data
-        * @param integer $uid          User ID
-        * @param boolean $push         Message had been pushed to our system
-        * @param boolean $trust_source Do we trust the source?
+        * @param array   $activity       Array with activity data
+        * @param integer $uid            User ID
+        * @param boolean $push           Message had been pushed to our system
+        * @param boolean $trust_source   Do we trust the source?
+        * @param string  $original_actor Actor of the original activity. Used for receiver detection. (Optional)
         *
         * @return array with object data
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       public static function prepareObjectData(array $activity, int $uid, bool $push, bool &$trust_source): array
+       public static function prepareObjectData(array $activity, int $uid, bool $push, bool &$trust_source, string $original_actor = ''): array
        {
                $id        = JsonLD::fetchElement($activity, '@id');
                $type      = JsonLD::fetchElement($activity, '@type');
@@ -319,7 +332,7 @@ class Receiver
                $fetched = false;
 
                if (!empty($id) && !$trust_source) {
-                       $fetch_uid = $uid ?: self::getBestUserForActivity($activity);
+                       $fetch_uid = $uid ?: self::getBestUserForActivity($activity, $original_actor);
 
                        $fetched_activity = Processor::fetchCachedActivity($fetch_id, $fetch_uid);
                        if (!empty($fetched_activity)) {
@@ -355,7 +368,7 @@ class Receiver
                $type = JsonLD::fetchElement($activity, '@type');
 
                // Fetch all receivers from to, cc, bto and bcc
-               $receiverdata = self::getReceivers($activity, $actor, [], false, $push || $fetched);
+               $receiverdata = self::getReceivers($activity, $original_actor ?: $actor, [], false, $push || $fetched);
                $receivers = $reception_types = [];
                foreach ($receiverdata as $key => $data) {
                        $receivers[$key] = $data['uid'];
@@ -379,7 +392,7 @@ class Receiver
 
                // We possibly need some user to fetch private content,
                // so we fetch one out of the receivers if no uid is provided.
-               $fetch_uid = $uid ?: self::getBestUserForActivity($activity);
+               $fetch_uid = $uid ?: self::getBestUserForActivity($activity, $original_actor);
 
                $object_id = JsonLD::fetchElement($activity, 'as:object', '@id');
                if (empty($object_id)) {
@@ -394,21 +407,6 @@ class Receiver
 
                $object_type = self::fetchObjectType($activity, $object_id, $fetch_uid);
 
-               // Fetch the activity on Lemmy "Announce" messages (announces of activities)
-               if (($type == 'as:Announce') && in_array($object_type, array_merge(self::ACTIVITY_TYPES, ['as:Delete', 'as:Undo', 'as:Update']))) {
-                       Logger::debug('Fetch announced activity', ['object' => $object_id]);
-                       $data = Processor::fetchCachedActivity($object_id, $fetch_uid);
-                       if (!empty($data)) {
-                               $type = $object_type;
-                               $activity = JsonLD::compact($data);
-
-                               // Some variables need to be refetched since the activity changed
-                               $actor = JsonLD::fetchElement($activity, 'as:actor', '@id');
-                               $object_id = JsonLD::fetchElement($activity, 'as:object', '@id');
-                               $object_type = self::fetchObjectType($activity, $object_id, $fetch_uid);
-                       }
-               }
-
                // Any activities on account types must not be altered
                if (in_array($type, ['as:Flag'])) {
                        $object_data = [];
@@ -443,11 +441,15 @@ class Receiver
                                $object_data['directmessage'] = true;
                        } else {
                                $object_data['directmessage'] = JsonLD::fetchElement($activity, 'litepub:directMessage');
+
+                               if (!empty(JsonLD::fetchElement($activity['as:object'], 'misskey:_misskey_talk'))) {
+                                       $object_data = self::setChatData($object_data, $receivers);
+                               }
                        }
                } elseif (in_array($type, array_merge(self::ACTIVITY_TYPES, ['as:Announce', 'as:Follow'])) && in_array($object_type, self::CONTENT_TYPES)) {
                        // Create a mostly empty array out of the activity data (instead of the object).
                        // This way we later don't have to check for the existence of each individual array element.
-                       $object_data = self::processObject($activity);
+                       $object_data = self::processObject($activity, $original_actor);
                        $object_data['name'] = $type;
                        $object_data['author'] = JsonLD::fetchElement($activity, 'as:actor', '@id');
                        $object_data['object_id'] = $object_id;
@@ -521,6 +523,26 @@ class Receiver
                return $object_data;
        }
 
+       private static function setChatData(array $object_data, array $receivers): array
+       {
+               if (count($receivers) != 1) {
+                       return $object_data;
+               }
+
+               $user = User::getById(array_key_first($receivers), ['language']);
+               $l10n = DI::l10n()->withLang($user['language']);
+               $object_data['name'] = $l10n->t('Chat');
+
+               $mail = DBA::selectFirst('mail', ['uri'], ['uid' => array_key_first($receivers), 'title' => $object_data['name']], ['order' => ['id' => true]]);
+               if (!empty($mail['uri'])) {
+                       $object_data['reply-to-id'] = $mail['uri'];
+               }
+
+               $object_data['directmessage'] = true;
+               Logger::debug('Got Misskey Chat');
+               return $object_data;
+       }
+
        /**
         * Fetches the first user id from the receiver array
         *
@@ -591,20 +613,34 @@ class Receiver
                        }
                }
 
+               // Lemmy announces activities.
+               // To simplify the further processing, we modify the received object.
+               // For announced "create" activities we remove the middle layer.
+               // For the rest (like, dislike, update, ...) we just process the activity directly.
+               $original_actor = '';
+               $object_type = JsonLD::fetchElement($activity['as:object'] ?? [], '@type');
+               if (($type == 'as:Announce') && !empty($object_type) && !in_array($object_type, self::CONTENT_TYPES) && self::isGroup($actor)) {
+                       $object_object_type = JsonLD::fetchElement($activity['as:object']['as:object'] ?? [], '@type');
+                       if (in_array($object_type, ['as:Create']) && in_array($object_object_type, self::CONTENT_TYPES)) {
+                               Logger::debug('Replace "create" activity with inner object', ['type' => $object_type, 'object_type' => $object_object_type]);
+                               $activity['as:object'] = $activity['as:object']['as:object'];
+                       } elseif (in_array($object_type, array_merge(self::ACTIVITY_TYPES, ['as:Delete', 'as:Undo', 'as:Update']))) {
+                               Logger::debug('Change announced activity to activity', ['type' => $object_type]);
+                               $original_actor = $actor;
+                               $type = $object_type;
+                               $activity = $activity['as:object'];
+                       } else {
+                               Logger::info('Unhandled announced activity', ['type' => $object_type, 'object_type' => $object_object_type]);
+                       }
+               }
+
                // $trust_source is called by reference and is set to true if the content was retrieved successfully
-               $object_data = self::prepareObjectData($activity, $uid, $push, $trust_source);
+               $object_data = self::prepareObjectData($activity, $uid, $push, $trust_source, $original_actor);
                if (empty($object_data)) {
                        Logger::info('No object data found', ['activity' => $activity]);
                        return true;
                }
 
-               // Lemmy is announcing activities.
-               // We are changing the announces into regular activities.
-               if (($type == 'as:Announce') && in_array($object_data['type'] ?? '', array_merge(self::ACTIVITY_TYPES, ['as:Delete', 'as:Undo', 'as:Update']))) {
-                       Logger::debug('Change type of announce to activity', ['type' => $object_data['type']]);
-                       $type = $object_data['type'];
-               }
-
                if (!empty($body) && empty($object_data['raw'])) {
                        $object_data['raw'] = $body;
                }
@@ -657,7 +693,7 @@ class Receiver
                        return true;
                }
 
-               if (!empty($object_data['entry-id']) && $decouple && ($push || ($completion == self::COMPLETION_RELAY))) {
+               if (!empty($object_data['entry-id']) && $decouple && ($push || in_array($completion, [self::COMPLETION_RELAY, self::COMPLETION_ASYNC]))) {
                        if (Queue::isProcessable($object_data['entry-id'])) {
                                // We delay by 5 seconds to allow to accumulate all receivers
                                $delayed = date(DateTimeFormat::MYSQL, time() + 5);
@@ -681,6 +717,18 @@ class Receiver
                return true;
        }
 
+       /**
+        * Checks if the provided actor is a group account
+        *
+        * @param string $actor
+        * @return boolean
+        */
+       private static function isGroup(string $actor): bool
+       {
+               $profile = APContact::getByURL($actor);
+               return ($profile['type'] ?? '') == 'Group';
+       }
+
        /**
         * Route activities
         *
@@ -1002,10 +1050,10 @@ class Receiver
         *
         * @return int   user id
         */
-       public static function getBestUserForActivity(array $activity): int
+       public static function getBestUserForActivity(array $activity, string $actor = ''): int
        {
                $uid = 0;
-               $actor = JsonLD::fetchElement($activity, 'as:actor', '@id') ?? '';
+               $actor = $actor ?: JsonLD::fetchElement($activity, 'as:actor', '@id') ?? '';
 
                $receivers = self::getReceivers($activity, $actor, [], false, false);
                foreach ($receivers as $receiver) {
@@ -1041,7 +1089,7 @@ class Receiver
 
                        foreach ($receiver_list as $receiver) {
                                if ($receiver == 'Public') {
-                                       Logger::warning('Not compacted public collection found', ['activity' => $activity, 'callstack' => System::callstack(20)]);
+                                       Logger::warning('Not compacted public collection found', ['activity' => $activity]);
                                        $receiver = ActivityPub::PUBLIC_COLLECTION;
                                }
                                if ($receiver == self::PUBLIC_COLLECTION) {
@@ -1091,7 +1139,7 @@ class Receiver
                if (!empty($actor)) {
                        $profile   = APContact::getByURL($actor);
                        $followers = $profile['followers'] ?? '';
-                       $is_forum  = ($actor['type'] ?? '') == 'Group';
+                       $isGroup  = ($profile['type'] ?? '') == 'Group';
                        if ($push) {
                                Contact::updateByUrlIfNeeded($actor);
                        }
@@ -1099,7 +1147,7 @@ class Receiver
                } else {
                        Logger::info('Empty actor', ['activity' => $activity]);
                        $followers = '';
-                       $is_forum  = false;
+                       $isGroup  = false;
                }
 
                // We have to prevent false follower assumptions upon thread completions
@@ -1122,7 +1170,7 @@ class Receiver
                                }
 
                                // Fetch the receivers for the public and the followers collection
-                               if ((($receiver == $followers) || (($receiver == self::PUBLIC_COLLECTION) && !$is_forum)) && !empty($actor)) {
+                               if ((($receiver == $followers) || (($receiver == self::PUBLIC_COLLECTION) && !$isGroup) || ($isGroup && ($element == 'as:audience'))) && !empty($actor)) {
                                        $receivers = self::getReceiverForActor($actor, $tags, $receivers, $follower_target, $profile);
                                        continue;
                                }
@@ -1141,7 +1189,7 @@ class Receiver
                                        $condition = ['nurl' => Strings::normaliseLink($actor), 'rel' => [Contact::SHARING, Contact::FRIEND],
                                                'network' => $networks, 'archive' => false, 'pending' => false, 'uid' => $contact['uid']];
 
-                                       // Forum posts are only accepted from forum contacts
+                                       // Group posts are only accepted from group contacts
                                        if ($contact['contact-type'] == Contact::TYPE_COMMUNITY) {
                                                $condition['rel'] = [Contact::SHARING, Contact::FRIEND, Contact::FOLLOWER];
                                        }
@@ -1169,7 +1217,7 @@ class Receiver
                                                case 'as:audience':
                                                        $type = self::TARGET_AUDIENCE;
                                                        break;
-                                               }
+                                       }
 
                                        $receivers[$contact['uid']] = ['uid' => $contact['uid'], 'type' => $type];
                                }
@@ -1189,12 +1237,16 @@ class Receiver
                // "birdsitelive" is a service that mirrors tweets into the fediverse
                // These posts can be fetched without authentication, but are not marked as public
                // We treat them as unlisted posts to be able to handle them.
+               // We always process deletion activities.
+               $activity_type = JsonLD::fetchElement($activity, '@type');
                if (empty($receivers) && $fetch_unlisted && Contact::isPlatform($actor, 'birdsitelive')) {
                        $receivers[0]  = ['uid' => 0, 'type' => self::TARGET_GLOBAL];
                        $receivers[-1] = ['uid' => -1, 'type' => self::TARGET_GLOBAL];
                        Logger::notice('Post from "birdsitelive" is set to "unlisted"', ['id' => JsonLD::fetchElement($activity, '@id')]);
+               } elseif (empty($receivers) && in_array($activity_type, ['as:Delete', 'as:Undo'])) {
+                       $receivers[0] = ['uid' => 0, 'type' => self::TARGET_GLOBAL];
                } elseif (empty($receivers)) {
-                       Logger::notice('Post has got no receivers', ['fetch_unlisted' => $fetch_unlisted, 'actor' => $actor, 'id' => JsonLD::fetchElement($activity, '@id'), 'type' => JsonLD::fetchElement($activity, '@type')]);
+                       Logger::notice('Post has got no receivers', ['fetch_unlisted' => $fetch_unlisted, 'actor' => $actor, 'id' => JsonLD::fetchElement($activity, '@id'), 'type' => $activity_type]);
                }
 
                return $receivers;
@@ -1430,21 +1482,9 @@ class Receiver
                        return false;
                }
 
-               // Lemmy is resharing "create" activities instead of content
-               // We fetch the content from the activity.
-               if (in_array($type, ['as:Create'])) {
-                       $object = $object['as:object'];
-                       $type = JsonLD::fetchElement($object, '@type');
-                       if (empty($type)) {
-                               Logger::info('Empty type');
-                               return false;
-                       }
-                       $object_data = self::processObject($object);
-               }
-
                // We currently don't handle 'pt:CacheFile', but with this step we avoid logging
                if (in_array($type, self::CONTENT_TYPES) || ($type == 'pt:CacheFile')) {
-                       $object_data = self::processObject($object);
+                       $object_data = self::processObject($object, '');
 
                        if (!empty($data)) {
                                $object_data['raw-object'] = json_encode($data);
@@ -1848,12 +1888,13 @@ class Receiver
        /**
         * Fetches data from the object part of an activity
         *
-        * @param array $object
+        * @param array  $object
+        * @param string $actor
         *
         * @return array|bool Object data or FALSE if $object does not contain @id element
         * @throws \Exception
         */
-       private static function processObject(array $object)
+       private static function processObject(array $object, string $actor)
        {
                if (!JsonLD::fetchElement($object, '@id')) {
                        return false;
@@ -1861,7 +1902,7 @@ class Receiver
 
                $object_data = self::getObjectDataFromActivity($object);
 
-               $receiverdata = self::getReceivers($object, $object_data['actor'] ?? '', $object_data['tags'], true, false);
+               $receiverdata = self::getReceivers($object, $actor ?: $object_data['actor'] ?? '', $object_data['tags'], true, false);
                $receivers = $reception_types = [];
                foreach ($receiverdata as $key => $data) {
                        $receivers[$key] = $data['uid'];