]> git.mxchange.org Git - friendica.git/blobdiff - src/Worker/Delivery.php
Merge remote-tracking branch 'upstream/develop' into write-tags
[friendica.git] / src / Worker / Delivery.php
index 541583be5a51512acd02d493cc71f80313a91342..94a0f4902b0ba3c441e0cfe915b6c740fd985711 100644 (file)
@@ -1,25 +1,40 @@
 <?php
 /**
- * @file src/Worker/Delivery.php
+ * @copyright Copyright (C) 2020, Friendica
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
  */
+
 namespace Friendica\Worker;
 
-use Friendica\BaseObject;
-use Friendica\Core\Config;
-use Friendica\Core\L10n;
 use Friendica\Core\Logger;
 use Friendica\Core\Protocol;
-use Friendica\Core\System;
 use Friendica\Database\DBA;
+use Friendica\DI;
 use Friendica\Model;
 use Friendica\Protocol\DFRN;
 use Friendica\Protocol\Diaspora;
 use Friendica\Protocol\Email;
+use Friendica\Protocol\Activity;
 use Friendica\Util\Strings;
 use Friendica\Util\Network;
 use Friendica\Core\Worker;
 
-class Delivery extends BaseObject
+class Delivery
 {
        const MAIL          = 'mail';
        const SUGGESTION    = 'suggest';
@@ -139,7 +154,7 @@ class Delivery extends BaseObject
                        // if $parent['wall'] == 1 we will already have the parent message in our array
                        // and we will relay the whole lot.
 
-                       $localhost = self::getApp()->getHostName();
+                       $localhost = DI::baseUrl()->getHostname();
                        if (strpos($localhost, ':')) {
                                $localhost = substr($localhost, 0, strpos($localhost, ':'));
                        }
@@ -161,7 +176,7 @@ class Delivery extends BaseObject
                                && empty($parent['allow_gid'])
                                && empty($parent['deny_cid'])
                                && empty($parent['deny_gid'])
-                               && !$parent["private"]) {
+                               && ($parent["private"] != Model\Item::PRIVATE)) {
                                $public_message = true;
                        }
                }
@@ -197,6 +212,11 @@ class Delivery extends BaseObject
                        $contact['network'] = Protocol::DIASPORA;
                }
 
