]> git.mxchange.org Git - friendica.git/blobdiff - src/Protocol/ActivityPub/Transmitter.php
Fix PHPDoc comments project-wide
[friendica.git] / src / Protocol / ActivityPub / Transmitter.php
index 9f98409c3340f0984ed9b90d2924e1cd032f44ec..5136bc72c1b8231aa75b8bb0c6e7a6bed5307c24 100644 (file)
@@ -6,6 +6,7 @@ namespace Friendica\Protocol\ActivityPub;
 
 use Friendica\BaseObject;
 use Friendica\Database\DBA;
+use Friendica\Core\Logger;
 use Friendica\Core\System;
 use Friendica\Util\HTTPSignature;
 use Friendica\Core\Protocol;
@@ -26,6 +27,7 @@ use Friendica\Protocol\ActivityPub;
 use Friendica\Protocol\Diaspora;
 use Friendica\Core\Cache;
 use Friendica\Util\Map;
+use Friendica\Util\Network;
 
 require_once 'include/api.php';
 
@@ -40,15 +42,16 @@ class Transmitter
        /**
         * collects the lost of followers of the given owner
         *
-        * @param array $owner Owner array
-        * @param integer $page Page number
+        * @param array   $owner Owner array
+        * @param integer $page  Page number
         *
         * @return array of owners
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function getFollowers($owner, $page = null)
        {
                $condition = ['rel' => [Contact::FOLLOWER, Contact::FRIEND], 'network' => Protocol::NATIVE_SUPPORT, 'uid' => $owner['uid'],
-                       'self' => false, 'hidden' => false, 'archive' => false, 'pending' => false];
+                       'self' => false, 'deleted' => false, 'hidden' => false, 'archive' => false, 'pending' => false];
                $count = DBA::count('contact', $condition);
 
                $data = ['@context' => ActivityPub::CONTEXT];
@@ -87,15 +90,16 @@ class Transmitter
        /**
         * Create list of following contacts
         *
-        * @param array $owner Owner array
-        * @param integer $page Page numbe
+        * @param array   $owner Owner array
+        * @param integer $page  Page numbe
         *
         * @return array of following contacts
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function getFollowing($owner, $page = null)
        {
                $condition = ['rel' => [Contact::SHARING, Contact::FRIEND], 'network' => Protocol::NATIVE_SUPPORT, 'uid' => $owner['uid'],
-                       'self' => false, 'hidden' => false, 'archive' => false, 'pending' => false];
+                       'self' => false, 'deleted' => false, 'hidden' => false, 'archive' => false, 'pending' => false];
                $count = DBA::count('contact', $condition);
 
                $data = ['@context' => ActivityPub::CONTEXT];
@@ -134,10 +138,12 @@ class Transmitter
        /**
         * Public posts for the given owner
         *
-        * @param array $owner Owner array
-        * @param integer $page Page numbe
+        * @param array   $owner Owner array
+        * @param integer $page  Page numbe
         *
         * @return array of posts
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function getOutbox($owner, $page = null)
        {
@@ -184,6 +190,7 @@ class Transmitter
         *
         * @param integer $uid User ID
         * @return array with profile data
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function getProfile($uid)
        {
@@ -244,6 +251,8 @@ class Transmitter
         * @param array $item
         *
         * @return array with permissions
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        private static function fetchPermissionBlockFromConversation($item)
        {
@@ -282,8 +291,7 @@ class Transmitter
                        foreach ($activity[$element] as $receiver) {
                                if ($receiver == $profile['followers'] && !empty($item_profile['followers'])) {
                                        $permissions[$element][] = $item_profile['followers'];
-                               }
-                               if (!in_array($receiver, $exclude)) {
+                               } elseif (!in_array($receiver, $exclude)) {
                                        $permissions[$element][] = $receiver;
                                }
                        }
@@ -294,13 +302,30 @@ class Transmitter
        /**
         * Creates an array of permissions from an item thread
         *
-        * @param array $item
+        * @param array   $item
         * @param boolean $blindcopy
+        * @param boolean $last_id
         *
         * @return array with permission data
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
-       private static function createPermissionBlockForItem($item, $blindcopy)
+       private static function createPermissionBlockForItem($item, $blindcopy, $last_id = 0)
        {
+               if ($last_id == 0) {
+                       $last_id = $item['id'];
+               }
+
+               $always_bcc = false;
+
+               // Check if we should always deliver our stuff via BCC
+               if (!empty($item['uid'])) {
+                       $profile = Profile::getByUID($item['uid']);
+                       if (!empty($profile)) {
+                               $always_bcc = $profile['hide-friends'];
+                       }
+               }
+
                // Will be activated in a later step
                // $networks = [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS];
 
@@ -309,13 +334,13 @@ class Transmitter
 
                $data = ['to' => [], 'cc' => [], 'bcc' => []];
 
-               $data = array_merge($data, self::fetchPermissionBlockFromConversation($item));
-
                $actor_profile = APContact::getByURL($item['author-link']);
 
                $terms = Term::tagArrayFromItemId($item['id'], TERM_MENTION);
 
                if (!$item['private']) {
+                       $data = array_merge($data, self::fetchPermissionBlockFromConversation($item));
+
                        $data['to'][] = ActivityPub::PUBLIC_COLLECTION;
                        if (!empty($actor_profile['followers'])) {
                                $data['cc'][] = $actor_profile['followers'];
@@ -343,11 +368,13 @@ class Transmitter
                        }
 
                        foreach ($receiver_list as $receiver) {
-                               $contact = DBA::selectFirst('contact', ['url'], ['id' => $receiver, 'network' => $networks]);
+                               $contact = DBA::selectFirst('contact', ['url', 'hidden'], ['id' => $receiver, 'network' => $networks]);
                                if (DBA::isResult($contact) && !empty($profile = APContact::getByURL($contact['url'], false))) {
-                                       // BCC is currently deactivated, due to Pleroma and Mastodon not reacting like expected
-                                       // $data['bcc'][] = $profile['url'];
-                                       $data['cc'][] = $profile['url'];
+                                       if ($contact['hidden'] || $always_bcc) {
+                                               $data['bcc'][] = $profile['url'];
+                                       } else {
+                                               $data['cc'][] = $profile['url'];
+                                       }
                                }
                        }
                }
@@ -355,7 +382,7 @@ class Transmitter
                $parents = Item::select(['id', 'author-link', 'owner-link', 'gravity', 'uri'], ['parent' => $item['parent']]);
                while ($parent = Item::fetch($parents)) {
                        // Don't include data from future posts
-                       if ($parent['id'] >= $item['id']) {
+                       if ($parent['id'] >= $last_id) {
                                continue;
                        }
 
@@ -423,10 +450,12 @@ class Transmitter
        /**
         * Fetches a list of inboxes of followers of a given user
         *
-        * @param integer $uid User ID
+        * @param integer $uid      User ID
         * @param boolean $personal fetch personal inboxes
         *
         * @return array of follower inboxes
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function fetchTargetInboxesforUser($uid, $personal = false)
        {
@@ -446,6 +475,10 @@ class Transmitter
 
                $contacts = DBA::select('contact', ['url'], $condition);
                while ($contact = DBA::fetch($contacts)) {
+                       if (Network::isUrlBlocked($contact['url'])) {
+                               continue;
+                       }
+
                        $profile = APContact::getByURL($contact['url'], false);
                        if (!empty($profile)) {
                                if (empty($profile['sharedinbox']) || $personal) {
@@ -464,15 +497,18 @@ class Transmitter
        /**
         * Fetches an array of inboxes for the given item and user
         *
-        * @param array $item
-        * @param integer $uid User ID
+        * @param array   $item
+        * @param integer $uid      User ID
         * @param boolean $personal fetch personal inboxes
+        * @param integer $last_id Last item id for adding receivers
         *
         * @return array with inboxes
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
-       public static function fetchTargetInboxes($item, $uid, $personal = false)
+       public static function fetchTargetInboxes($item, $uid, $personal = false, $last_id = 0)
        {
-               $permissions = self::createPermissionBlockForItem($item, true);
+               $permissions = self::createPermissionBlockForItem($item, true, $last_id);
                if (empty($permissions)) {
                        return [];
                }
@@ -493,6 +529,10 @@ class Transmitter
                        $blindcopy = in_array($element, ['bto', 'bcc']);
 
                        foreach ($permissions[$element] as $receiver) {
+                               if (Network::isUrlBlocked($receiver)) {
+                                       continue;
+                               }
+
                                if ($receiver == $item_profile['followers']) {
                                        $inboxes = array_merge($inboxes, self::fetchTargetInboxesforUser($uid, $personal));
                                } else {
@@ -518,10 +558,20 @@ class Transmitter
         * @param array $item
         *
         * @return string with activity type
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        private static function getTypeOfItem($item)
        {
-               if (!empty(Diaspora::isReshare($item['body'], false))) {
+               $reshared = false;
+
+               // Only check for a reshare, if it is a real reshare and no quoted reshare
+               if (strpos($item['body'], "[share") === 0) {
+                       $announce = api_share_as_retweet($item);
+                       $reshared = !empty($announce['plink']);
+               }
+
+               if ($reshared) {
                        $type = 'Announce';
                } elseif ($item['verb'] == ACTIVITY_POST) {
                        if ($item['created'] == $item['edited']) {
@@ -550,15 +600,20 @@ class Transmitter
         * Creates the activity or fetches it from the cache
         *
         * @param integer $item_id
+        * @param boolean $force Force new cache entry
         *
         * @return array with the activity
+        * @throws \Exception
         */
