]> git.mxchange.org Git - friendica.git/blobdiff - src/Protocol/Diaspora.php
Issue 10947: Send correct accept-header for AP
[friendica.git] / src / Protocol / Diaspora.php
index feb1111443a4bcb06324853eaeb0d3bbc66489e8..02d6d270a696fd1ee468ccb14f21215387d3a8a3 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2021, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
 namespace Friendica\Protocol;
 
 use Friendica\Content\Feature;
-use Friendica\Content\PageInfo;
 use Friendica\Content\Text\BBCode;
 use Friendica\Content\Text\Markdown;
-use Friendica\Core\Cache\Duration;
+use Friendica\Core\Cache\Enum\Duration;
 use Friendica\Core\Logger;
 use Friendica\Core\Protocol;
 use Friendica\Core\System;
@@ -34,6 +33,8 @@ use Friendica\Database\DBA;
 use Friendica\DI;
 use Friendica\Model\Contact;
 use Friendica\Model\Conversation;
+use Friendica\Model\FContact;
+use Friendica\Model\GServer;
 use Friendica\Model\Item;
 use Friendica\Model\ItemURI;
 use Friendica\Model\Mail;
@@ -55,196 +56,6 @@ use SimpleXMLElement;
  */
 class Diaspora
 {
-       /**
-        * Mark the relay contact of the given contact for archival
-        * This is called whenever there is a communication issue with the server.
-        * It avoids sending stuff to servers who don't exist anymore.
-        * The relay contact is a technical contact entry that exists once per server.
-        *
-        * @param array $contact of the relay contact
-        */
-       public static function markRelayForArchival(array $contact)
-       {
-               if (!empty($contact['contact-type']) && ($contact['contact-type'] == Contact::TYPE_RELAY)) {
-                       // This is already the relay contact, we don't need to fetch it
-                       $relay_contact = $contact;
-               } elseif (empty($contact['baseurl'])) {
-                       if (!empty($contact['batch'])) {
-                               $condition = ['uid' => 0, 'network' => Protocol::FEDERATED, 'batch' => $contact['batch'], 'contact-type' => Contact::TYPE_RELAY];
-                               $relay_contact = DBA::selectFirst('contact', [], $condition);
-                       } else {
-                               return;
-                       }
-               } else {
-                       $relay_contact = self::getRelayContact($contact['baseurl'], []);
-               }
-
-               if (!empty($relay_contact)) {
-                       Logger::info('Relay contact will be marked for archival', ['id' => $relay_contact['id'], 'url' => $relay_contact['url']]);
-                       Contact::markForArchival($relay_contact);
-               }
-       }
-
-       /**
-        * Return a list of relay servers
-        *
-        * The list contains not only the official relays but also servers that we serve directly
-        *
-        * @param integer $item_id  The id of the item that is sent
-        * @param array   $contacts The previously fetched contacts
-        *
-        * @return array of relay servers
-        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
-        */
-       public static function relayList($item_id, array $contacts = [])
-       {
-               $serverlist = [];
-
-               // Fetching relay servers
-               $serverdata = DI::config()->get("system", "relay_server");
-
-               if (!empty($serverdata)) {
-                       $servers = explode(",", $serverdata);
-                       foreach ($servers as $server) {
-                               $serverlist[$server] = trim($server);
-                       }
-               }
-
-               if (DI::config()->get("system", "relay_directly", false)) {
-                       // We distribute our stuff based on the parent to ensure that the thread will be complete
-                       $parent = Item::selectFirst(['uri-id'], ['id' => $item_id]);
-                       if (!DBA::isResult($parent)) {
-                               return;
-                       }
-
-                       // Servers that want to get all content
-                       $servers = DBA::select('gserver', ['url'], ['relay-subscribe' => true, 'relay-scope' => 'all']);
-                       while ($server = DBA::fetch($servers)) {
-                               $serverlist[$server['url']] = $server['url'];
-                       }
-                       DBA::close($servers);
-
-                       // All tags of the current post
-                       $tags = DBA::select('tag-view', ['name'], ['uri-id' => $parent['uri-id'], 'type' => Tag::HASHTAG]);
-                       $taglist = [];
-                       while ($tag = DBA::fetch($tags)) {
-                               $taglist[] = $tag['name'];
-                       }
-                       DBA::close($tags);
-
-                       // All servers who wants content with this tag
-                       $tagserverlist = [];
-                       if (!empty($taglist)) {
-                               $tagserver = DBA::select('gserver-tag', ['gserver-id'], ['tag' => $taglist]);
-                               while ($server = DBA::fetch($tagserver)) {
-                                       $tagserverlist[] = $server['gserver-id'];
-                               }
-                               DBA::close($tagserver);
-                       }
-
-                       // All adresses with the given id
-                       if (!empty($tagserverlist)) {
-                               $servers = DBA::select('gserver', ['url'], ['relay-subscribe' => true, 'relay-scope' => 'tags', 'id' => $tagserverlist]);
-                               while ($server = DBA::fetch($servers)) {
-                                       $serverlist[$server['url']] = $server['url'];
-                               }
-                               DBA::close($servers);
-                       }
-               }
-
-               // Now we are collecting all relay contacts
-               foreach ($serverlist as $server_url) {
-                       // We don't send messages to ourselves
-                       if (Strings::compareLink($server_url, DI::baseUrl())) {
-                               continue;
-                       }
-                       $contact = self::getRelayContact($server_url);
-                       if (is_bool($contact)) {
-                               continue;
-                       }
-
-                       $exists = false;
-                       foreach ($contacts as $entry) {
-                               if ($entry['batch'] == $contact['batch']) {
-                                       $exists = true;
-                               }
-                       }
-
-                       if (!$exists) {
-                               $contacts[] = $contact;
-                       }
-               }
-
-               return $contacts;
-       }
-
-       /**
-        * Return a contact for a given server address or creates a dummy entry
-        *
-        * @param string $server_url The url of the server
-        * @param array $fields Fieldlist
-        * @return array with the contact
-        * @throws \Exception
-        */
-       private static function getRelayContact(string $server_url, array $fields = ['batch', 'id', 'url', 'name', 'network', 'protocol', 'archive', 'blocked'])
-       {
-               // Fetch the relay contact
-               $condition = ['uid' => 0, 'nurl' => Strings::normaliseLink($server_url),
-                       'contact-type' => Contact::TYPE_RELAY];
-               $contact = DBA::selectFirst('contact', $fields, $condition);
-
-               if (DBA::isResult($contact)) {
-                       if ($contact['archive'] || $contact['blocked']) {
-                               return false;
-                       }
-                       return $contact;
-               } else {
-                       self::setRelayContact($server_url);
-
-                       $contact = DBA::selectFirst('contact', $fields, $condition);
-                       if (DBA::isResult($contact)) {
-                               return $contact;
-                       }
-               }
-
-               // It should never happen that we arrive here
-               return [];
-       }
-
-       /**
-        * Update or insert a relay contact
-        *
-        * @param string $server_url     The url of the server
-        * @param array  $network_fields Optional network specific fields
-        * @throws \Exception
-        */
-       public static function setRelayContact($server_url, array $network_fields = [])
-       {
-               $fields = ['created' => DateTimeFormat::utcNow(),
-                       'name' => 'relay', 'nick' => 'relay', 'url' => $server_url,
-                       'nurl' => Strings::normaliseLink($server_url),
-                       'network' => Protocol::DIASPORA, 'uid' => 0,
-                       'batch' => $server_url . '/receive/public',
-                       'rel' => Contact::FOLLOWER, 'blocked' => false,
-                       'pending' => false, 'writable' => true,
-                       'baseurl' => $server_url, 'contact-type' => Contact::TYPE_RELAY];
-
-               $fields = array_merge($fields, $network_fields);
-
-               $condition = ['uid' => 0, 'nurl' => Strings::normaliseLink($server_url)];
-               $old = DBA::selectFirst('contact', [], $condition);
-               if (DBA::isResult($old)) {
-                       unset($fields['created']);
-                       $condition = ['id' => $old['id']];
-
-                       Logger::info('Update relay contact', ['fields' => $fields, 'condition' => $condition]);
-                       DBA::update('contact', $fields, $condition, $old);
-               } else {
-                       Logger::info('Create relay contact', ['fields' => $fields]);
-                       Contact::insert($fields);
-               }
-       }
-
        /**
         * Return a list of participating contacts for a thread
         *
@@ -265,9 +76,9 @@ class Diaspora
                        return $contacts;
                }
 
-               $items = Item::select(['author-id', 'author-link', 'parent-author-link', 'parent-guid', 'guid'],
+               $items = Post::select(['author-id', 'author-link', 'parent-author-link', 'parent-guid', 'guid'],
                        ['parent' => $item['parent'], 'gravity' => [GRAVITY_COMMENT, GRAVITY_ACTIVITY]]);
-               while ($item = DBA::fetch($items)) {
+               while ($item = Post::fetch($items)) {
                        $contact = DBA::selectFirst('contact', ['id', 'url', 'name', 'protocol', 'batch', 'network'],
                                ['id' => $item['author-id']]);
                        if (!DBA::isResult($contact) || empty($contact['batch']) ||
@@ -307,14 +118,14 @@ class Diaspora
                $basedom = XML::parseString($envelope, true);
 
                if (!is_object($basedom)) {
-                       Logger::log("Envelope is no XML file");
+                       Logger::notice("Envelope is no XML file");
                        return false;
                }
 
                $children = $basedom->children('http://salmon-protocol.org/ns/magic-env');
 
                if (sizeof($children) == 0) {
-                       Logger::log("XML has no children");
+                       Logger::notice("XML has no children");
                        return false;
                }
 
@@ -339,19 +150,19 @@ class Diaspora
                $signable_data = $msg.".".Strings::base64UrlEncode($type).".".Strings::base64UrlEncode($encoding).".".Strings::base64UrlEncode($alg);
 
                if ($handle == '') {
-                       Logger::log('No author could be decoded. Discarding. Message: ' . $envelope);
+                       Logger::notice('No author could be decoded. Discarding. Message: ' . $envelope);
                        return false;
                }
 
                $key = self::key($handle);
                if ($key == '') {
-                       Logger::log("Couldn't get a key for handle " . $handle . ". Discarding.");
+                       Logger::notice("Couldn't get a key for handle " . $handle . ". Discarding.");
                        return false;
                }
 
                $verify = Crypto::rsaVerify($signable_data, $sig, $key);
                if (!$verify) {
-                       Logger::log('Message from ' . $handle . ' did not verify. Discarding.');
+                       Logger::notice('Message from ' . $handle . ' did not verify. Discarding.');
                        return false;
                }
 
@@ -414,7 +225,7 @@ class Diaspora
                        $j_outer_key_bundle = json_decode($outer_key_bundle);
 
                        if (!is_object($j_outer_key_bundle)) {
-                               Logger::log('Outer Salmon did not verify. Discarding.');
+                               Logger::notice('Outer Salmon did not verify. Discarding.');
                                if ($no_exit) {
                                        return false;
                                } else {
@@ -433,7 +244,7 @@ class Diaspora
                $basedom = XML::parseString($xml, true);
 
                if (!is_object($basedom)) {
-                       Logger::log('Received data does not seem to be an XML. Discarding. '.$xml);
+                       Logger::notice('Received data does not seem to be an XML. Discarding. '.$xml);
                        if ($no_exit) {
                                return false;
                        } else {
@@ -459,7 +270,7 @@ class Diaspora
                $key_id = $base->sig[0]->attributes()->key_id[0];
                $author_addr = base64_decode($key_id);
                if ($author_addr == '') {
-                       Logger::log('No author could be decoded. Discarding. Message: ' . $xml);
+                       Logger::notice('No author could be decoded. Discarding. Message: ' . $xml);
                        if ($no_exit) {
                                return false;
                        } else {
@@ -469,7 +280,7 @@ class Diaspora
 
                $key = self::key($author_addr);
                if ($key == '') {
-                       Logger::log("Couldn't get a key for handle " . $author_addr . ". Discarding.");
+                       Logger::notice("Couldn't get a key for handle " . $author_addr . ". Discarding.");
                        if ($no_exit) {
                                return false;
                        } else {
@@ -479,7 +290,7 @@ class Diaspora
 
                $verify = Crypto::rsaVerify($signed_data, $signature, $key);
                if (!$verify) {
-                       Logger::log('Message did not verify. Discarding.');
+                       Logger::notice('Message did not verify. Discarding.');
                        if ($no_exit) {
                                return false;
                        } else {
@@ -567,7 +378,7 @@ class Diaspora
                }
 
                if (!$base) {
-                       Logger::log('unable to locate salmon data in xml');
+                       Logger::notice('unable to locate salmon data in xml');
                        throw new \Friendica\Network\HTTPException\BadRequestException();
                }
 
@@ -605,29 +416,29 @@ class Diaspora
                }
 
                if (!$author_link) {
-                       Logger::log('Could not retrieve author URI.');
+                       Logger::notice('Could not retrieve author URI.');
                        throw new \Friendica\Network\HTTPException\BadRequestException();
                }
                // Once we have the author URI, go to the web and try to find their public key
                // (first this will look it up locally if it is in the fcontact cache)
                // This will also convert diaspora public key from pkcs#1 to pkcs#8
 
-               Logger::log('Fetching key for '.$author_link);
+               Logger::notice('Fetching key for '.$author_link);
                $key = self::key($author_link);
 
                if (!$key) {
-                       Logger::log('Could not retrieve author key.');
+                       Logger::notice('Could not retrieve author key.');
                        throw new \Friendica\Network\HTTPException\BadRequestException();
                }
 
                $verify = Crypto::rsaVerify($signed_data, $signature, $key);
 
                if (!$verify) {
-                       Logger::log('Message did not verify. Discarding.');
+                       Logger::notice('Message did not verify. Discarding.');
                        throw new \Friendica\Network\HTTPException\BadRequestException();
                }
 
-               Logger::log('Message verified.');
+               Logger::notice('Message verified.');
 
                return ['message' => (string)$inner_decrypted,
                                'author' => XML::unescape($author_link),
@@ -638,27 +449,28 @@ class Diaspora
        /**
         * Dispatches public messages and find the fitting receivers
         *
-        * @param array $msg The post that will be dispatched
+        * @param array $msg     The post that will be dispatched
+        * @param bool  $fetched The message had been fetched (default "false")
         *
         * @return int The message id of the generated message, "true" or "false" if there was an error
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       public static function dispatchPublic($msg)
+       public static function dispatchPublic($msg, bool $fetched = false)
        {
                $enabled = intval(DI::config()->get("system", "diaspora_enabled"));
                if (!$enabled) {
-                       Logger::log("diaspora is disabled");
+                       Logger::notice("diaspora is disabled");
                        return false;
                }
 
                if (!($fields = self::validPosting($msg))) {
-                       Logger::log("Invalid posting");
+                       Logger::notice("Invalid posting");
                        return false;
                }
 
                $importer = ["uid" => 0, "page-flags" => User::PAGE_FLAGS_FREELOVE];
-               $success = self::dispatch($importer, $msg, $fields);
+               $success = self::dispatch($importer, $msg, $fields, $fetched);
 
                return $success;
        }
@@ -669,12 +481,13 @@ class Diaspora
         * @param array            $importer Array of the importer user
         * @param array            $msg      The post that will be dispatched
         * @param SimpleXMLElement $fields   SimpleXML object that contains the message
+        * @param bool             $fetched  The message had been fetched (default "false")
         *
         * @return int The message id of the generated message, "true" or "false" if there was an error
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       public static function dispatch(array $importer, $msg, SimpleXMLElement $fields = null)
+       public static function dispatch(array $importer, $msg, SimpleXMLElement $fields = null, bool $fetched = false)
        {
                // The sender is the handle of the contact that sent the message.
                // This will often be different with relayed messages (for example "like" and "comment")
@@ -684,7 +497,7 @@ class Diaspora
                if (is_null($fields)) {
                        $private = true;
                        if (!($fields = self::validPosting($msg))) {
-                               Logger::log("Invalid posting");
+                               Logger::notice("Invalid posting");
                                return false;
                        }
                } else {
@@ -698,7 +511,7 @@ class Diaspora
                switch ($type) {
                        case "account_migration":
                                if (!$private) {
-                                       Logger::log('Message with type ' . $type . ' is not private, quitting.');
+                                       Logger::notice('Message with type ' . $type . ' is not private, quitting.');
                                        return false;
                                }
                                return self::receiveAccountMigration($importer, $fields);
@@ -707,38 +520,38 @@ class Diaspora
                                return self::receiveAccountDeletion($fields);
 
                        case "comment":
-                               return self::receiveComment($importer, $sender, $fields, $msg["message"]);
+                               return self::receiveComment($importer, $sender, $fields, $msg["message"], $fetched);
 
                        case "contact":
                                if (!$private) {
-                                       Logger::log('Message with type ' . $type . ' is not private, quitting.');
+                                       Logger::notice('Message with type ' . $type . ' is not private, quitting.');
                                        return false;
                                }
                                return self::receiveContactRequest($importer, $fields);
 
                        case "conversation":
                                if (!$private) {
-                                       Logger::log('Message with type ' . $type . ' is not private, quitting.');
+                                       Logger::notice('Message with type ' . $type . ' is not private, quitting.');
                                        return false;
                                }
                                return self::receiveConversation($importer, $msg, $fields);
 
                        case "like":
-                               return self::receiveLike($importer, $sender, $fields);
+                               return self::receiveLike($importer, $sender, $fields, $fetched);
 
                        case "message":
                                if (!$private) {
-                                       Logger::log('Message with type ' . $type . ' is not private, quitting.');
+                                       Logger::notice('Message with type ' . $type . ' is not private, quitting.');
                                        return false;
                                }
                                return self::receiveMessage($importer, $fields);
 
                        case "participation":
                                if (!$private) {
-                                       Logger::log('Message with type ' . $type . ' is not private, quitting.');
+                                       Logger::notice('Message with type ' . $type . ' is not private, quitting.');
                                        return false;
                                }
-                               return self::receiveParticipation($importer, $fields);
+                               return self::receiveParticipation($importer, $fields, $fetched);
 
                        case "photo": // Not implemented
                                return self::receivePhoto($importer, $fields);
@@ -748,22 +561,22 @@ class Diaspora
 
                        case "profile":
                                if (!$private) {
-                                       Logger::log('Message with type ' . $type . ' is not private, quitting.');
+                                       Logger::notice('Message with type ' . $type . ' is not private, quitting.');
                                        return false;
                                }
                                return self::receiveProfile($importer, $fields);
 
                        case "reshare":
-                               return self::receiveReshare($importer, $fields, $msg["message"]);
+                               return self::receiveReshare($importer, $fields, $msg["message"], $fetched);
 
                        case "retraction":
                                return self::receiveRetraction($importer, $sender, $fields);
 
                        case "status_message":
-                               return self::receiveStatusMessage($importer, $fields, $msg["message"]);
+                               return self::receiveStatusMessage($importer, $fields, $msg["message"], $fetched);
 
                        default:
-                               Logger::log("Unknown message type ".$type);
+                               Logger::notice("Unknown message type ".$type);
                                return false;
                }
        }
@@ -803,7 +616,7 @@ class Diaspora
                $type = $element->getName();
                $orig_type = $type;
 
-               Logger::log("Got message type ".$type.": ".$msg["message"], Logger::DATA);
+               Logger::debug("Got message type ".$type.": ".$msg["message"]);
 
                // All retractions are handled identically from now on.
                // In the new version there will only be "retraction".
@@ -879,7 +692,7 @@ class Diaspora
                // This is something that shouldn't happen at all.
                if (in_array($type, ["status_message", "reshare", "profile"])) {
                        if ($msg["author"] != $fields->author) {
-                               Logger::log("Message handle is not the same as envelope sender. Quitting this message.");
+                               Logger::notice("Message handle is not the same as envelope sender. Quitting this message.");
                                return false;
                        }
                }
@@ -890,7 +703,7 @@ class Diaspora
                }
                // No author_signature? This is a must, so we quit.
                if (!isset($author_signature)) {
-                       Logger::log("No author signature for type ".$type." - Message: ".$msg["message"], Logger::DEBUG);
+                       Logger::info("No author signature for type ".$type." - Message: ".$msg["message"]);
                        return false;
                }
 
@@ -902,7 +715,7 @@ class Diaspora
                        }
 
                        if (!Crypto::rsaVerify($signed_data, $parent_author_signature, $key, "sha256")) {
-                               Logger::log("No valid parent author signature for parent author ".$msg["author"]. " in type ".$type." - signed data: ".$signed_data." - Message: ".$msg["message"]." - Signature ".$parent_author_signature, Logger::DEBUG);
+                               Logger::info("No valid parent author signature for parent author ".$msg["author"]. " in type ".$type." - signed data: ".$signed_data." - Message: ".$msg["message"]." - Signature ".$parent_author_signature);
                                return false;
                        }
                }
@@ -914,7 +727,7 @@ class Diaspora
                }
 
                if (!Crypto::rsaVerify($signed_data, $author_signature, $key, "sha256")) {
-                       Logger::log("No valid author signature for author ".$fields->author. " in type ".$type." - signed data: ".$signed_data." - Message: ".$msg["message"]." - Signature ".$author_signature, Logger::DEBUG);
+                       Logger::info("No valid author signature for author ".$fields->author. " in type ".$type." - signed data: ".$signed_data." - Message: ".$msg["message"]." - Signature ".$author_signature);
                        return false;
                } else {
                        return $fields;
@@ -934,9 +747,9 @@ class Diaspora
        {
                $handle = strval($handle);
 
-               Logger::log("Fetching diaspora key for: ".$handle);
+               Logger::notice("Fetching diaspora key for: ".$handle);
 
-               $r = self::personByHandle($handle);
+               $r = FContact::getByURL($handle);
                if ($r) {
                        return $r["pubkey"];
                }
@@ -944,81 +757,6 @@ class Diaspora
                return "";
        }
 
-       /**
-        * Fetches data for a given handle
-        *
-        * @param string $handle The handle
-        * @param boolean $update true = always update, false = never update, null = update when not found or outdated
-        *
-        * @return array the queried data
-        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
-        * @throws \ImagickException
-        */
-       public static function personByHandle($handle, $update = null)
-       {
-               $person = DBA::selectFirst('fcontact', [], ['network' => Protocol::DIASPORA, 'addr' => $handle]);
-               if (!DBA::isResult($person)) {
-                       $urls = [$handle, str_replace('http://', 'https://', $handle), Strings::normaliseLink($handle)];
-                       $person = DBA::selectFirst('fcontact', [], ['network' => Protocol::DIASPORA, 'url' => $urls]);
-               }
-
-               if (DBA::isResult($person)) {
-                       Logger::debug('In cache', ['person' => $person]);
-
-                       if (is_null($update)) {
-                               // update record occasionally so it doesn't get stale
-                               $d = strtotime($person["updated"]." +00:00");
-                               if ($d < strtotime("now - 14 days")) {
-                                       $update = true;
-                               }
-
-                               if ($person["guid"] == "") {
-                                       $update = true;
-                               }
-                       }
-               } elseif (is_null($update)) {
-                       $update = !DBA::isResult($person);
-               } else {
-                       $person = [];
-               }
-
-               if ($update) {
-                       Logger::log("create or refresh", Logger::DEBUG);
-                       $r = Probe::uri($handle, Protocol::DIASPORA);
-
-                       // Note that Friendica contacts will return a "Diaspora person"
-                       // if Diaspora connectivity is enabled on their server
-                       if ($r && ($r["network"] === Protocol::DIASPORA)) {
-                               self::updateFContact($r);
-
-                               $person = self::personByHandle($handle, false);
-                       }
-               }
-
-               return $person;
-       }
-
-       /**
-        * Updates the fcontact table
-        *
-        * @param array $arr The fcontact data
-        * @throws \Exception
-        */
-       private static function updateFContact($arr)
-       {
-               $fields = ['name' => $arr["name"], 'photo' => $arr["photo"],
-                       'request' => $arr["request"], 'nick' => $arr["nick"],
-                       'addr' => strtolower($arr["addr"]), 'guid' => $arr["guid"],
-                       'batch' => $arr["batch"], 'notify' => $arr["notify"],
-                       'poll' => $arr["poll"], 'confirm' => $arr["confirm"],
-                       'alias' => $arr["alias"], 'pubkey' => $arr["pubkey"],
-                       'updated' => DateTimeFormat::utcNow()];
-
-               $condition = ['url' => $arr["url"], 'network' => $arr["network"]];
-
-               DBA::update('fcontact', $fields, $condition, true);
-       }
-
        /**
         * get a handle (user@domain.tld) from a given contact id
         *
@@ -1030,68 +768,25 @@ class Diaspora
         */
        private static function handleFromContact($contact_id, $pcontact_id = 0)
        {
-               $handle = false;
-
-               Logger::log("contact id is ".$contact_id." - pcontact id is ".$pcontact_id, Logger::DEBUG);
+               $handle = '';
 
                if ($pcontact_id != 0) {
-                       $contact = DBA::selectFirst('contact', ['addr'], ['id' => $pcontact_id]);
-
-                       if (DBA::isResult($contact) && !empty($contact["addr"])) {
-                               return strtolower($contact["addr"]);
+                       $contact = Contact::getById($pcontact_id, ['addr']);
+                       if (DBA::isResult($contact)) {
+                               $handle = $contact['addr'];
                        }
                }
 
-               $r = q(
-                       "SELECT `network`, `addr`, `self`, `url`, `nick` FROM `contact` WHERE `id` = %d",
-                       intval($contact_id)
-               );
-
-               if (DBA::isResult($r)) {
-                       $contact = $r[0];
-
-                       Logger::log("contact 'self' = ".$contact['self']." 'url' = ".$contact['url'], Logger::DEBUG);
-
-                       if ($contact['addr'] != "") {
+               if (empty($handle)) {
+                       $contact = Contact::getById($contact_id, ['addr']);
+                       if (DBA::isResult($contact)) {
                                $handle = $contact['addr'];
-                       } else {
-                               $baseurl_start = strpos($contact['url'], '://') + 3;
-                               // allows installations in a subdirectory--not sure how Diaspora will handle
-                               $baseurl_length = strpos($contact['url'], '/profile') - $baseurl_start;
-                               $baseurl = substr($contact['url'], $baseurl_start, $baseurl_length);
-                               $handle = $contact['nick'].'@'.$baseurl;
                        }
                }
 
                return strtolower($handle);
        }
 
-       /**
-        * get a url (scheme://domain.tld/u/user) from a given Diaspora*
-        * fcontact guid
-        *
-        * @param mixed $fcontact_guid Hexadecimal string guid
-        *
-        * @return string the contact url or null
-        * @throws \Exception
-        */
-       public static function urlFromContactGuid($fcontact_guid)
-       {
-               Logger::info('fcontact', ['guid' => $fcontact_guid]);
-
-               $r = q(
-                       "SELECT `url` FROM `fcontact` WHERE `url` != '' AND `network` = '%s' AND `guid` = '%s'",
-                       DBA::escape(Protocol::DIASPORA),
-                       DBA::escape($fcontact_guid)
-               );
-
-               if (DBA::isResult($r)) {
-                       return $r[0]['url'];
-               }
-
-               return null;
-       }
-
        /**
         * Get a contact id for a given handle
         *
@@ -1106,20 +801,7 @@ class Diaspora
         */
        private static function contactByHandle($uid, $handle)
        {
-               $cid = Contact::getIdForURL($handle, $uid);
-               if (!$cid) {
-                       Logger::log("Haven't found a contact for user " . $uid . " and handle " . $handle, Logger::DEBUG);
-                       return false;
-               }
-
-               $contact = DBA::selectFirst('contact', [], ['id' => $cid]);
-               if (!DBA::isResult($contact)) {
-                       // This here shouldn't happen at all
-                       Logger::log("Haven't found a contact for user " . $uid . " and handle " . $handle, Logger::DEBUG);
-                       return false;
-               }
-
-               return $contact;
+               return Contact::getByURL($handle, null, [], $uid);
        }
 
        /**
@@ -1133,7 +815,7 @@ class Diaspora
         */
        public static function isSupportedByContactUrl($url, $update = null)
        {
-               return !empty(self::personByHandle($url, $update));
+               return !empty(FContact::getByURL($url, $update));
        }
 
        /**
@@ -1162,7 +844,7 @@ class Diaspora
                //      );
                //
                //      $contact["rel"] = Contact::FRIEND;
-               //      Logger::log("defining user ".$contact["nick"]." as friend");
+               //      Logger::notice("defining user ".$contact["nick"]." as friend");
                //}
 
                // Contact server is blocked
@@ -1203,7 +885,7 @@ class Diaspora
        {
                $contact = self::contactByHandle($importer["uid"], $handle);
                if (!$contact) {
-                       Logger::log("A Contact for handle ".$handle." and user ".$importer["uid"]." was not found");
+                       Logger::notice("A Contact for handle ".$handle." and user ".$importer["uid"]." was not found");
                        // If a contact isn't found, we accept it anyway if it is a comment
                        if ($is_comment && ($importer["uid"] != 0)) {
                                return self::contactByHandle(0, $handle);
@@ -1215,7 +897,7 @@ class Diaspora
                }
 
                if (!self::postAllow($importer, $contact, $is_comment)) {
-                       Logger::log("The handle: ".$handle." is not allowed to post to user ".$importer["uid"]);
+                       Logger::notice("The handle: ".$handle." is not allowed to post to user ".$importer["uid"]);
                        return false;
                }
                return $contact;
@@ -1232,9 +914,9 @@ class Diaspora
         */
        private static function messageExists($uid, $guid)
        {
-               $item = Item::selectFirst(['id'], ['uid' => $uid, 'guid' => $guid]);
+               $item = Post::selectFirst(['id'], ['uid' => $uid, 'guid' => $guid]);
                if (DBA::isResult($item)) {
-                       Logger::log("message ".$guid." already exists for user ".$uid);
+                       Logger::notice("message ".$guid." already exists for user ".$uid);
                        return $item["id"];
                }
 
@@ -1285,7 +967,7 @@ class Diaspora
                                // 0 => '[url=/people/0123456789abcdef]Foo Bar[/url]'
                                // 1 => '0123456789abcdef'
                                // 2 => 'Foo Bar'
-                               $handle = self::urlFromContactGuid($match[1]);
+                               $handle = FContact::getUrlByGuid($match[1]);
 
                                if ($handle) {
                                        $return = '@[url='.$handle.']'.$match[2].'[/url]';
@@ -1340,7 +1022,7 @@ class Diaspora
 
                $server = $serverparts["scheme"]."://".$serverparts["host"];
 
-               Logger::log("Trying to fetch item ".$guid." from ".$server, Logger::DEBUG);
+               Logger::info("Trying to fetch item ".$guid." from ".$server);
 
                $msg = self::message($guid, $server);
 
@@ -1348,10 +1030,10 @@ class Diaspora
                        return false;
                }
 
-               Logger::log("Successfully fetched item ".$guid." from ".$server, Logger::DEBUG);
+               Logger::info("Successfully fetched item ".$guid." from ".$server);
 
                // Now call the dispatcher
-               return self::dispatchPublic($msg);
+               return self::dispatchPublic($msg, true);
        }
 
        /**
@@ -1367,7 +1049,7 @@ class Diaspora
         *      'key' => The public key of the author
         * @throws \Exception
         */
-       private static function message($guid, $server, $level = 0)
+       public static function message($guid, $server, $level = 0)
        {
                if ($level > 5) {
                        return false;
@@ -1376,16 +1058,16 @@ class Diaspora
                // This will work for new Diaspora servers and Friendica servers from 3.5
                $source_url = $server."/fetch/post/".urlencode($guid);
 
-               Logger::log("Fetch post from ".$source_url, Logger::DEBUG);
+               Logger::info("Fetch post from ".$source_url);
 
-               $envelope = DI::httpRequest()->fetch($source_url);
+               $envelope = DI::httpClient()->fetch($source_url);
                if ($envelope) {
-                       Logger::log("Envelope was fetched.", Logger::DEBUG);
+                       Logger::info("Envelope was fetched.");
                        $x = self::verifyMagicEnvelope($envelope);
                        if (!$x) {
-                               Logger::log("Envelope could not be verified.", Logger::DEBUG);
+                               Logger::info("Envelope could not be verified.");
                        } else {
-                               Logger::log("Envelope was verified.", Logger::DEBUG);
+                               Logger::info("Envelope was verified.");
                        }
                } else {
                        $x = false;
@@ -1403,11 +1085,11 @@ class Diaspora
 
                if ($source_xml->post->reshare) {
                        // Reshare of a reshare - old Diaspora version
-                       Logger::log("Message is a reshare", Logger::DEBUG);
+                       Logger::info("Message is a reshare");
                        return self::message($source_xml->post->reshare->root_guid, $server, ++$level);
                } elseif ($source_xml->getName() == "reshare") {
                        // Reshare of a reshare - new Diaspora version
-                       Logger::log("Message is a new reshare", Logger::DEBUG);
+                       Logger::info("Message is a new reshare");
                        return self::message($source_xml->root_guid, $server, ++$level);
                }
 
@@ -1422,7 +1104,7 @@ class Diaspora
 
                // If this isn't a "status_message" then quit
                if (!$author) {
-                       Logger::log("Message doesn't seem to be a status message", Logger::DEBUG);
+                       Logger::info("Message doesn't seem to be a status message");
                        return false;
                }
 
@@ -1452,7 +1134,7 @@ class Diaspora
 
                $guid = urldecode($matches[2]);
 
-               $item = Item::selectFirst(['id'], ['guid' => $guid, 'uid' => $uid]);
+               $item = Post::selectFirst(['id'], ['guid' => $guid, 'uid' => $uid]);
                if (DBA::isResult($item)) {
                        Logger::info('Found', ['id' => $item['id']]);
                        return $item['id'];
@@ -1462,7 +1144,7 @@ class Diaspora
                $ret = self::storeByGuid($guid, $matches[1], $uid);
                Logger::info('Result', ['ret' => $ret]);
 
-               $item = Item::selectFirst(['id'], ['guid' => $guid, 'uid' => $uid]);
+               $item = Post::selectFirst(['id'], ['guid' => $guid, 'uid' => $uid]);
                if (DBA::isResult($item)) {
                        Logger::info('Found', ['id' => $item['id']]);
                        return $item['id'];
@@ -1489,10 +1171,10 @@ class Diaspora
                        'author-name', 'author-link', 'author-avatar', 'gravity',
                        'owner-name', 'owner-link', 'owner-avatar'];
                $condition = ['uid' => $uid, 'guid' => $guid];
-               $item = Item::selectFirst($fields, $condition);
+               $item = Post::selectFirst($fields, $condition);
 
                if (!DBA::isResult($item)) {
-                       $person = self::personByHandle($author);
+                       $person = FContact::getByURL($author);
                        $result = self::storeByGuid($guid, $person["url"], $uid);
 
                        // We don't have an url for items that arrived at the public dispatcher
@@ -1501,17 +1183,17 @@ class Diaspora
                        }
 
                        if ($result) {
-                               Logger::log("Fetched missing item ".$guid." - result: ".$result, Logger::DEBUG);
+                               Logger::info("Fetched missing item ".$guid." - result: ".$result);
 
-                               $item = Item::selectFirst($fields, $condition);
+                               $item = Post::selectFirst($fields, $condition);
                        }
                }
 
                if (!DBA::isResult($item)) {
-                       Logger::log("parent item not found: parent: ".$guid." - user: ".$uid);
+                       Logger::notice("parent item not found: parent: ".$guid." - user: ".$uid);
                        return false;
                } else {
-                       Logger::log("parent item found: parent: ".$guid." - user: ".$uid);
+                       Logger::notice("parent item found: parent: ".$guid." - user: ".$uid);
                        return $item;
                }
        }
@@ -1566,7 +1248,7 @@ class Diaspora
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       private static function plink($addr, $guid, $parent_guid = '')
+       private static function plink(string $addr, string $guid, string $parent_guid = '')
        {
                $contact = Contact::getByURL($addr);
                if (empty($contact)) {
@@ -1632,23 +1314,23 @@ class Diaspora
         */
        private static function receiveAccountMigration(array $importer, $data)
        {
-               $old_handle = Strings::escapeTags(XML::unescape($data->author));
-               $new_handle = Strings::escapeTags(XML::unescape($data->profile->author));
-               $signature = Strings::escapeTags(XML::unescape($data->signature));
+               $old_handle = XML::unescape($data->author);
+               $new_handle = XML::unescape($data->profile->author);
+               $signature = XML::unescape($data->signature);
 
                $contact = self::contactByHandle($importer["uid"], $old_handle);
                if (!$contact) {
-                       Logger::log("cannot find contact for sender: ".$old_handle." and user ".$importer["uid"]);
+                       Logger::notice("cannot find contact for sender: ".$old_handle." and user ".$importer["uid"]);
                        return false;
                }
 
-               Logger::log("Got migration for ".$old_handle.", to ".$new_handle." with user ".$importer["uid"]);
+               Logger::notice("Got migration for ".$old_handle.", to ".$new_handle." with user ".$importer["uid"]);
 
                // Check signature
                $signed_text = 'AccountMigration:'.$old_handle.':'.$new_handle;
                $key = self::key($old_handle);
                if (!Crypto::rsaVerify($signed_text, $signature, $key, "sha256")) {
-                       Logger::log('No valid signature for migration.');
+                       Logger::notice('No valid signature for migration.');
                        return false;
                }
 
@@ -1658,7 +1340,7 @@ class Diaspora
                // change the technical stuff in contact
                $data = Probe::uri($new_handle);
                if ($data['network'] == Protocol::PHANTOM) {
-                       Logger::log('Account for '.$new_handle." couldn't be probed.");
+                       Logger::notice('Account for '.$new_handle." couldn't be probed.");
                        return false;
                }
 
@@ -1668,9 +1350,9 @@ class Diaspora
                                'notify' => $data['notify'], 'poll' => $data['poll'],
                                'network' => $data['network']];
 
-               DBA::update('contact', $fields, ['addr' => $old_handle]);
+               Contact::update($fields, ['addr' => $old_handle]);
 
-               Logger::log('Contacts are updated.');
+               Logger::notice('Contacts are updated.');
 
                return true;
        }
@@ -1685,7 +1367,7 @@ class Diaspora
         */
        private static function receiveAccountDeletion($data)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
+               $author = XML::unescape($data->author);
 
                $contacts = DBA::select('contact', ['id'], ['addr' => $author]);
                while ($contact = DBA::fetch($contacts)) {
@@ -1693,7 +1375,7 @@ class Diaspora
                }
                DBA::close($contacts);
 
-               Logger::log('Removed contacts for ' . $author);
+               Logger::notice('Removed contacts for ' . $author);
 
                return true;
        }
@@ -1711,11 +1393,11 @@ class Diaspora
         */
        private static function getUriFromGuid($author, $guid, $onlyfound = false)
        {
-               $item = Item::selectFirst(['uri'], ['guid' => $guid]);
+               $item = Post::selectFirst(['uri'], ['guid' => $guid]);
                if (DBA::isResult($item)) {
                        return $item["uri"];
                } elseif (!$onlyfound) {
-                       $person = self::personByHandle($author);
+                       $person = FContact::getByURL($author);
 
                        $parts = parse_url($person['url']);
                        unset($parts['path']);
@@ -1727,46 +1409,6 @@ class Diaspora
                return "";
        }
 
-       /**
-        * Fetch the guid from our database with a given uri
-        *
-        * @param string $uri Message uri
-        * @param string $uid Author handle
-        *
-        * @return string The post guid
-        * @throws \Exception
-        */
-       private static function getGuidFromUri($uri, $uid)
-       {
-               $item = Item::selectFirst(['guid'], ['uri' => $uri, 'uid' => $uid]);
-               if (DBA::isResult($item)) {
-                       return $item["guid"];
-               } else {
-                       return false;
-               }
-       }
-
-       /**
-        * Find the best importer for a comment, like, ...
-        *
-        * @param string $guid The guid of the item
-        *
-        * @return array|boolean the origin owner of that post - or false
-        * @throws \Exception
-        */
-       private static function importerForGuid($guid)
-       {
-               $item = Item::selectFirst(['uid'], ['origin' => true, 'guid' => $guid]);
-               if (DBA::isResult($item)) {
-                       Logger::log("Found user ".$item['uid']." as owner of item ".$guid, Logger::DEBUG);
-                       $contact = DBA::selectFirst('contact', [], ['self' => true, 'uid' => $item['uid']]);
-                       if (DBA::isResult($contact)) {
-                               return $contact;
-                       }
-               }
-               return false;
-       }
-
        /**
         * Store the mentions in the tag table
         *
@@ -1792,7 +1434,7 @@ class Diaspora
                                continue;
                        }
 
-                       $person = self::personByHandle($match[3]);
+                       $person = FContact::getByURL($match[3]);
                        if (empty($person)) {
                                continue;
                        }
@@ -1808,29 +1450,30 @@ class Diaspora
         * @param string $sender   The sender of the message
         * @param object $data     The message object
         * @param string $xml      The original XML of the message
+        * @param bool   $fetched  The message had been fetched and not pushed
         *
         * @return int The message id of the generated comment or "false" if there was an error
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       private static function receiveComment(array $importer, $sender, $data, $xml)
+       private static function receiveComment(array $importer, $sender, $data, $xml, bool $fetched)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
-               $guid = Strings::escapeTags(XML::unescape($data->guid));
-               $parent_guid = Strings::escapeTags(XML::unescape($data->parent_guid));
+               $author = XML::unescape($data->author);
+               $guid = XML::unescape($data->guid);
+               $parent_guid = XML::unescape($data->parent_guid);
                $text = XML::unescape($data->text);
 
                if (isset($data->created_at)) {
-                       $created_at = DateTimeFormat::utc(Strings::escapeTags(XML::unescape($data->created_at)));
+                       $created_at = DateTimeFormat::utc(XML::unescape($data->created_at));
                } else {
                        $created_at = DateTimeFormat::utcNow();
                }
 
                if (isset($data->thread_parent_guid)) {
-                       $thread_parent_guid = Strings::escapeTags(XML::unescape($data->thread_parent_guid));
-                       $thr_uri = self::getUriFromGuid("", $thread_parent_guid, true);
+                       $thread_parent_guid = XML::unescape($data->thread_parent_guid);
+                       $thr_parent = self::getUriFromGuid("", $thread_parent_guid, true);
                } else {
-                       $thr_uri = "";
+                       $thr_parent = "";
                }
 
                $contact = self::allowedContactByHandle($importer, $sender, true);
@@ -1838,19 +1481,23 @@ class Diaspora
                        return false;
                }
 
+               if (!empty($contact['gsid'])) {
+                       GServer::setProtocol($contact['gsid'], Post\DeliveryData::DIASPORA);
+               }
+
                $message_id = self::messageExists($importer["uid"], $guid);
                if ($message_id) {
                        return true;
                }
 
-               $parent_item = self::parentItem($importer["uid"], $parent_guid, $author, $contact);
-               if (!$parent_item) {
+               $toplevel_parent_item = self::parentItem($importer["uid"], $parent_guid, $author, $contact);
+               if (!$toplevel_parent_item) {
                        return false;
                }
 
-               $person = self::personByHandle($author);
+               $person = FContact::getByURL($author);
                if (!is_array($person)) {
-                       Logger::log("unable to find author details");
+                       Logger::notice("unable to find author details");
                        return false;
                }
 
@@ -1869,6 +1516,15 @@ class Diaspora
                $datarray["owner-link"] = $contact["url"];
                $datarray["owner-id"] = Contact::getIdForURL($contact["url"], 0);
 
+               // Will be overwritten for sharing accounts in Item::insert
+               if ($fetched) {
+                       $datarray["post-reason"] = Item::PR_FETCHED;
+               } elseif ($datarray["uid"] == 0) {
+                       $datarray["post-reason"] = Item::PR_GLOBAL;
+               } else {
+                       $datarray["post-reason"] = Item::PR_COMMENT;
+               }
+
                $datarray["guid"] = $guid;
                $datarray["uri"] = self::getUriFromGuid($author, $guid);
                $datarray['uri-id'] = ItemURI::insert(['uri' => $datarray['uri'], 'guid' => $datarray['guid']]);
@@ -1876,20 +1532,18 @@ class Diaspora
                $datarray["verb"] = Activity::POST;
                $datarray["gravity"] = GRAVITY_COMMENT;
 
-               if ($thr_uri != "") {
-                       $datarray["parent-uri"] = $thr_uri;
-               } else {
-                       $datarray["parent-uri"] = $parent_item["uri"];
-               }
+               $datarray['thr-parent'] = $thr_parent ?: $toplevel_parent_item['uri'];
 
                $datarray["object-type"] = Activity\ObjectType::COMMENT;
+               $datarray["post-type"] = Item::PT_NOTE;
 
                $datarray["protocol"] = Conversation::PARCEL_DIASPORA;
                $datarray["source"] = $xml;
+               $datarray["direction"] = $fetched ? Conversation::PULL : Conversation::PUSH;
 
                $datarray["changed"] = $datarray["created"] = $datarray["edited"] = $created_at;
 
-               $datarray["plink"] = self::plink($author, $guid, $parent_item['guid']);
+               $datarray["plink"] = self::plink($author, $guid, $toplevel_parent_item['guid']);
                $body = Markdown::toBBCode($text);
 
                $datarray["body"] = self::replacePeopleGuid($body, $person["url"]);
@@ -1901,10 +1555,15 @@ class Diaspora
 
                // If we are the origin of the parent we store the original data.
                // We notify our followers during the item storage.
-               if ($parent_item["origin"]) {
+               if ($toplevel_parent_item["origin"]) {
                        $datarray['diaspora_signed_text'] = json_encode($data);
                }
 
+               if (Item::isTooOld($datarray)) {
+                       Logger::info('Comment is too old', ['created' => $datarray['created'], 'uid' => $datarray['uid'], 'guid' => $datarray['guid']]);
+                       return false;
+               }
+
                $message_id = Item::insert($datarray);
 
                if ($message_id <= 0) {
@@ -1912,7 +1571,7 @@ class Diaspora
                }
 
                if ($message_id) {
-                       Logger::log("Stored comment ".$datarray["guid"]." with message id ".$message_id, Logger::DEBUG);
+                       Logger::info("Stored comment ".$datarray["guid"]." with message id ".$message_id);
                        if ($datarray['uid'] == 0) {
                                Item::distribute($message_id, json_encode($data));
                        }
@@ -1936,34 +1595,34 @@ class Diaspora
         */
        private static function receiveConversationMessage(array $importer, array $contact, $data, $msg, $mesg, $conversation)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
-               $guid = Strings::escapeTags(XML::unescape($data->guid));
-               $subject = Strings::escapeTags(XML::unescape($data->subject));
+               $author = XML::unescape($data->author);
+               $guid = XML::unescape($data->guid);
+               $subject = XML::unescape($data->subject);
 
                // "diaspora_handle" is the element name from the old version
                // "author" is the element name from the new version
                if ($mesg->author) {
-                       $msg_author = Strings::escapeTags(XML::unescape($mesg->author));
+                       $msg_author = XML::unescape($mesg->author);
                } elseif ($mesg->diaspora_handle) {
-                       $msg_author = Strings::escapeTags(XML::unescape($mesg->diaspora_handle));
+                       $msg_author = XML::unescape($mesg->diaspora_handle);
                } else {
                        return false;
                }
 
-               $msg_guid = Strings::escapeTags(XML::unescape($mesg->guid));
-               $msg_conversation_guid = Strings::escapeTags(XML::unescape($mesg->conversation_guid));
+               $msg_guid = XML::unescape($mesg->guid);
+               $msg_conversation_guid = XML::unescape($mesg->conversation_guid);
                $msg_text = XML::unescape($mesg->text);
-               $msg_created_at = DateTimeFormat::utc(Strings::escapeTags(XML::unescape($mesg->created_at)));
+               $msg_created_at = DateTimeFormat::utc(XML::unescape($mesg->created_at));
 
                if ($msg_conversation_guid != $guid) {
-                       Logger::log("message conversation guid does not belong to the current conversation.");
+                       Logger::notice("message conversation guid does not belong to the current conversation.");
                        return false;
                }
 
                $body = Markdown::toBBCode($msg_text);
                $message_uri = $msg_author.":".$msg_guid;
 
-               $person = self::personByHandle($msg_author);
+               $person = FContact::getByURL($msg_author);
 
                return Mail::insert([
                        'uid'        => $importer['uid'],
@@ -1993,16 +1652,16 @@ class Diaspora
         */
        private static function receiveConversation(array $importer, $msg, $data)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
-               $guid = Strings::escapeTags(XML::unescape($data->guid));
-               $subject = Strings::escapeTags(XML::unescape($data->subject));
-               $created_at = DateTimeFormat::utc(Strings::escapeTags(XML::unescape($data->created_at)));
-               $participants = Strings::escapeTags(XML::unescape($data->participants));
+               $author = XML::unescape($data->author);
+               $guid = XML::unescape($data->guid);
+               $subject = XML::unescape($data->subject);
+               $created_at = DateTimeFormat::utc(XML::unescape($data->created_at));
+               $participants = XML::unescape($data->participants);
 
                $messages = $data->message;
 
                if (!count($messages)) {
-                       Logger::log("empty conversation");
+                       Logger::notice("empty conversation");
                        return false;
                }
 
@@ -2011,25 +1670,26 @@ class Diaspora
                        return false;
                }
 
+               if (!empty($contact['gsid'])) {
+                       GServer::setProtocol($contact['gsid'], Post\DeliveryData::DIASPORA);
+               }
+
                $conversation = DBA::selectFirst('conv', [], ['uid' => $importer["uid"], 'guid' => $guid]);
                if (!DBA::isResult($conversation)) {
-                       $r = q(
-                               "INSERT INTO `conv` (`uid`, `guid`, `creator`, `created`, `updated`, `subject`, `recips`)
-                               VALUES (%d, '%s', '%s', '%s', '%s', '%s', '%s')",
-                               intval($importer["uid"]),
-                               DBA::escape($guid),
-                               DBA::escape($author),
-                               DBA::escape($created_at),
-                               DBA::escape(DateTimeFormat::utcNow()),
-                               DBA::escape($subject),
-                               DBA::escape($participants)
-                       );
+                       $r = DBA::insert('conv', [
+                               'uid'     => $importer['uid'],
+                               'guid'    => $guid,
+                               'creator' => $author,
+                               'created' => $created_at,
+                               'updated' => DateTimeFormat::utcNow(),
+                               'subject' => $subject,
+                               'recips'  => $participants]);
                        if ($r) {
                                $conversation = DBA::selectFirst('conv', [], ['uid' => $importer["uid"], 'guid' => $guid]);
                        }
                }
                if (!$conversation) {
-                       Logger::log("unable to create conversation.");
+                       Logger::notice("unable to create conversation.");
                        return false;
                }
 
@@ -2051,13 +1711,13 @@ class Diaspora
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       private static function receiveLike(array $importer, $sender, $data)
+       private static function receiveLike(array $importer, $sender, $data, bool $fetched)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
-               $guid = Strings::escapeTags(XML::unescape($data->guid));
-               $parent_guid = Strings::escapeTags(XML::unescape($data->parent_guid));
-               $parent_type = Strings::escapeTags(XML::unescape($data->parent_type));
-               $positive = Strings::escapeTags(XML::unescape($data->positive));
+               $author = XML::unescape($data->author);
+               $guid = XML::unescape($data->guid);
+               $parent_guid = XML::unescape($data->parent_guid);
+               $parent_type = XML::unescape($data->parent_type);
+               $positive = XML::unescape($data->positive);
 
                // likes on comments aren't supported by Diaspora - only on posts
                // But maybe this will be supported in the future, so we will accept it.
@@ -2070,19 +1730,23 @@ class Diaspora
                        return false;
                }
 
+               if (!empty($contact['gsid'])) {
+                       GServer::setProtocol($contact['gsid'], Post\DeliveryData::DIASPORA);
+               }
+
                $message_id = self::messageExists($importer["uid"], $guid);
                if ($message_id) {
                        return true;
                }
 
-               $parent_item = self::parentItem($importer["uid"], $parent_guid, $author, $contact);
-               if (!$parent_item) {
+               $toplevel_parent_item = self::parentItem($importer["uid"], $parent_guid, $author, $contact);
+               if (!$toplevel_parent_item) {
                        return false;
                }
 
-               $person = self::personByHandle($author);
+               $person = FContact::getByURL($author);
                if (!is_array($person)) {
-                       Logger::log("unable to find author details");
+                       Logger::notice("unable to find author details");
                        return false;
                }
 
@@ -2100,6 +1764,7 @@ class Diaspora
                $datarray = [];
 
                $datarray["protocol"] = Conversation::PARCEL_DIASPORA;
+               $datarray["direction"] = $fetched ? Conversation::PULL : Conversation::PUSH;
 
                $datarray["uid"] = $importer["uid"];
                $datarray["contact-id"] = $author_contact["cid"];
@@ -2113,7 +1778,7 @@ class Diaspora
 
                $datarray["verb"] = $verb;
                $datarray["gravity"] = GRAVITY_ACTIVITY;
-               $datarray["parent-uri"] = $parent_item["uri"];
+               $datarray['thr-parent'] = $toplevel_parent_item['uri'];
 
                $datarray["object-type"] = Activity\ObjectType::NOTE;
 
@@ -2123,11 +1788,11 @@ class Diaspora
                $datarray["changed"] = $datarray["created"] = $datarray["edited"] = DateTimeFormat::utcNow();
 
                // like on comments have the comment as parent. So we need to fetch the toplevel parent
-               if ($parent_item['gravity'] != GRAVITY_PARENT) {
-                       $toplevel = Item::selectFirst(['origin'], ['id' => $parent_item['parent']]);
+               if ($toplevel_parent_item['gravity'] != GRAVITY_PARENT) {
+                       $toplevel = Post::selectFirst(['origin'], ['id' => $toplevel_parent_item['parent']]);
                        $origin = $toplevel["origin"];
                } else {
-                       $origin = $parent_item["origin"];
+                       $origin = $toplevel_parent_item["origin"];
                }
 
                // If we are the origin of the parent we store the original data.
@@ -2136,6 +1801,11 @@ class Diaspora
                        $datarray['diaspora_signed_text'] = json_encode($data);
                }
 
+               if (Item::isTooOld($datarray)) {
+                       Logger::info('Like is too old', ['created' => $datarray['created'], 'uid' => $datarray['uid'], 'guid' => $datarray['guid']]);
+                       return false;
+               }
+
                $message_id = Item::insert($datarray);
 
                if ($message_id <= 0) {
@@ -2143,7 +1813,7 @@ class Diaspora
                }
 
                if ($message_id) {
-                       Logger::log("Stored like ".$datarray["guid"]." with message id ".$message_id, Logger::DEBUG);
+                       Logger::info("Stored like ".$datarray["guid"]." with message id ".$message_id);
                        if ($datarray['uid'] == 0) {
                                Item::distribute($message_id, json_encode($data));
                        }
@@ -2163,32 +1833,36 @@ class Diaspora
         */
        private static function receiveMessage(array $importer, $data)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
-               $guid = Strings::escapeTags(XML::unescape($data->guid));
-               $conversation_guid = Strings::escapeTags(XML::unescape($data->conversation_guid));
+               $author = XML::unescape($data->author);
+               $guid = XML::unescape($data->guid);
+               $conversation_guid = XML::unescape($data->conversation_guid);
                $text = XML::unescape($data->text);
-               $created_at = DateTimeFormat::utc(Strings::escapeTags(XML::unescape($data->created_at)));
+               $created_at = DateTimeFormat::utc(XML::unescape($data->created_at));
 
                $contact = self::allowedContactByHandle($importer, $author, true);
                if (!$contact) {
                        return false;
                }
 
+               if (!empty($contact['gsid'])) {
+                       GServer::setProtocol($contact['gsid'], Post\DeliveryData::DIASPORA);
+               }
+
                $conversation = null;
 
                $condition = ['uid' => $importer["uid"], 'guid' => $conversation_guid];
                $conversation = DBA::selectFirst('conv', [], $condition);
 
                if (!DBA::isResult($conversation)) {
-                       Logger::log("conversation not available.");
+                       Logger::notice("conversation not available.");
                        return false;
                }
 
                $message_uri = $author.":".$guid;
 
-               $person = self::personByHandle($author);
+               $person = FContact::getByURL($author);
                if (!$person) {
-                       Logger::log("unable to find author details");
+                       Logger::notice("unable to find author details");
                        return false;
                }
 
@@ -2223,38 +1897,42 @@ class Diaspora
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       private static function receiveParticipation(array $importer, $data)
+       private static function receiveParticipation(array $importer, $data, bool $fetched)
        {
-               $author = strtolower(Strings::escapeTags(XML::unescape($data->author)));
-               $guid = Strings::escapeTags(XML::unescape($data->guid));
-               $parent_guid = Strings::escapeTags(XML::unescape($data->parent_guid));
+               $author = strtolower(XML::unescape($data->author));
+               $guid = XML::unescape($data->guid);
+               $parent_guid = XML::unescape($data->parent_guid);
 
                $contact = self::allowedContactByHandle($importer, $author, true);
                if (!$contact) {
                        return false;
                }
 
+               if (!empty($contact['gsid'])) {
+                       GServer::setProtocol($contact['gsid'], Post\DeliveryData::DIASPORA);
+               }
+
                if (self::messageExists($importer["uid"], $guid)) {
                        return true;
                }
 
-               $parent_item = self::parentItem($importer["uid"], $parent_guid, $author, $contact);
-               if (!$parent_item) {
+               $toplevel_parent_item = self::parentItem($importer["uid"], $parent_guid, $author, $contact);
+               if (!$toplevel_parent_item) {
                        return false;
                }
 
-               if (!$parent_item['origin']) {
+               if (!$toplevel_parent_item['origin']) {
                        Logger::info('Not our origin. Participation is ignored', ['parent_guid' => $parent_guid, 'guid' => $guid, 'author' => $author]);
                }
 
-               if (!in_array($parent_item['private'], [Item::PUBLIC, Item::UNLISTED])) {
+               if (!in_array($toplevel_parent_item['private'], [Item::PUBLIC, Item::UNLISTED])) {
                        Logger::info('Item is not public, participation is ignored', ['parent_guid' => $parent_guid, 'guid' => $guid, 'author' => $author]);
                        return false;
                }
 
-               $person = self::personByHandle($author);
+               $person = FContact::getByURL($author);
                if (!is_array($person)) {
-                       Logger::log("Person not found: ".$author);
+                       Logger::notice("Person not found: ".$author);
                        return false;
                }
 
@@ -2264,6 +1942,7 @@ class Diaspora
                $datarray = [];
 
                $datarray["protocol"] = Conversation::PARCEL_DIASPORA;
+               $datarray["direction"] = $fetched ? Conversation::PULL : Conversation::PUSH;
 
                $datarray["uid"] = $importer["uid"];
                $datarray["contact-id"] = $author_contact["cid"];
@@ -2277,7 +1956,7 @@ class Diaspora
 
                $datarray["verb"] = Activity::FOLLOW;
                $datarray["gravity"] = GRAVITY_ACTIVITY;
-               $datarray["parent-uri"] = $parent_item["uri"];
+               $datarray['thr-parent'] = $toplevel_parent_item['uri'];
 
                $datarray["object-type"] = Activity\ObjectType::NOTE;
 
@@ -2286,14 +1965,19 @@ class Diaspora
                // Diaspora doesn't provide a date for a participation
                $datarray["changed"] = $datarray["created"] = $datarray["edited"] = DateTimeFormat::utcNow();
 
+               if (Item::isTooOld($datarray)) {
+                       Logger::info('Participation is too old', ['created' => $datarray['created'], 'uid' => $datarray['uid'], 'guid' => $datarray['guid']]);
+                       return false;
+               }
+
                $message_id = Item::insert($datarray);
 
                Logger::info('Participation stored', ['id' => $message_id, 'guid' => $guid, 'parent_guid' => $parent_guid, 'author' => $author]);
 
                // Send all existing comments and likes to the requesting server
-               $comments = Item::select(['id', 'uri-id', 'parent-author-network', 'author-network', 'verb'],
-                       ['parent' => $parent_item['id'], 'gravity' => [GRAVITY_COMMENT, GRAVITY_ACTIVITY]]);
-               while ($comment = Item::fetch($comments)) {
+               $comments = Post::select(['id', 'uri-id', 'parent-author-network', 'author-network', 'verb'],
+                       ['parent' => $toplevel_parent_item['id'], 'gravity' => [GRAVITY_COMMENT, GRAVITY_ACTIVITY]]);
+               while ($comment = Post::fetch($comments)) {
                        if (in_array($comment['verb'], [Activity::FOLLOW, Activity::TAG])) {
                                Logger::info('participation messages are not relayed', ['item' => $comment['id']]);
                                continue;
@@ -2360,7 +2044,7 @@ class Diaspora
         */
        private static function receiveProfile(array $importer, $data)
        {
-               $author = strtolower(Strings::escapeTags(XML::unescape($data->author)));
+               $author = strtolower(XML::unescape($data->author));
 
                $contact = self::contactByHandle($importer["uid"], $author);
                if (!$contact) {
@@ -2425,9 +2109,9 @@ class Diaspora
                        $fields['bd'] = $birthday;
                }
 
-               DBA::update('contact', $fields, ['id' => $contact['id']]);
+               Contact::update($fields, ['id' => $contact['id']]);
 
-               Logger::log("Profile of contact ".$contact["id"]." stored for user ".$importer["uid"], Logger::DEBUG);
+               Logger::info("Profile of contact ".$contact["id"]." stored for user ".$importer["uid"]);
 
                return true;
        }
@@ -2489,7 +2173,7 @@ class Diaspora
                // That makes us friends.
                if ($contact) {
                        if ($following) {
-                               Logger::log("Author ".$author." (Contact ".$contact["id"].") wants to follow us.", Logger::DEBUG);
+                               Logger::info("Author ".$author." (Contact ".$contact["id"].") wants to follow us.");
                                self::receiveRequestMakeFriend($importer, $contact);
 
                                // refetch the contact array
@@ -2500,36 +2184,36 @@ class Diaspora
                                if (in_array($contact["rel"], [Contact::FRIEND])) {
                                        $user = DBA::selectFirst('user', [], ['uid' => $importer["uid"]]);
                                        if (DBA::isResult($user)) {
-                                               Logger::log("Sending share message to author ".$author." - Contact: ".$contact["id"]." - User: ".$importer["uid"], Logger::DEBUG);
+                                               Logger::info("Sending share message to author ".$author." - Contact: ".$contact["id"]." - User: ".$importer["uid"]);
                                                self::sendShare($user, $contact);
                                        }
                                }
                                return true;
                        } else {
-                               Logger::log("Author ".$author." doesn't want to follow us anymore.", Logger::DEBUG);
-                               Contact::removeFollower($importer, $contact);
+                               Logger::info("Author ".$author." doesn't want to follow us anymore.");
+                               Contact::removeFollower($contact);
                                return true;
                        }
                }
 
                if (!$following && $sharing && in_array($importer["page-flags"], [User::PAGE_FLAGS_SOAPBOX, User::PAGE_FLAGS_NORMAL])) {
-                       Logger::log("Author ".$author." wants to share with us - but doesn't want to listen. Request is ignored.", Logger::DEBUG);
+                       Logger::info("Author ".$author." wants to share with us - but doesn't want to listen. Request is ignored.");
                        return false;
                } elseif (!$following && !$sharing) {
-                       Logger::log("Author ".$author." doesn't want anything - and we don't know the author. Request is ignored.", Logger::DEBUG);
+                       Logger::info("Author ".$author." doesn't want anything - and we don't know the author. Request is ignored.");
                        return false;
                } elseif (!$following && $sharing) {
-                       Logger::log("Author ".$author." wants to share with us.", Logger::DEBUG);
+                       Logger::info("Author ".$author." wants to share with us.");
                } elseif ($following && $sharing) {
-                       Logger::log("Author ".$author." wants to have a bidirectional conection.", Logger::DEBUG);
+                       Logger::info("Author ".$author." wants to have a bidirectional conection.");
                } elseif ($following && !$sharing) {
-                       Logger::log("Author ".$author." wants to listen to us.", Logger::DEBUG);
+                       Logger::info("Author ".$author." wants to listen to us.");
                }
 
-               $ret = self::personByHandle($author);
+               $ret = FContact::getByURL($author);
 
                if (!$ret || ($ret["network"] != Protocol::DIASPORA)) {
-                       Logger::log("Cannot resolve diaspora handle ".$author." for ".$recipient);
+                       Logger::notice("Cannot resolve diaspora handle ".$author." for ".$recipient);
                        return false;
                }
 
@@ -2575,18 +2259,18 @@ class Diaspora
        public static function originalItem($guid, $orig_author)
        {
                if (empty($guid)) {
-                       Logger::log('Empty guid. Quitting.');
+                       Logger::notice('Empty guid. Quitting.');
                        return false;
                }
 
                // Do we already have this item?
-               $fields = ['body', 'title', 'attach', 'app', 'created', 'object-type', 'uri', 'guid',
-                       'author-name', 'author-link', 'author-avatar', 'plink'];
+               $fields = ['body', 'title', 'app', 'created', 'object-type', 'uri', 'guid',
+                       'author-name', 'author-link', 'author-avatar', 'plink', 'uri-id'];
                $condition = ['guid' => $guid, 'visible' => true, 'deleted' => false, 'private' => [Item::PUBLIC, Item::UNLISTED]];
-               $item = Item::selectFirst($fields, $condition);
+               $item = Post::selectFirst($fields, $condition);
 
                if (DBA::isResult($item)) {
-                       Logger::log("reshared message ".$guid." already exists on system.");
+                       Logger::notice("reshared message ".$guid." already exists on system.");
 
                        // Maybe it is already a reshared item?
                        // Then refetch the content, if it is a reshare from a reshare.
@@ -2598,9 +2282,6 @@ class Diaspora
 
                                $item["body"] = self::replacePeopleGuid($item["body"], $item["author-link"]);
 
-                               // Add OEmbed and other information to the body
-                               $item["body"] = PageInfo::searchAndAppendToBody($item["body"], false, true);
-
                                return $item;
                        } else {
                                return $item;
@@ -2609,25 +2290,25 @@ class Diaspora
 
                if (!DBA::isResult($item)) {
                        if (empty($orig_author)) {
-                               Logger::log('Empty author for guid ' . $guid . '. Quitting.');
+                               Logger::notice('Empty author for guid ' . $guid . '. Quitting.');
                                return false;
                        }
 
                        $server = "https://".substr($orig_author, strpos($orig_author, "@") + 1);
-                       Logger::log("1st try: reshared message ".$guid." will be fetched via SSL from the server ".$server);
+                       Logger::notice("1st try: reshared message ".$guid." will be fetched via SSL from the server ".$server);
                        $stored = self::storeByGuid($guid, $server);
 
                        if (!$stored) {
                                $server = "http://".substr($orig_author, strpos($orig_author, "@") + 1);
-                               Logger::log("2nd try: reshared message ".$guid." will be fetched without SSL from the server ".$server);
+                               Logger::notice("2nd try: reshared message ".$guid." will be fetched without SSL from the server ".$server);
                                $stored = self::storeByGuid($guid, $server);
                        }
 
                        if ($stored) {
-                               $fields = ['body', 'title', 'attach', 'app', 'created', 'object-type', 'uri', 'guid',
-                                       'author-name', 'author-link', 'author-avatar', 'plink'];
+                               $fields = ['body', 'title', 'app', 'created', 'object-type', 'uri', 'guid',
+                                       'author-name', 'author-link', 'author-avatar', 'plink', 'uri-id'];
                                $condition = ['guid' => $guid, 'visible' => true, 'deleted' => false, 'private' => [Item::PUBLIC, Item::UNLISTED]];
-                               $item = Item::selectFirst($fields, $condition);
+                               $item = Post::selectFirst($fields, $condition);
 
                                if (DBA::isResult($item)) {
                                        // If it is a reshared post from another network then reformat to avoid display problems with two share elements
@@ -2653,7 +2334,7 @@ class Diaspora
         */
        private static function addReshareActivity($item, $parent_message_id, $guid, $author)
        {
-               $parent = Item::selectFirst(['uri', 'guid'], ['id' => $parent_message_id]);
+               $parent = Post::selectFirst(['uri', 'guid'], ['id' => $parent_message_id]);
 
                $datarray = [];
 
@@ -2669,18 +2350,25 @@ class Diaspora
 
                $datarray['guid'] = $parent['guid'] . '-' . $guid;
                $datarray['uri'] = self::getUriFromGuid($author, $datarray['guid']);
-               $datarray['parent-uri'] = $parent['uri'];
+               $datarray['thr-parent'] = $parent['uri'];
 
                $datarray['verb'] = $datarray['body'] = Activity::ANNOUNCE;
                $datarray['gravity'] = GRAVITY_ACTIVITY;
                $datarray['object-type'] = Activity\ObjectType::NOTE;
 
                $datarray['protocol'] = $item['protocol'];
+               $datarray['source'] = $item['source'];
+               $datarray['direction'] = $item['direction'];
 
                $datarray['plink'] = self::plink($author, $datarray['guid']);
                $datarray['private'] = $item['private'];
                $datarray['changed'] = $datarray['created'] = $datarray['edited'] = $item['created'];
 
+               if (Item::isTooOld($datarray)) {
+                       Logger::info('Reshare activity is too old', ['created' => $datarray['created'], 'uid' => $datarray['uid'], 'guid' => $datarray['guid']]);
+                       return false;
+               }
+
                $message_id = Item::insert($datarray);
 
                if ($message_id) {
@@ -2702,21 +2390,25 @@ class Diaspora
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       private static function receiveReshare(array $importer, $data, $xml)
+       private static function receiveReshare(array $importer, $data, $xml, bool $fetched)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
-               $guid = Strings::escapeTags(XML::unescape($data->guid));
-               $created_at = DateTimeFormat::utc(Strings::escapeTags(XML::unescape($data->created_at)));
-               $root_author = Strings::escapeTags(XML::unescape($data->root_author));
-               $root_guid = Strings::escapeTags(XML::unescape($data->root_guid));
+               $author = XML::unescape($data->author);
+               $guid = XML::unescape($data->guid);
+               $created_at = DateTimeFormat::utc(XML::unescape($data->created_at));
+               $root_author = XML::unescape($data->root_author);
+               $root_guid = XML::unescape($data->root_guid);
                /// @todo handle unprocessed property "provider_display_name"
-               $public = Strings::escapeTags(XML::unescape($data->public));
+               $public = XML::unescape($data->public);
 
                $contact = self::allowedContactByHandle($importer, $author, false);
                if (!$contact) {
                        return false;
                }
 
+               if (!empty($contact['gsid'])) {
+                       GServer::setProtocol($contact['gsid'], Post\DeliveryData::DIASPORA);
+               }
+
                $message_id = self::messageExists($importer["uid"], $guid);
                if ($message_id) {
                        return true;
@@ -2727,6 +2419,10 @@ class Diaspora
                        return false;
                }
 
+               if (empty($original_item['plink'])) {
+                       $original_item['plink'] = self::plink($root_author, $root_guid);
+               }
+
                $datarray = [];
 
                $datarray["uid"] = $importer["uid"];
@@ -2740,7 +2436,7 @@ class Diaspora
                $datarray["owner-id"] = $datarray["author-id"];
 
                $datarray["guid"] = $guid;
-               $datarray["uri"] = $datarray["parent-uri"] = self::getUriFromGuid($author, $guid);
+               $datarray["uri"] = $datarray["thr-parent"] = self::getUriFromGuid($author, $guid);
                $datarray['uri-id'] = ItemURI::insert(['uri' => $datarray['uri'], 'guid' => $datarray['guid']]);
 
                $datarray["verb"] = Activity::POST;
@@ -2748,6 +2444,7 @@ class Diaspora
 
                $datarray["protocol"] = Conversation::PARCEL_DIASPORA;
                $datarray["source"] = $xml;
+               $datarray["direction"] = $fetched ? Conversation::PULL : Conversation::PUSH;
 
                /// @todo Copy tag data from original post
 
@@ -2768,7 +2465,6 @@ class Diaspora
 
                Tag::storeFromBody($datarray['uri-id'], $datarray["body"]);
 
-               $datarray["attach"] = $original_item["attach"];
                $datarray["app"]  = $original_item["app"];
 
                $datarray["plink"] = self::plink($author, $guid);
@@ -2778,6 +2474,12 @@ class Diaspora
                $datarray["object-type"] = $original_item["object-type"];
 
                self::fetchGuid($datarray);
+
+               if (Item::isTooOld($datarray)) {
+                       Logger::info('Reshare is too old', ['created' => $datarray['created'], 'uid' => $datarray['uid'], 'guid' => $datarray['guid']]);
+                       return false;
+               }
+
                $message_id = Item::insert($datarray);
 
                self::sendParticipation($contact, $datarray);
@@ -2788,7 +2490,7 @@ class Diaspora
                }
 
                if ($message_id) {
-                       Logger::log("Stored reshare ".$datarray["guid"]." with message id ".$message_id, Logger::DEBUG);
+                       Logger::info("Stored reshare ".$datarray["guid"]." with message id ".$message_id);
                        if ($datarray['uid'] == 0) {
                                Item::distribute($message_id);
                        }
@@ -2810,13 +2512,13 @@ class Diaspora
         */
        private static function itemRetraction(array $importer, array $contact, $data)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
-               $target_guid = Strings::escapeTags(XML::unescape($data->target_guid));
-               $target_type = Strings::escapeTags(XML::unescape($data->target_type));
+               $author = XML::unescape($data->author);
+               $target_guid = XML::unescape($data->target_guid);
+               $target_type = XML::unescape($data->target_type);
 
-               $person = self::personByHandle($author);
+               $person = FContact::getByURL($author);
                if (!is_array($person)) {
-                       Logger::log("unable to find author detail for ".$author);
+                       Logger::notice("unable to find author detail for ".$author);
                        return false;
                }
 
@@ -2825,7 +2527,7 @@ class Diaspora
                }
 
                // Fetch items that are about to be deleted
-               $fields = ['uid', 'id', 'parent', 'parent-uri', 'author-link', 'file'];
+               $fields = ['uid', 'id', 'parent', 'author-link', 'uri-id'];
 
                // When we receive a public retraction, we delete every item that we find.
                if ($importer['uid'] == 0) {
@@ -2834,31 +2536,32 @@ class Diaspora
                        $condition = ['guid' => $target_guid, 'deleted' => false, 'uid' => $importer['uid']];
                }
 
-               $r = Item::select($fields, $condition);
+               $r = Post::select($fields, $condition);
                if (!DBA::isResult($r)) {
-                       Logger::log("Target guid ".$target_guid." was not found on this system for user ".$importer['uid'].".");
+                       Logger::notice("Target guid ".$target_guid." was not found on this system for user ".$importer['uid'].".");
                        return false;
                }
 
-               while ($item = Item::fetch($r)) {
-                       if (strstr($item['file'], '[')) {
-                               Logger::log("Target guid " . $target_guid . " for user " . $item['uid'] . " is filed. So it won't be deleted.", Logger::DEBUG);
+               while ($item = Post::fetch($r)) {
+                       if (DBA::exists('post-category', ['uri-id' => $item['uri-id'], 'uid' => $item['uid'], 'type' => Post\Category::FILE])) {
+                               Logger::info("Target guid " . $target_guid . " for user " . $item['uid'] . " is filed. So it won't be deleted.");
                                continue;
                        }
 
                        // Fetch the parent item
-                       $parent = Item::selectFirst(['author-link'], ['id' => $item['parent']]);
+                       $parent = Post::selectFirst(['author-link'], ['id' => $item['parent']]);
 
                        // Only delete it if the parent author really fits
                        if (!Strings::compareLink($parent["author-link"], $contact["url"]) && !Strings::compareLink($item["author-link"], $contact["url"])) {
-                               Logger::log("Thread author ".$parent["author-link"]." and item author ".$item["author-link"]." don't fit to expected contact ".$contact["url"], Logger::DEBUG);
+                               Logger::info("Thread author ".$parent["author-link"]." and item author ".$item["author-link"]." don't fit to expected contact ".$contact["url"]);
                                continue;
                        }
 
                        Item::markForDeletion(['id' => $item['id']]);
 
-                       Logger::log("Deleted target ".$target_guid." (".$item["id"].") from user ".$item["uid"]." parent: ".$item['parent'], Logger::DEBUG);
+                       Logger::info("Deleted target ".$target_guid." (".$item["id"].") from user ".$item["uid"]." parent: ".$item['parent']);
                }
+               DBA::close($r);
 
                return true;
        }
@@ -2875,11 +2578,11 @@ class Diaspora
         */
        private static function receiveRetraction(array $importer, $sender, $data)
        {
-               $target_type = Strings::escapeTags(XML::unescape($data->target_type));
+               $target_type = XML::unescape($data->target_type);
 
                $contact = self::contactByHandle($importer["uid"], $sender);
                if (!$contact && (in_array($target_type, ["Contact", "Person"]))) {
-                       Logger::log("cannot find contact for sender: ".$sender." and user ".$importer["uid"]);
+                       Logger::notice("cannot find contact for sender: ".$sender." and user ".$importer["uid"]);
                        return false;
                }
 
@@ -2887,7 +2590,7 @@ class Diaspora
                        $contact = [];
                }
 
-               Logger::log("Got retraction for ".$target_type.", sender ".$sender." and user ".$importer["uid"], Logger::DEBUG);
+               Logger::info("Got retraction for ".$target_type.", sender ".$sender." and user ".$importer["uid"]);
 
                switch ($target_type) {
                        case "Comment":
@@ -2903,37 +2606,84 @@ class Diaspora
                                break;
 
                        default:
-                               Logger::log("Unknown target type ".$target_type);
+                               Logger::notice("Unknown target type ".$target_type);
                                return false;
                }
                return true;
        }
 
+       /**
+        * Checks if an incoming message is wanted
+        *
+        * @param string $url
+        * @param integer $uriid
+        * @param string $author
+        * @param string $body
+        * @return boolean Is the message wanted?
+        */
+       private static function isSolicitedMessage(string $url, int $uriid, string $author, string $body)
+       {
+               $contact = Contact::getByURL($author);
+               if (DBA::exists('contact', ["`nurl` = ? AND `uid` != ? AND `rel` IN (?, ?)",
+                       $contact['nurl'], 0, Contact::FRIEND, Contact::SHARING])) {
+                       Logger::info('Author has got followers - accepted', ['url' => $url, 'author' => $author]);
+                       return true;
+               }
+
+               $taglist = Tag::getByURIId($uriid, [Tag::HASHTAG]);
+               $tags = array_column($taglist, 'name');
+               return Relay::isSolicitedPost($tags, $body, $contact['id'], $url, Protocol::DIASPORA);
+       }
+
+       /**
+        * Store an attached photo in the post-media table
+        *
+        * @param int $uriid
+        * @param object $photo
+        * @return void
+        */
+       private static function storePhotoAsMedia(int $uriid, $photo)
+       {
+               $data = [];
+               $data['uri-id'] = $uriid;
+               $data['type'] = Post\Media::IMAGE;
+               $data['url'] = XML::unescape($photo->remote_photo_path) . XML::unescape($photo->remote_photo_name);
+               $data['height'] = (int)XML::unescape($photo->height ?? 0);
+               $data['width'] = (int)XML::unescape($photo->width ?? 0);
+               $data['description'] = XML::unescape($photo->text ?? '');
+
+               Post\Media::insert($data);
+       }
+
        /**
         * Receives status messages
         *
         * @param array            $importer Array of the importer user
         * @param SimpleXMLElement $data     The message object
         * @param string           $xml      The original XML of the message
-        *
+        * @param bool             $fetched  The message had been fetched and not pushed
         * @return int The message id of the newly created item
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
-       private static function receiveStatusMessage(array $importer, SimpleXMLElement $data, $xml)
+       private static function receiveStatusMessage(array $importer, SimpleXMLElement $data, $xml, bool $fetched)
        {
-               $author = Strings::escapeTags(XML::unescape($data->author));
-               $guid = Strings::escapeTags(XML::unescape($data->guid));
-               $created_at = DateTimeFormat::utc(Strings::escapeTags(XML::unescape($data->created_at)));
-               $public = Strings::escapeTags(XML::unescape($data->public));
+               $author = XML::unescape($data->author);
+               $guid = XML::unescape($data->guid);
+               $created_at = DateTimeFormat::utc(XML::unescape($data->created_at));
+               $public = XML::unescape($data->public);
                $text = XML::unescape($data->text);
-               $provider_display_name = Strings::escapeTags(XML::unescape($data->provider_display_name));
+               $provider_display_name = XML::unescape($data->provider_display_name);
 
                $contact = self::allowedContactByHandle($importer, $author, false);
                if (!$contact) {
                        return false;
                }
 
+               if (!empty($contact['gsid'])) {
+                       GServer::setProtocol($contact['gsid'], Post\DeliveryData::DIASPORA);
+               }
+
                $message_id = self::messageExists($importer["uid"], $guid);
                if ($message_id) {
                        return true;
@@ -2942,34 +2692,34 @@ class Diaspora
                $address = [];
                if ($data->location) {
                        foreach ($data->location->children() as $fieldname => $data) {
-                               $address[$fieldname] = Strings::escapeTags(XML::unescape($data));
+                               $address[$fieldname] = XML::unescape($data);
                        }
                }
 
-               $body = Markdown::toBBCode($text);
+               $raw_body = $body = Markdown::toBBCode($text);
 
                $datarray = [];
 
+               $datarray["guid"] = $guid;
+               $datarray["uri"] = $datarray["thr-parent"] = self::getUriFromGuid($author, $guid);
+               $datarray['uri-id'] = ItemURI::insert(['uri' => $datarray['uri'], 'guid' => $datarray['guid']]);
+
                // Attach embedded pictures to the body
                if ($data->photo) {
                        foreach ($data->photo as $photo) {
-                               $body = "[img]".XML::unescape($photo->remote_photo_path).
-                                       XML::unescape($photo->remote_photo_name)."[/img]\n".$body;
+                               self::storePhotoAsMedia($datarray['uri-id'], $photo);
                        }
 
                        $datarray["object-type"] = Activity\ObjectType::IMAGE;
+                       $datarray["post-type"] = Item::PT_IMAGE;
                } else {
                        $datarray["object-type"] = Activity\ObjectType::NOTE;
-
-                       // Add OEmbed and other information to the body
-                       if (!self::isHubzilla($contact["url"])) {
-                               $body = PageInfo::searchAndAppendToBody($body, false, true);
-                       }
+                       $datarray["post-type"] = Item::PT_NOTE;
                }
 
                /// @todo enable support for polls
                //if ($data->poll) {
-               //      foreach ($data->poll AS $poll)
+               //      foreach ($data->poll as $poll)
                //              print_r($poll);
                //      die("poll!\n");
                //}
@@ -2986,21 +2736,30 @@ class Diaspora
                $datarray["owner-link"] = $datarray["author-link"];
                $datarray["owner-id"] = $datarray["author-id"];
 
-               $datarray["guid"] = $guid;
-               $datarray["uri"] = $datarray["parent-uri"] = self::getUriFromGuid($author, $guid);
-               $datarray['uri-id'] = ItemURI::insert(['uri' => $datarray['uri'], 'guid' => $datarray['guid']]);
-
                $datarray["verb"] = Activity::POST;
                $datarray["gravity"] = GRAVITY_PARENT;
 
                $datarray["protocol"] = Conversation::PARCEL_DIASPORA;
                $datarray["source"] = $xml;
+               $datarray["direction"] = $fetched ? Conversation::PULL : Conversation::PUSH;
+
+               if ($fetched) {
+                       $datarray["post-reason"] = Item::PR_FETCHED;
+               } elseif ($datarray["uid"] == 0) {
+                       $datarray["post-reason"] = Item::PR_GLOBAL;
+               }
 
                $datarray["body"] = self::replacePeopleGuid($body, $contact["url"]);
+               $datarray["raw-body"] = self::replacePeopleGuid($raw_body, $contact["url"]);
 
                self::storeMentions($datarray['uri-id'], $text);
                Tag::storeRawTagsFromBody($datarray['uri-id'], $datarray["body"]);
 
+               if (!$fetched && !self::isSolicitedMessage($datarray["uri"], $datarray['uri-id'], $author, $body)) {
+                       DBA::delete('item-uri', ['uri' => $datarray['uri']]);
+                       return false;
+               }
+
                if ($provider_display_name != "") {
                        $datarray["app"] = $provider_display_name;
                }
@@ -3018,12 +2777,18 @@ class Diaspora
                }
 
                self::fetchGuid($datarray);
+
+               if (Item::isTooOld($datarray)) {
+                       Logger::info('Status is too old', ['created' => $datarray['created'], 'uid' => $datarray['uid'], 'guid' => $datarray['guid']]);
+                       return false;
+               }
+
                $message_id = Item::insert($datarray);
 
                self::sendParticipation($contact, $datarray);
 
                if ($message_id) {
-                       Logger::log("Stored item ".$datarray["guid"]." with message id ".$message_id, Logger::DEBUG);
+                       Logger::info("Stored item ".$datarray["guid"]." with message id ".$message_id);
                        if ($datarray['uid'] == 0) {
                                Item::distribute($message_id);
                        }
@@ -3077,17 +2842,17 @@ class Diaspora
         */
        public static function encodePrivateData($msg, array $user, array $contact, $prvkey, $pubkey)
        {
-               Logger::log("Message: ".$msg, Logger::DATA);
+               Logger::debug("Message: ".$msg);
 
                // without a public key nothing will work
                if (!$pubkey) {
-                       Logger::log("pubkey missing: contact id: ".$contact["id"]);
+                       Logger::notice("pubkey missing: contact id: ".$contact["id"]);
                        return false;
                }
 
-               $aes_key = openssl_random_pseudo_bytes(32);
+               $aes_key = random_bytes(32);
                $b_aes_key = base64_encode($aes_key);
-               $iv = openssl_random_pseudo_bytes(16);
+               $iv = random_bytes(16);
                $b_iv = base64_encode($iv);
 
                $ciphertext = self::aesEncrypt($aes_key, $iv, $msg);
@@ -3217,7 +2982,7 @@ class Diaspora
                // We always try to use the data from the fcontact table.
                // This is important for transmitting data to Friendica servers.
                if (!empty($contact['addr'])) {
-                       $fcontact = self::personByHandle($contact['addr']);
+                       $fcontact = FContact::getByURL($contact['addr']);
                        if (!empty($fcontact)) {
                                $dest_url = ($public_batch ? $fcontact["batch"] : $fcontact["notify"]);
                        }
@@ -3228,23 +2993,23 @@ class Diaspora
                }
 
                if (!$dest_url) {
-                       Logger::log("no url for contact: ".$contact["id"]." batch mode =".$public_batch);
+                       Logger::notice("no url for contact: ".$contact["id"]." batch mode =".$public_batch);
                        return 0;
                }
 
-               Logger::log("transmit: ".$logid."-".$guid." ".$dest_url);
+               Logger::notice("transmit: ".$logid."-".$guid." ".$dest_url);
 
                if (!intval(DI::config()->get("system", "diaspora_test"))) {
                        $content_type = (($public_batch) ? "application/magic-envelope+xml" : "application/json");
 
-                       $postResult = DI::httpRequest()->post($dest_url . "/", $envelope, ["Content-Type: " . $content_type]);
+                       $postResult = DI::httpClient()->post($dest_url . "/", $envelope, ['Content-Type' => $content_type]);
                        $return_code = $postResult->getReturnCode();
                } else {
-                       Logger::log("test_mode");
+                       Logger::notice("test_mode");
                        return 200;
                }
 
-               Logger::log("transmit: ".$logid."-".$guid." to ".$dest_url." returns: ".$return_code);
+               Logger::notice("transmit: ".$logid."-".$guid." to ".$dest_url." returns: ".$return_code);
 
                return $return_code ? $return_code : -1;
        }
@@ -3283,19 +3048,32 @@ class Diaspora
        {
                $msg = self::buildPostXml($type, $message);
 
-               Logger::log('message: '.$msg, Logger::DATA);
-               Logger::log('send guid '.$guid, Logger::DEBUG);
-
                // Fallback if the private key wasn't transmitted in the expected field
                if (empty($owner['uprvkey'])) {
                        $owner['uprvkey'] = $owner['prvkey'];
                }
 
-               $envelope = self::buildMessage($msg, $owner, $contact, $owner['uprvkey'], $contact['pubkey'], $public_batch);
+               // When sending content to Friendica contacts using the Diaspora protocol
+               // we have to fetch the public key from the fcontact.
+               // This is due to the fact that legacy DFRN had unique keys for every contact.
+               $pubkey = $contact['pubkey'];
+               if (!empty($contact['addr'])) {
+                       $fcontact = FContact::getByURL($contact['addr']);
+                       if (!empty($fcontact)) {
+                               $pubkey = $fcontact['pubkey'];
+                       }
+               } else {
+                       // The "addr" field should always be filled.
+                       // If this isn't the case, it will raise a notice some lines later.
+                       // And in the log we will see where it came from and we can handle it there.
+                       Logger::notice('Empty addr', ['contact' => $contact ?? [], 'callstack' => System::callstack(20)]);
+               }
+
+               $envelope = self::buildMessage($msg, $owner, $contact, $owner['uprvkey'], $pubkey, $public_batch);
 
                $return_code = self::transmit($owner, $contact, $envelope, $public_batch, $guid);
 
-               Logger::log("guid: ".$guid." result ".$return_code, Logger::DEBUG);
+               Logger::info('Transmitted message', ['owner' => $owner['uid'], 'target' => $contact['addr'], 'type' => $type, 'guid' => $guid, 'result' => $return_code]);
 
                return $return_code;
        }
@@ -3327,8 +3105,10 @@ class Diaspora
                // In fact it doesn't matter which user sends this - but it is needed by the protocol.
                // If the item belongs to a user, we take this user id.
                if ($item['uid'] == 0) {
-                       $condition = ['verified' => true, 'blocked' => false, 'account_removed' => false, 'account_expired' => false];
-                       $first_user = DBA::selectFirst('user', ['uid'], $condition);
+                       // @todo Possibly use an administrator account?
+                       $condition = ['verified' => true, 'blocked' => false,
+                               'account_removed' => false, 'account_expired' => false, 'account-type' => User::ACCOUNT_TYPE_PERSON];
+                       $first_user = DBA::selectFirst('user', ['uid'], $condition, ['order' => ['uid']]);
                        $owner = User::getOwnerDataById($first_user['uid']);
                } else {
                        $owner = User::getOwnerDataById($item['uid']);
@@ -3341,7 +3121,7 @@ class Diaspora
                                "parent_type" => "Post",
                                "parent_guid" => $item["guid"]];
 
-               Logger::log("Send participation for ".$item["guid"]." by ".$author, Logger::DEBUG);
+               Logger::info("Send participation for ".$item["guid"]." by ".$author);
 
                // It doesn't matter what we store, we only want to avoid sending repeated notifications for the same item
                DI::cache()->set($cachekey, $item["guid"], Duration::QUARTER_HOUR);
@@ -3469,7 +3249,7 @@ class Diaspora
 
                if (!empty($reshared['guid']) && $complete) {
                        $condition = ['guid' => $reshared['guid'], 'network' => [Protocol::DFRN, Protocol::DIASPORA]];
-                       $item = Item::selectFirst(['contact-id'], $condition);
+                       $item = Post::selectFirst(['contact-id'], $condition);
                        if (DBA::isResult($item)) {
                                $ret = [];
                                $ret["root_handle"] = self::handleFromContact($item["contact-id"]);
@@ -3510,29 +3290,18 @@ class Diaspora
         */
        private static function buildEvent($event_id)
        {
-               $r = q("SELECT `guid`, `uid`, `start`, `finish`, `nofinish`, `summary`, `desc`, `location`, `adjust` FROM `event` WHERE `id` = %d", intval($event_id));
-               if (!DBA::isResult($r)) {
+               $event = DBA::selectFirst('event', [], ['id' => $event_id]);
+               if (!DBA::isResult($event)) {
                        return [];
                }
 
-               $event = $r[0];
-
                $eventdata = [];
 
-               $r = q("SELECT `timezone` FROM `user` WHERE `uid` = %d", intval($event['uid']));
-               if (!DBA::isResult($r)) {
-                       return [];
-               }
-
-               $user = $r[0];
-
-               $r = q("SELECT `addr`, `nick` FROM `contact` WHERE `uid` = %d AND `self`", intval($event['uid']));
-               if (!DBA::isResult($r)) {
+               $owner = User::getOwnerDataById($event['uid']);
+               if (!$owner) {
                        return [];
                }
 
-               $owner = $r[0];
-
                $eventdata['author'] = self::myHandle($owner);
 
                if ($event['guid']) {
@@ -3545,15 +3314,12 @@ class Diaspora
                $eventdata["all_day"] = "false";
 
                $eventdata['timezone'] = 'UTC';
-               if (!$event['adjust'] && $user['timezone']) {
-                       $eventdata['timezone'] = $user['timezone'];
-               }
 
                if ($event['start']) {
-                       $eventdata['start'] = DateTimeFormat::convert($event['start'], "UTC", $eventdata['timezone'], $mask);
+                       $eventdata['start'] = DateTimeFormat::utc($event['start'], $mask);
                }
                if ($event['finish'] && !$event['nofinish']) {
-                       $eventdata['end'] = DateTimeFormat::convert($event['finish'], "UTC", $eventdata['timezone'], $mask);
+                       $eventdata['end'] = DateTimeFormat::utc($event['finish'], $mask);
                }
                if ($event['summary']) {
                        $eventdata['summary'] = html_entity_decode(BBCode::toMarkdown($event['summary']));
@@ -3620,7 +3386,7 @@ class Diaspora
                        $type = "reshare";
                } else {
                        $title = $item["title"];
-                       $body = $item["body"];
+                       $body = Post\Media::addAttachmentsToBody($item['uri-id'], $item['body']);
 
                        // Fetch the title from an attached link - if there is one
                        if (empty($item["title"]) && DI::pConfig()->get($owner['uid'], 'system', 'attach_link_title')) {
@@ -3631,7 +3397,6 @@ class Diaspora
                        }
 
                        if ($item['author-link'] != $item['owner-link']) {
-                               require_once 'mod/share.php';
                                $body = BBCode::getShareOpeningTag($item['author-name'], $item['author-link'], $item['author-avatar'],
                                        $item['plink'], $item['created']) . $body . '[/share]';
                        }
@@ -3644,13 +3409,11 @@ class Diaspora
                                $body = "### ".html_entity_decode($title)."\n\n".$body;
                        }
 
-                       if ($item["attach"]) {
-                               $cnt = preg_match_all('/href=\"(.*?)\"(.*?)title=\"(.*?)\"/ism', $item["attach"], $matches, PREG_SET_ORDER);
-                               if ($cnt) {
-                                       $body .= "\n".DI::l10n()->t("Attachments:")."\n";
-                                       foreach ($matches as $mtch) {
-                                               $body .= "[".$mtch[3]."](".$mtch[1].")\n";
-                                       }
+                       $attachments = Post\Media::getByURIId($item['uri-id'], [Post\Media::DOCUMENT, Post\Media::TORRENT, Post\Media::UNKNOWN]);
+                       if (!empty($attachments)) {
+                               $body .= "\n".DI::l10n()->t("Attachments:")."\n";
+                               foreach ($attachments as $attachment) {
+                                       $body .= "[" . $attachment['description'] . "](" . $attachment['url'] . ")\n";
                                }
                        }
 
@@ -3749,12 +3512,12 @@ class Diaspora
         */
        private static function constructLike(array $item, array $owner)
        {
-               $parent = Item::selectFirst(['guid', 'uri', 'parent-uri'], ['uri' => $item["thr-parent"]]);
+               $parent = Post::selectFirst(['guid', 'uri', 'thr-parent'], ['uri' => $item["thr-parent"]]);
                if (!DBA::isResult($parent)) {
                        return false;
                }
 
-               $target_type = ($parent["uri"] === $parent["parent-uri"] ? "Post" : "Comment");
+               $target_type = ($parent["uri"] === $parent["thr-parent"] ? "Post" : "Comment");
                $positive = null;
                if ($item['verb'] === Activity::LIKE) {
                        $positive = "true";
@@ -3781,7 +3544,7 @@ class Diaspora
         */
        private static function constructAttend(array $item, array $owner)
        {
-               $parent = Item::selectFirst(['guid', 'uri', 'parent-uri'], ['uri' => $item["thr-parent"]]);
+               $parent = Post::selectFirst(['guid'], ['uri' => $item['thr-parent']]);
                if (!DBA::isResult($parent)) {
                        return false;
                }
@@ -3797,7 +3560,7 @@ class Diaspora
                                $attend_answer = 'tentative';
                                break;
                        default:
-                               Logger::log('Unknown verb '.$item['verb'].' in item '.$item['guid']);
+                               Logger::notice('Unknown verb '.$item['verb'].' in item '.$item['guid']);
                                return false;
                }
 
@@ -3826,7 +3589,7 @@ class Diaspora
                        return $result;
                }
 
-               $toplevel_item = Item::selectFirst(['guid', 'author-id', 'author-link'], ['id' => $item['parent'], 'parent' => $item['parent']]);
+               $toplevel_item = Post::selectFirst(['guid', 'author-id', 'author-link', 'gravity'], ['id' => $item['parent'], 'parent' => $item['parent']]);
                if (!DBA::isResult($toplevel_item)) {
                        Logger::error('Missing parent conversation item', ['parent' => $item['parent']]);
                        return false;
@@ -3834,10 +3597,10 @@ class Diaspora
 
                $thread_parent_item = $toplevel_item;
                if ($item['thr-parent'] != $item['parent-uri']) {
-                       $thread_parent_item = Item::selectFirst(['guid', 'author-id', 'author-link'], ['uri' => $item['thr-parent'], 'uid' => $item['uid']]);
+                       $thread_parent_item = Post::selectFirst(['guid', 'author-id', 'author-link', 'gravity'], ['uri' => $item['thr-parent'], 'uid' => $item['uid']]);
                }
 
-               $body = $item["body"];
+               $body = Post\Media::addAttachmentsToBody($item['uri-id'], $item['body']);
 
                // The replied to autor mention is prepended for clarity if:
                // - Item replied isn't yours
@@ -3845,6 +3608,7 @@ class Diaspora
                // - Implicit mentions are enabled
                if (
                        $item['author-id'] != $thread_parent_item['author-id']
+                       && ($thread_parent_item['gravity'] != GRAVITY_PARENT)
                        && (empty($item['uid']) || !Feature::isEnabled($item['uid'], 'explicit_mentions'))
                        && !DI::config()->get('system', 'disable_implicit_mentions')
                ) {
@@ -3909,51 +3673,6 @@ class Diaspora
                return self::buildAndTransmit($owner, $contact, $type, $message, $public_batch, $item["guid"]);
        }
 
-       /**
-        * Creates a message from a signature record entry
-        *
-        * @param array $item The item that will be exported
-        * @return array The message
-        */
-       private static function messageFromSignature(array $item)
-       {
-               // Split the signed text
-               $signed_parts = explode(";", $item['signed_text']);
-
-               if ($item["deleted"]) {
-                       $message = ["author" => $item['signer'],
-                                       "target_guid" => $signed_parts[0],
-                                       "target_type" => $signed_parts[1]];
-               } elseif (in_array($item["verb"], [Activity::LIKE, Activity::DISLIKE])) {
-                       $message = ["author" => $signed_parts[4],
-                                       "guid" => $signed_parts[1],
-                                       "parent_guid" => $signed_parts[3],
-                                       "parent_type" => $signed_parts[2],
-                                       "positive" => $signed_parts[0],
-                                       "author_signature" => $item['signature'],
-                                       "parent_author_signature" => ""];
-               } else {
-                       // Remove the comment guid
-                       $guid = array_shift($signed_parts);
-
-                       // Remove the parent guid
-                       $parent_guid = array_shift($signed_parts);
-
-                       // Remove the handle
-                       $handle = array_pop($signed_parts);
-
-                       $message = [
-                               "author" => $handle,
-                               "guid" => $guid,
-                               "parent_guid" => $parent_guid,
-                               "text" => implode(";", $signed_parts),
-                               "author_signature" => $item['signature'],
-                               "parent_author_signature" => ""
-                       ];
-               }
-               return $message;
-       }
-
        /**
         * Relays messages (like, comment, retraction) to other servers if we are the thread owner
         *
@@ -3975,7 +3694,7 @@ class Diaspora
                        $type = "comment";
                }
 
-               Logger::log("Got relayable data ".$type." for item ".$item["guid"]." (".$item["id"].")", Logger::DEBUG);
+               Logger::info("Got relayable data ".$type." for item ".$item["guid"]." (".$item["id"].")");
 
                $msg = json_decode($item['signed_text'], true);
 
@@ -3994,7 +3713,7 @@ class Diaspora
                                $message[$field] = $data;
                        }
                } else {
-                       Logger::log("Signature text for item ".$item["guid"]." (".$item["id"].") couldn't be extracted: ".$item['signed_text'], Logger::DEBUG);
+                       Logger::info("Signature text for item ".$item["guid"]." (".$item["id"].") couldn't be extracted: ".$item['signed_text']);
                }
 
                $message["parent_author_signature"] = self::signature($owner, $message);
@@ -4056,7 +3775,7 @@ class Diaspora
 
                $cnv = DBA::selectFirst('conv', [], ['id' => $item["convid"], 'uid' => $item["uid"]]);
                if (!DBA::isResult($cnv)) {
-                       Logger::log("conversation not found.");
+                       Logger::notice("conversation not found.");
                        return;
                }
 
@@ -4180,7 +3899,7 @@ class Diaspora
                        $dob = '';
 
                        if ($profile['dob'] && ($profile['dob'] > '0000-00-00')) {
-                               list($year, $month, $day) = sscanf($profile['dob'], '%4d-%2d-%2d');
+                               [$year, $month, $day] = sscanf($profile['dob'], '%4d-%2d-%2d');
                                if ($year < 1004) {
                                        $year = 1004;
                                }
@@ -4240,13 +3959,7 @@ class Diaspora
                }
 
                if (!$recips) {
-                       $recips = q(
-                               "SELECT `id`,`name`,`network`,`pubkey`,`notify` FROM `contact` WHERE `network` = '%s'
-                               AND `uid` = %d AND `rel` != %d",
-                               DBA::escape(Protocol::DIASPORA),
-                               intval($uid),
-                               intval(Contact::SHARING)
-                       );
+                       $recips = DBA::selectToArray('contact', [], ['network' => Protocol::DIASPORA, 'uid' => $uid, 'rel' => [Contact::FOLLOWER, Contact::FRIEND]]);
                }
 
                if (!$recips) {
@@ -4257,7 +3970,7 @@ class Diaspora
 
                // @ToDo Split this into single worker jobs
                foreach ($recips as $recip) {
-                       Logger::log("Send updated profile data for user ".$uid." to contact ".$recip["id"], Logger::DEBUG);
+                       Logger::info("Send updated profile data for user ".$uid." to contact ".$recip["id"]);
                        self::buildAndTransmit($owner, $recip, "profile", $message);
                }
        }
@@ -4296,30 +4009,41 @@ class Diaspora
        /**
         * Creates the signature for Comments that are created on our system
         *
-        * @param integer $uid  The user of that comment
         * @param array   $item Item array
         *
         * @return array Signed content
         * @throws \Exception
         */
-       public static function createCommentSignature($uid, array $item)
+       public static function createCommentSignature(array $item)
        {
+               if (!empty($item['author-link'])) {
+                       $url = $item['author-link'];
+               } else {
+                       $contact = Contact::getById($item['author-id'], ['url']);
+                       if (empty($contact['url'])) {
+                               Logger::warning('Author Contact not found', ['author-id' => $item['author-id']]);
+                               return false;
+                       }
+                       $url = $contact['url'];
+               }
+
+               $uid = User::getIdForURL($url);
+               if (empty($uid)) {
+                       Logger::info('No owner post, so not storing signature', ['url' => $contact['url']]);
+                       return false;
+               }
+
                $owner = User::getOwnerDataById($uid);
                if (empty($owner)) {
                        Logger::info('No owner post, so not storing signature');
                        return false;
                }
 
-               // This is a workaround for the behaviour of the "insert" function, see mod/item.php
-               $item['thr-parent'] = $item['parent-uri'];
-
-               $parent = Item::selectFirst(['parent-uri'], ['uri' => $item['parent-uri']]);
-               if (!DBA::isResult($parent)) {
-                       return;
+               // This is only needed for the automated tests
+               if (empty($owner['uprvkey'])) {
+                       return false;
                }
 
-               $item['parent-uri'] = $parent['parent-uri'];
-
                $message = self::constructComment($item, $owner);
                if ($message === false) {
                        return false;