]> git.mxchange.org Git - friendica.git/blobdiff - src/Protocol/ActivityPub/Transmitter.php
Improved AP distribution / Only process trusted content
[friendica.git] / src / Protocol / ActivityPub / Transmitter.php
index b92148e1ff14584e6f96277644c3d90c051fb1ee..7d9d6626ce8d918132d945f0b2e24b2c363fdc17 100644 (file)
@@ -6,7 +6,6 @@ namespace Friendica\Protocol\ActivityPub;
 
 use Friendica\Database\DBA;
 use Friendica\Core\System;
-use Friendica\BaseObject;
 use Friendica\Util\HTTPSignature;
 use Friendica\Core\Protocol;
 use Friendica\Model\Conversation;
@@ -19,18 +18,21 @@ use Friendica\Util\DateTimeFormat;
 use Friendica\Content\Text\BBCode;
 use Friendica\Util\JsonLD;
 use Friendica\Util\LDSignature;
-use Friendica\Protocol\ActivityPub;
 use Friendica\Model\Profile;
 use Friendica\Core\Config;
 use Friendica\Object\Image;
+use Friendica\Protocol\ActivityPub;
+use Friendica\Core\Cache;
 
 /**
  * @brief ActivityPub Transmitter Protocol class
  *
  * To-Do:
+ *
+ * Missing object types:
  * - Event
  *
- * Complicated:
+ * Complicated object types:
  * - Announce
  * - Undo Announce
  *
@@ -40,7 +42,7 @@ use Friendica\Object\Image;
 class Transmitter
 {
        /**
-        * @brief collects the lost of followers of the given owner
+        * collects the lost of followers of the given owner
         *
         * @param array $owner Owner array
         * @param integer $page Page number
@@ -87,7 +89,7 @@ class Transmitter
        }
 
        /**
-        * @brief Create list of following contacts
+        * Create list of following contacts
         *
         * @param array $owner Owner array
         * @param integer $page Page numbe
@@ -134,7 +136,7 @@ class Transmitter
        }
 
        /**
-        * @brief Public posts for the given owner
+        * Public posts for the given owner
         *
         * @param array $owner Owner array
         * @param integer $page Page numbe
@@ -185,7 +187,7 @@ class Transmitter
         * Return the ActivityPub profile of the given user
         *
         * @param integer $uid User ID
-        * @return profile array
+        * @return array with profile data
         */
        public static function getProfile($uid)
        {
@@ -236,7 +238,7 @@ class Transmitter
        }
 
        /**
-        * @brief Returns an array with permissions of a given item array
+        * Returns an array with permissions of a given item array
         *
         * @param array $item
         *
@@ -289,11 +291,11 @@ class Transmitter
        }
 
        /**
-        * @brief Creates an array of permissions from an item thread
+        * Creates an array of permissions from an item thread
         *
         * @param array $item
         *
-        * @return permission array
+        * @return array with permission data
         */
        private static function createPermissionBlockForItem($item)
        {
@@ -377,7 +379,7 @@ class Transmitter
        }
 
        /**
-        * @brief Fetches a list of inboxes of followers of a given user
+        * Fetches a list of inboxes of followers of a given user
         *
         * @param integer $uid User ID
         *
@@ -404,7 +406,7 @@ class Transmitter
        }
 
        /**
-        * @brief Fetches an array of inboxes for the given item and user
+        * Fetches an array of inboxes for the given item and user
         *
         * @param array $item
         * @param integer $uid User ID
@@ -448,11 +450,11 @@ class Transmitter
        }
 
        /**
-        * @brief Returns the activity type of a given item
+        * Returns the activity type of a given item
         *
         * @param array $item
         *
-        * @return activity type
+        * @return string with activity type
         */
        private static function getTypeOfItem($item)
        {
@@ -480,7 +482,28 @@ class Transmitter
        }
 
        /**
-        * @brief Creates an activity array for a given item id
+        * Creates the activity or fetches it from the cache
+        *
+        * @param integer $item_id
+        *
+        * @return array with the activity
+        */
+       public static function createCachedActivityFromItem($item_id)
+       {
+               $cachekey = 'APDelivery:createActivity:' . $item_id;
+               $data = Cache::get($cachekey);
+               if (!is_null($data)) {
+                       return $data;
+               }
+
+               $data = ActivityPub\Transmitter::createActivityFromItem($item_id);
+
+               Cache::set($cachekey, $data, CACHE_QUARTER_HOUR);
+               return $data;
+       }
+
+       /**
+        * Creates an activity array for a given item id
         *
         * @param integer $item_id
         * @param boolean $object_mode Is the activity item is used inside another object?
@@ -520,15 +543,9 @@ class Transmitter
 
                $data['id'] = $item['uri'] . '#' . $type;
                $data['type'] = $type;
-               $data['actor'] = $item['author-link'];
+               $data['actor'] = $item['owner-link'];
 
-               $data['published'] = DateTimeFormat::utc($item["created"]."+00:00", DateTimeFormat::ATOM);
-
-               if ($item["created"] != $item["edited"]) {
-                       $data['updated'] = DateTimeFormat::utc($item["edited"]."+00:00", DateTimeFormat::ATOM);
-               }
-
-               $data['context'] = self::fetchContextURLForItem($item);
+               $data['published'] = DateTimeFormat::utc($item['created'] . '+00:00', DateTimeFormat::ATOM);
 
                $data = array_merge($data, self::createPermissionBlockForItem($item));
 
@@ -547,14 +564,16 @@ class Transmitter
                } else {
                        return $data;
                }
+
+               /// @todo Create "conversation" entry
        }
 
        /**
-        * @brief Creates an object array for a given item id
+        * Creates an object array for a given item id
         *
         * @param integer $item_id
         *
-        * @return object array
+        * @return array with the object data
         */
        public static function createObjectFromItemID($item_id)
        {
@@ -571,7 +590,7 @@ class Transmitter
        }
 
        /**
-        * @brief Returns a tag array for a given item array
+        * Returns a tag array for a given item array
         *
         * @param array $item
         *
@@ -600,50 +619,17 @@ class Transmitter
        }
 
        /**
-        * @brief Adds attachment data to the JSON document
+        * Adds attachment data to the JSON document
         *
         * @param array $item Data of the item that is to be posted
-        * @return attachment array
+        * @param text $type Object type
+        *
+        * @return array with attachment data
         */