+               // Ensure that local contacts are delivered locally
+               if (Model\Contact::isLocal($contact['url'])) {
+                       $contact['network'] = Protocol::DFRN;
+               }
+
                Logger::notice('Delivering', ['cmd' => $cmd, 'target' => $target_id, 'followup' => $followup, 'network' => $contact['network']]);
 
                switch ($contact['network']) {
@@ -235,7 +255,7 @@ class Delivery extends BaseObject
        }
 
        /**
-        * @brief Deliver content via DFRN
+        * Deliver content via DFRN
         *
         * @param string  $cmd            Command
         * @param array   $contact        Contact record of the receiver
@@ -250,6 +270,13 @@ class Delivery extends BaseObject
         */
        private static function deliverDFRN($cmd, $contact, $owner, $items, $target_item, $public_message, $top_level, $followup)
        {
+               // Transmit Diaspora reshares via Diaspora if the Friendica contact support Diaspora
+               if (Diaspora::isReshare($target_item['body']) && !empty(Diaspora::personByHandle($contact['addr'], false))) {
+                       Logger::info('Reshare will be transmitted via Diaspora', ['url' => $contact['url'], 'guid' => ($target_item['guid'] ?? '') ?: $target_item['id']]);
+                       self::deliverDiaspora($cmd, $contact, $owner, $items, $target_item, $public_message, $top_level, $followup);
+                       return;
+               }
+
                Logger::info('Deliver ' . (($target_item['guid'] ?? '') ?: $target_item['id']) . ' via DFRN to ' . (($contact['addr'] ?? '') ?: $contact['url']));
 
                if ($cmd == self::MAIL) {
@@ -280,11 +307,8 @@ class Delivery extends BaseObject
 
                Logger::debug('Notifier entry: ' . $contact["url"] . ' ' . (($target_item['guid'] ?? '') ?: $target_item['id']) . ' entry: ' . $atom);
 
-               $basepath =  implode('/', array_slice(explode('/', $contact['url']), 0, 3));
-
                // perform local delivery if we are on the same site
-
-               if (Strings::compareLink($basepath, System::baseUrl())) {
+               if (Model\Contact::isLocal($contact['url'])) {
                        $condition = ['nurl' => Strings::normaliseLink($contact['url']), 'self' => true];
                        $target_self = DBA::selectFirst('contact', ['uid'], $condition);
                        if (!DBA::isResult($target_self)) {
@@ -380,7 +404,7 @@ class Delivery extends BaseObject
        }
 
        /**
-        * @brief Deliver content via Diaspora
+        * Deliver content via Diaspora
         *
         * @param string  $cmd            Command
         * @param array   $contact        Contact record of the receiver
@@ -406,7 +430,7 @@ class Delivery extends BaseObject
 
                Logger::notice('Deliver via Diaspora', ['target' => $target_item['id'], 'guid' => $target_item['guid'], 'to' => $loc]);
 
-               if (Config::get('system', 'dfrn_only') || !Config::get('system', 'diaspora_enabled')) {
+               if (DI::config()->get('system', 'dfrn_only') || !DI::config()->get('system', 'diaspora_enabled')) {
                        return;
                }
 
@@ -475,22 +499,21 @@ class Delivery extends BaseObject
        }
 
        /**
-        * @brief Deliver content via mail
+        * Deliver content via mail
         *
         * @param string $cmd         Command
         * @param array  $contact     Contact record of the receiver
         * @param array  $owner       Owner record of the sender
         * @param array  $target_item Item record of the content
-        * @param array  $thr_parent  Item record of the thread parent
+        * @param array  $thr_parent  Item record of the direct parent in the thread
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         * @throws \ImagickException
         */
        private static function deliverMail($cmd, $contact, $owner, $target_item, $thr_parent)
        {
-               if (Config::get('system','dfrn_only')) {
+               if (DI::config()->get('system','dfrn_only')) {
                        return;
                }
-               // WARNING: does not currently convert to RFC2047 header encodings, etc.
 
                $addr = $contact['addr'];
                if (!strlen($addr)) {
@@ -501,6 +524,10 @@ class Delivery extends BaseObject
                        return;
                }
 
+               if ($target_item['verb'] != Activity::POST) {
+                       return;
+               }
+
                if (!empty($thr_parent['object'])) {
                        $data = json_decode($thr_parent['object'], true);
                        if (!empty($data['reply_to'])) {
@@ -525,7 +552,7 @@ class Delivery extends BaseObject
                        $reply_to = $mailacct['reply_to'];
                }
 
-               $subject  = ($target_item['title'] ? Email::encodeHeader($target_item['title'], 'UTF-8') : L10n::t("\x28no subject\x29"));
+               $subject  = ($target_item['title'] ? Email::encodeHeader($target_item['title'], 'UTF-8') : DI::l10n()->t("\x28no subject\x29"));
 
                // only expose our real email address to true friends
 
@@ -537,7 +564,8 @@ class Delivery extends BaseObject
                                $headers  = 'From: ' . Email::encodeHeader($local_user['username'],'UTF-8') . ' <' . $local_user['email'] . '>' . "\n";
                        }
                } else {
-                       $headers  = 'From: '. Email::encodeHeader($local_user['username'], 'UTF-8') . ' <noreply@' . self::getApp()->getHostName() . '>' . "\n";
+                       $sender = DI::config()->get('config', 'sender_email', 'noreply@' . DI::baseUrl()->getHostname());
+                       $headers  = 'From: '. Email::encodeHeader($local_user['username'], 'UTF-8') . ' <' . $sender . '>' . "\n";
                }
 
                $headers .= 'Message-Id: <' . Email::iri2msgid($target_item['uri']) . '>' . "\n";
@@ -545,7 +573,7 @@ class Delivery extends BaseObject
                if ($target_item['uri'] !== $target_item['parent-uri']) {
                        $headers .= 'References: <' . Email::iri2msgid($target_item['parent-uri']) . '>';
 
-                       // If Threading is enabled, write down the correct parent
+                       // Export more references on deeper nested threads
                        if (($target_item['thr-parent'] != '') && ($target_item['thr-parent'] != $target_item['parent-uri'])) {
                                $headers .= ' <' . Email::iri2msgid($target_item['thr-parent']) . '>';
                        }
@@ -575,7 +603,7 @@ class Delivery extends BaseObject
 
                Email::send($addr, $subject, $headers, $target_item);
 
-               Model\ItemDeliveryData::incrementQueueFailed($target_item['id']);
+               Model\ItemDeliveryData::incrementQueueDone($target_item['id'], Model\ItemDeliveryData::MAIL);
 
                Logger::info('Delivered via mail', ['guid' => $target_item['guid'], 'to' => $addr, 'subject' => $subject]);
        }