-       public static function createCachedActivityFromItem($item_id)
+       public static function createCachedActivityFromItem($item_id, $force = false)
        {
                $cachekey = 'APDelivery:createActivity:' . $item_id;
-               $data = Cache::get($cachekey);
-               if (!is_null($data)) {
-                       return $data;
+
+               if (!$force) {
+                       $data = Cache::get($cachekey);
+                       if (!is_null($data)) {
+                               return $data;
+                       }
                }
 
                $data = ActivityPub\Transmitter::createActivityFromItem($item_id);
@@ -574,6 +629,7 @@ class Transmitter
         * @param boolean $object_mode Is the activity item is used inside another object?
         *
         * @return array of activity
+        * @throws \Exception
         */
        public static function createActivityFromItem($item_id, $object_mode = false)
        {
@@ -624,12 +680,21 @@ class Transmitter
                        $data['object'] = self::createActivityFromItem($item_id, true);
                } else {
                        $data['diaspora:guid'] = $item['guid'];
+                       if (!empty($item['signed_text'])) {
+                               $data['diaspora:like'] = $item['signed_text'];
+                       }
                        $data['object'] = $item['thr-parent'];
                }
 
-               $owner = User::getOwnerDataById($item['uid']);
+               if (!empty($item['contact-uid'])) {
+                       $uid = $item['contact-uid'];
+               } else {
+                       $uid = $item['uid'];
+               }
 
-               if (!$object_mode) {
+               $owner = User::getOwnerDataById($uid);
+
+               if (!$object_mode && !empty($owner)) {
                        return LDSignature::sign($data, $owner);
                } else {
                        return $data;
@@ -644,6 +709,8 @@ class Transmitter
         * @param integer $item_id
         *
         * @return array with the object data
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function createObjectFromItemID($item_id)
        {
@@ -699,6 +766,7 @@ class Transmitter
         * @param array $item
         *
         * @return array of tags
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        private static function createTagList($item)
        {
@@ -707,7 +775,8 @@ class Transmitter
                $terms = Term::tagArrayFromItemId($item['id']);
                foreach ($terms as $term) {
                        if ($term['type'] == TERM_HASHTAG) {
-                               $tags[] = ['type' => 'Hashtag', 'href' => $term['url'], 'name' => '#' . $term['term']];
+                               $url = System::baseUrl() . '/search?tag=' . urlencode($term['term']);
+                               $tags[] = ['type' => 'Hashtag', 'href' => $url, 'name' => '#' . $term['term']];
                        } elseif ($term['type'] == TERM_MENTION) {
                                $contact = Contact::getDetailsByURL($term['url']);
                                if (!empty($contact['addr'])) {
@@ -725,10 +794,11 @@ class Transmitter
        /**
         * Adds attachment data to the JSON document
         *
-        * @param array $item Data of the item that is to be posted
-        * @param text $type Object type
+        * @param array  $item Data of the item that is to be posted
+        * @param string $type Object type
         *
         * @return array with attachment data
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        private static function createAttachmentList($item, $type)
        {
@@ -777,11 +847,12 @@ class Transmitter
                return $attachments;
        }
 
-        /**
+       /**
         * @brief Callback function to replace a Friendica style mention in a mention that is used on AP
         *
         * @param array $match Matching values for the callback
         * @return string Replaced mention
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        private static function mentionCallback($match)
        {
@@ -821,6 +892,7 @@ class Transmitter
         * @param array $item
         *
         * @return string with context url
+        * @throws \Exception
         */
        private static function fetchContextURLForItem($item)
        {
@@ -841,6 +913,7 @@ class Transmitter
         * @param integer $item_id
         *
         * @return boolean
+        * @throws \Exception
         */
        private static function isSensitive($item_id)
        {
@@ -854,6 +927,7 @@ class Transmitter
         * @param array $item
         *
         * @return array with the event data
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function createEvent($item)
        {
@@ -880,6 +954,8 @@ class Transmitter
         * @param array $item
         *
         * @return array with the object data
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function createNote($item)
        {
@@ -970,6 +1046,8 @@ class Transmitter
         * @param array $item
         *
         * @return string with announced object url
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function createAnnounce($item)
        {
@@ -978,26 +1056,55 @@ class Transmitter
                        return self::createNote($item);
                }
 
-               return $announce['plink'];
+               // Fetch the original id of the object
+               $activity = ActivityPub::fetchContent($announce['plink'], $item['uid']);
+               if (!empty($activity)) {
+                       $ldactivity = JsonLD::compact($activity);
+                       $id = JsonLD::fetchElement($ldactivity, '@id');
+                       if (!empty($id)) {
+                               return $id;
+                       }
+               }
+
+               return self::createNote($item);
+       }
+
+       /**
+        * Creates an activity id for a given contact id
+        *
+        * @param integer $cid Contact ID of target
+        *
+        * @return bool|string activity id
+        */
+       public static function activityIDFromContact($cid)
+       {
+               $contact = DBA::selectFirst('contact', ['uid', 'id', 'created'], ['id' => $cid]);
+               if (!DBA::isResult($contact)) {
+                       return false;
+               }
+
+               $hash = hash('ripemd128', $contact['uid'].'-'.$contact['id'].'-'.$contact['created']);
+               $uuid = substr($hash, 0, 8). '-' . substr($hash, 8, 4) . '-' . substr($hash, 12, 4) . '-' . substr($hash, 16, 4) . '-' . substr($hash, 20, 12);
+               return System::baseUrl() . '/activity/' . $uuid;
        }
 
        /**
         * Transmits a contact suggestion to a given inbox
         *
-        * @param integer $uid User ID
-        * @param string $inbox Target inbox
+        * @param integer $uid           User ID
+        * @param string  $inbox         Target inbox
         * @param integer $suggestion_id Suggestion ID
         *
         * @return boolean was the transmission successful?
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function sendContactSuggestion($uid, $inbox, $suggestion_id)
        {
                $owner = User::getOwnerDataById($uid);
-               $profile = APContact::getByURL($owner['url']);
 
                $suggestion = DBA::selectFirst('fsuggest', ['url', 'note', 'created'], ['id' => $suggestion_id]);
 
-               $data = ['@context' => 'https://www.w3.org/ns/activitystreams',
+               $data = ['@context' => ActivityPub::CONTEXT,
                        'id' => System::baseUrl() . '/activity/' . System::createGUID(),
                        'type' => 'Announce',
                        'actor' => $owner['url'],
@@ -1009,24 +1116,53 @@ class Transmitter
 
                $signed = LDSignature::sign($data, $owner);
 
-               logger('Deliver profile deletion for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', LOGGER_DEBUG);
+               Logger::log('Deliver profile deletion for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', Logger::DEBUG);
+               return HTTPSignature::transmit($signed, $inbox, $uid);
+       }
+
+       /**
+        * Transmits a profile relocation to a given inbox
+        *
+        * @param integer $uid   User ID
+        * @param string  $inbox Target inbox
+        *
+        * @return boolean was the transmission successful?
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        */
+       public static function sendProfileRelocation($uid, $inbox)
+       {
+               $owner = User::getOwnerDataById($uid);
+
+               $data = ['@context' => ActivityPub::CONTEXT,
+                       'id' => System::baseUrl() . '/activity/' . System::createGUID(),
+                       'type' => 'dfrn:relocate',
+                       'actor' => $owner['url'],
+                       'object' => $owner['url'],
+                       'published' => DateTimeFormat::utcNow(DateTimeFormat::ATOM),
+                       'instrument' => ['type' => 'Service', 'name' => BaseObject::getApp()->getUserAgent()],
+                       'to' => [ActivityPub::PUBLIC_COLLECTION],
+                       'cc' => []];
+
+               $signed = LDSignature::sign($data, $owner);
+
+               Logger::log('Deliver profile relocation for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', Logger::DEBUG);
                return HTTPSignature::transmit($signed, $inbox, $uid);
        }
 
        /**
         * Transmits a profile deletion to a given inbox
         *
-        * @param integer $uid User ID
-        * @param string $inbox Target inbox
+        * @param integer $uid   User ID
+        * @param string  $inbox Target inbox
         *
         * @return boolean was the transmission successful?
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public static function sendProfileDeletion($uid, $inbox)
        {
                $owner = User::getOwnerDataById($uid);
-               $profile = APContact::getByURL($owner['url']);
 
-               $data = ['@context' => 'https://www.w3.org/ns/activitystreams',
+               $data = ['@context' => ActivityPub::CONTEXT,
                        'id' => System::baseUrl() . '/activity/' . System::createGUID(),
                        'type' => 'Delete',
                        'actor' => $owner['url'],
@@ -1038,24 +1174,26 @@ class Transmitter
 
                $signed = LDSignature::sign($data, $owner);
 
-               logger('Deliver profile deletion for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', LOGGER_DEBUG);
+               Logger::log('Deliver profile deletion for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', Logger::DEBUG);
                return HTTPSignature::transmit($signed, $inbox, $uid);
        }
 
        /**
         * Transmits a profile change to a given inbox
         *
-        * @param integer $uid User ID
-        * @param string $inbox Target inbox
+        * @param integer $uid   User ID
+        * @param string  $inbox Target inbox
         *
         * @return boolean was the transmission successful?
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function sendProfileUpdate($uid, $inbox)
        {
                $owner = User::getOwnerDataById($uid);
                $profile = APContact::getByURL($owner['url']);
 
-               $data = ['@context' => 'https://www.w3.org/ns/activitystreams',
+               $data = ['@context' => ActivityPub::CONTEXT,
                        'id' => System::baseUrl() . '/activity/' . System::createGUID(),
                        'type' => 'Update',
                        'actor' => $owner['url'],
@@ -1067,32 +1205,39 @@ class Transmitter
 
                $signed = LDSignature::sign($data, $owner);
 
-               logger('Deliver profile update for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', LOGGER_DEBUG);
+               Logger::log('Deliver profile update for user ' . $uid . ' to ' . $inbox . ' via ActivityPub', Logger::DEBUG);
                return HTTPSignature::transmit($signed, $inbox, $uid);
        }
 
        /**
         * Transmits a given activity to a target
         *
-        * @param array $activity
-        * @param string $target Target profile
-        * @param integer $uid User ID
+        * @param array   $activity
+        * @param string  $target Target profile
+        * @param integer $uid    User ID
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
+        * @throws \Exception
         */
-       public static function sendActivity($activity, $target, $uid)
+       public static function sendActivity($activity, $target, $uid, $id = '')
        {
                $profile = APContact::getByURL($target);
 
                $owner = User::getOwnerDataById($uid);
 
-               $data = ['@context' => 'https://www.w3.org/ns/activitystreams',
-                       'id' => System::baseUrl() . '/activity/' . System::createGUID(),
+               if (empty($id)) {
+                       $id = System::baseUrl() . '/activity/' . System::createGUID();
+               }
+
+               $data = ['@context' => ActivityPub::CONTEXT,
+                       'id' => $id,
                        'type' => $activity,
                        'actor' => $owner['url'],
                        'object' => $profile['url'],
                        'instrument' => ['type' => 'Service', 'name' => BaseObject::getApp()->getUserAgent()],
-                       'to' => $profile['url']];
+                       'to' => [$profile['url']]];
 
-               logger('Sending activity ' . $activity . ' to ' . $target . ' for user ' . $uid, LOGGER_DEBUG);
+               Logger::log('Sending activity ' . $activity . ' to ' . $target . ' for user ' . $uid, Logger::DEBUG);
 
                $signed = LDSignature::sign($data, $owner);
                HTTPSignature::transmit($signed, $profile['inbox'], $uid);
@@ -1101,16 +1246,18 @@ class Transmitter
        /**
         * Transmit a message that the contact request had been accepted
         *
-        * @param string $target Target profile
-        * @param $id
-        * @param integer $uid User ID
+        * @param string  $target Target profile
+        * @param         $id
+        * @param integer $uid    User ID
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function sendContactAccept($target, $id, $uid)
        {
                $profile = APContact::getByURL($target);
 
                $owner = User::getOwnerDataById($uid);
-               $data = ['@context' => 'https://www.w3.org/ns/activitystreams',
+               $data = ['@context' => ActivityPub::CONTEXT,
                        'id' => System::baseUrl() . '/activity/' . System::createGUID(),
                        'type' => 'Accept',
                        'actor' => $owner['url'],
@@ -1118,9 +1265,9 @@ class Transmitter
                                'actor' => $profile['url'],
                                'object' => $owner['url']],
                        'instrument' => ['type' => 'Service', 'name' => BaseObject::getApp()->getUserAgent()],
-                       'to' => $profile['url']];
+                       'to' => [$profile['url']]];
 
-               logger('Sending accept to ' . $target . ' for user ' . $uid . ' with id ' . $id, LOGGER_DEBUG);
+               Logger::log('Sending accept to ' . $target . ' for user ' . $uid . ' with id ' . $id, Logger::DEBUG);
 
                $signed = LDSignature::sign($data, $owner);
                HTTPSignature::transmit($signed, $profile['inbox'], $uid);
@@ -1129,16 +1276,18 @@ class Transmitter
        /**
         * Reject a contact request or terminates the contact relation
         *
-        * @param string $target Target profile
-        * @param $id
-        * @param integer $uid User ID
+        * @param string  $target Target profile
+        * @param         $id
+        * @param integer $uid    User ID
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        public static function sendContactReject($target, $id, $uid)
        {
                $profile = APContact::getByURL($target);
 
                $owner = User::getOwnerDataById($uid);
-               $data = ['@context' => 'https://www.w3.org/ns/activitystreams',
+               $data = ['@context' => ActivityPub::CONTEXT,
                        'id' => System::baseUrl() . '/activity/' . System::createGUID(),
                        'type' => 'Reject',
                        'actor' => $owner['url'],
@@ -1146,9 +1295,9 @@ class Transmitter
                                'actor' => $profile['url'],
                                'object' => $owner['url']],
                        'instrument' => ['type' => 'Service', 'name' => BaseObject::getApp()->getUserAgent()],
-                       'to' => $profile['url']];
+                       'to' => [$profile['url']]];
 
-               logger('Sending reject to ' . $target . ' for user ' . $uid . ' with id ' . $id, LOGGER_DEBUG);
+               Logger::log('Sending reject to ' . $target . ' for user ' . $uid . ' with id ' . $id, Logger::DEBUG);
 
                $signed = LDSignature::sign($data, $owner);
                HTTPSignature::transmit($signed, $profile['inbox'], $uid);
@@ -1157,27 +1306,35 @@ class Transmitter
        /**
         * Transmits a message that we don't want to follow this contact anymore
         *
-        * @param string $target Target profile
-        * @param integer $uid User ID
+        * @param string  $target Target profile
+        * @param integer $uid    User ID
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
+        * @throws \Exception
         */
-       public static function sendContactUndo($target, $uid)
+       public static function sendContactUndo($target, $cid, $uid)
        {
                $profile = APContact::getByURL($target);
 
+               $object_id = self::activityIDFromContact($cid);
+               if (empty($object_id)) {
+                       return;
+               }
+
                $id = System::baseUrl() . '/activity/' . System::createGUID();
 
                $owner = User::getOwnerDataById($uid);
-               $data = ['@context' => 'https://www.w3.org/ns/activitystreams',
+               $data = ['@context' => ActivityPub::CONTEXT,
                        'id' => $id,
                        'type' => 'Undo',
                        'actor' => $owner['url'],
-                       'object' => ['id' => $id, 'type' => 'Follow',
+                       'object' => ['id' => $object_id, 'type' => 'Follow',
                                'actor' => $owner['url'],
                                'object' => $profile['url']],
                        'instrument' => ['type' => 'Service', 'name' => BaseObject::getApp()->getUserAgent()],
-                       'to' => $profile['url']];
+                       'to' => [$profile['url']]];
 
-               logger('Sending undo to ' . $target . ' for user ' . $uid . ' with id ' . $id, LOGGER_DEBUG);
+               Logger::log('Sending undo to ' . $target . ' for user ' . $uid . ' with id ' . $id, Logger::DEBUG);
 
                $signed = LDSignature::sign($data, $owner);
                HTTPSignature::transmit($signed, $profile['inbox'], $uid);