-
-       private static function createAttachmentList($item)
+       private static function createAttachmentList($item, $type)
        {
                $attachments = [];
 
-               $siteinfo = BBCode::getAttachedData($item['body']);
-
-               switch ($siteinfo['type']) {
-                       case 'photo':
-                               if (!empty($siteinfo['image'])) {
-                                       $imgdata = Image::getInfoFromURL($siteinfo['image']);
-                                       if ($imgdata) {
-                                               $attachments[] = ['type' => 'Document',
-                                                               'mediaType' => $imgdata['mime'],
-                                                               'url' => $siteinfo['image'],
-                                                               'name' => null];
-                                       }
-                               }
-                               break;
-                       case 'video':
-                               $attachments[] = ['type' => 'Document',
-                                               'mediaType' => 'text/html; charset=UTF-8',
-                                               'url' => $siteinfo['url'],
-                                               'name' => defaults($siteinfo, 'title', $siteinfo['url'])];
-                               break;
-                       default:
-                               break;
-               }
-
-               if (!Config::get('system', 'ostatus_not_attach_preview') && ($siteinfo['type'] != 'photo') && isset($siteinfo['image'])) {
-                       $imgdata = Image::getInfoFromURL($siteinfo['image']);
-                       if ($imgdata) {
-                               $attachments[] = ['type' => 'Document',
-                                               'mediaType' => $imgdata['mime'],
-                                               'url' => $siteinfo['image'],
-                                               'name' => null];
-                       }
-               }
-
                $arr = explode('[/attach],', $item['attach']);
                if (count($arr)) {
                        foreach ($arr as $r) {
@@ -664,11 +650,49 @@ class Transmitter
                        }
                }
 
+               if ($type != 'Note') {
+                       return $attachments;
+               }
+
+               // Simplify image codes
+               $body = preg_replace("/\[img\=([0-9]*)x([0-9]*)\](.*?)\[\/img\]/ism", '[img]$3[/img]', $item['body']);
+
+               // Grab all pictures and create attachments out of them
+               if (preg_match_all("/\[img\]([^\[\]]*)\[\/img\]/Usi", $body, $pictures)) {
+                       foreach ($pictures[1] as $picture) {
+                               $imgdata = Image::getInfoFromURL($picture);
+                               if ($imgdata) {
+                                       $attachments[] = ['type' => 'Document',
+                                               'mediaType' => $imgdata['mime'],
+                                               'url' => $picture,
+                                               'name' => null];
+                               }
+                       }
+               }
+
                return $attachments;
        }
 
        /**
-        * @brief Fetches the "context" value for a givem item array from the "conversation" table
+        * Remove image elements and replaces them with links to the image
+        *
+        * @param string $body
+        *
+        * @return string with replaced elements
+        */
+       private static function removePictures($body)
+       {
+               // Simplify image codes
+               $body = preg_replace("/\[img\=([0-9]*)x([0-9]*)\](.*?)\[\/img\]/ism", '[img]$3[/img]', $body);
+
+               $body = preg_replace("/\[url=([^\[\]]*)\]\[img\](.*)\[\/img\]\[\/url\]/Usi", '[url]$1[/url]', $body);
+               $body = preg_replace("/\[img\]([^\[\]]*)\[\/img\]/Usi", '[url]$1[/url]', $body);
+
+               return $body;
+       }
+
+       /**
+        * Fetches the "context" value for a givem item array from the "conversation" table
         *
         * @param array $item
         *
@@ -682,23 +706,30 @@ class Transmitter
                } elseif (DBA::isResult($conversation) && !empty($conversation['conversation-uri'])) {
                        $context_uri = $conversation['conversation-uri'];
                } else {
-                       $context_uri = str_replace('/objects/', '/context/', $item['parent-uri']);
+                       $context_uri = $item['parent-uri'] . '#context';
                }
                return $context_uri;
        }
 
-       private static function fetchSensitive($item_id)
+       /**
+        * Returns if the post contains sensitive content ("nsfw")
+        *
+        * @param integer $item_id
+        *
+        * @return boolean
+        */
+       private static function isSensitive($item_id)
        {
                $condition = ['otype' => TERM_OBJ_POST, 'oid' => $item_id, 'type' => TERM_HASHTAG, 'term' => 'nsfw'];
                return DBA::exists('term', $condition);
        }
 
        /**
-        * @brief Creates a note/article object array
+        * Creates a note/article object array
         *
         * @param array $item
         *
-        * @return object array
+        * @return array with the object data
         */
        public static function createNote($item)
        {
@@ -729,30 +760,36 @@ class Transmitter
                }
 
                $data['diaspora:guid'] = $item['guid'];
-               $data['published'] = DateTimeFormat::utc($item["created"]."+00:00", DateTimeFormat::ATOM);
+               $data['published'] = DateTimeFormat::utc($item['created'] . '+00:00', DateTimeFormat::ATOM);
 
-               if ($item["created"] != $item["edited"]) {
-                       $data['updated'] = DateTimeFormat::utc($item["edited"]."+00:00", DateTimeFormat::ATOM);
+               if ($item['created'] != $item['edited']) {
+                       $data['updated'] = DateTimeFormat::utc($item['edited'] . '+00:00', DateTimeFormat::ATOM);
                }
 
                $data['url'] = $item['plink'];
                $data['attributedTo'] = $item['author-link'];
                $data['actor'] = $item['author-link'];
-               $data['sensitive'] = self::fetchSensitive($item['id']);
+               $data['sensitive'] = self::isSensitive($item['id']);
                $data['context'] = self::fetchContextURLForItem($item);
 
                if (!empty($item['title'])) {
                        $data['name'] = BBCode::convert($item['title'], false, 7);
                }
 
-               $data['content'] = BBCode::convert($item['body'], false, 7);
+               $body = $item['body'];
+
+               if ($type == 'Note') {
+                       $body = self::removePictures($body);
+               }
+
+               $data['content'] = BBCode::convert($body, false, 7);
                $data['source'] = ['content' => $item['body'], 'mediaType' => "text/bbcode"];
 
                if (!empty($item['signed_text']) && ($item['uri'] != $item['thr-parent'])) {
                        $data['diaspora:comment'] = $item['signed_text'];
                }
 
-               $data['attachment'] = self::createAttachmentList($item);
+               $data['attachment'] = self::createAttachmentList($item, $type);
                $data['tag'] = self::createTagList($item);
                $data = array_merge($data, self::createPermissionBlockForItem($item));
 
@@ -760,7 +797,7 @@ class Transmitter
        }
 
        /**
-        * @brief Transmits a profile deletion to a given inbox
+        * Transmits a profile deletion to a given inbox
         *
         * @param integer $uid User ID
         * @param string $inbox Target inbox
@@ -781,12 +818,12 @@ class Transmitter
 
                $signed = LDSignature::sign($data, $owner);
 
-               logger('Deliver profile deletion for user ' . $uid . ' to ' . $inbox .' via ActivityPub', LOGGER_DEBUG);
+               logger('Deliver profile deletion for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', LOGGER_DEBUG);
                HTTPSignature::transmit($signed, $inbox, $uid);
        }
 
        /**
-        * @brief Transmits a profile change to a given inbox
+        * Transmits a profile change to a given inbox
         *
         * @param integer $uid User ID
         * @param string $inbox Target inbox
@@ -807,12 +844,12 @@ class Transmitter
 
                $signed = LDSignature::sign($data, $owner);
 
-               logger('Deliver profile update for user ' . $uid . ' to ' . $inbox .' via ActivityPub', LOGGER_DEBUG);
+               logger('Deliver profile update for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', LOGGER_DEBUG);
                HTTPSignature::transmit($signed, $inbox, $uid);
        }
 
        /**
-        * @brief Transmits a given activity to a target
+        * Transmits a given activity to a target
         *
         * @param array $activity
         * @param string $target Target profile
@@ -838,7 +875,7 @@ class Transmitter
        }
 
        /**
-        * @brief Transmit a message that the contact request had been accepted
+        * Transmit a message that the contact request had been accepted
         *
         * @param string $target Target profile
         * @param $id
@@ -865,7 +902,7 @@ class Transmitter
        }
 
        /**
-        * @brief 
+        * Reject a contact request or terminates the contact relation
         *
         * @param string $target Target profile
         * @param $id
@@ -892,7 +929,7 @@ class Transmitter
        }
 
        /**
-        * @brief 
+        * Transmits a message that we don't want to follow this contact anymore
         *
         * @param string $target Target profile
         * @param integer $uid User ID