Merge pull request #6082 from annando/diasppora-delivery
Improved delivery
This commit is contained in:
commit
e8891e9ba4
1 changed files with 58 additions and 61 deletions
|
@ -58,7 +58,7 @@ class Notifier
|
||||||
{
|
{
|
||||||
$a = BaseObject::getApp();
|
$a = BaseObject::getApp();
|
||||||
|
|
||||||
Logger::log('notifier: invoked: '.$cmd.': '.$item_id, Logger::DEBUG);
|
Logger::log('Invoked: ' . $cmd . ': ' . $item_id, Logger::DEBUG);
|
||||||
|
|
||||||
$top_level = false;
|
$top_level = false;
|
||||||
$recipients = [];
|
$recipients = [];
|
||||||
|
@ -148,7 +148,7 @@ class Notifier
|
||||||
}
|
}
|
||||||
|
|
||||||
if ((count($items) == 1) && ($items[0]['id'] === $target_item['id']) && ($items[0]['uri'] === $items[0]['parent-uri'])) {
|
if ((count($items) == 1) && ($items[0]['id'] === $target_item['id']) && ($items[0]['uri'] === $items[0]['parent-uri'])) {
|
||||||
Logger::log('notifier: top level post');
|
Logger::log('Top level post');
|
||||||
$top_level = true;
|
$top_level = true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -174,7 +174,6 @@ class Notifier
|
||||||
|
|
||||||
$followup = false;
|
$followup = false;
|
||||||
$recipients_followup = [];
|
$recipients_followup = [];
|
||||||
$conversants = [];
|
|
||||||
|
|
||||||
if (!in_array($cmd, [Delivery::MAIL, Delivery::SUGGESTION, Delivery::RELOCATION])) {
|
if (!in_array($cmd, [Delivery::MAIL, Delivery::SUGGESTION, Delivery::RELOCATION])) {
|
||||||
$parent = $items[0];
|
$parent = $items[0];
|
||||||
|
@ -245,11 +244,10 @@ class Notifier
|
||||||
// local followup to remote post
|
// local followup to remote post
|
||||||
$followup = true;
|
$followup = true;
|
||||||
$public_message = false; // not public
|
$public_message = false; // not public
|
||||||
$conversant_str = DBA::escape($parent['contact-id']);
|
|
||||||
$recipients = [$parent['contact-id']];
|
$recipients = [$parent['contact-id']];
|
||||||
$recipients_followup = [$parent['contact-id']];
|
$recipients_followup = [$parent['contact-id']];
|
||||||
|
|
||||||
Logger::log('notifier: followup '.$target_item["guid"].' to '.$conversant_str, Logger::DEBUG);
|
Logger::log('Followup ' . $target_item['guid'] . ' to ' . $parent['contact-id'], Logger::DEBUG);
|
||||||
|
|
||||||
//if (!$target_item['private'] && $target_item['wall'] &&
|
//if (!$target_item['private'] && $target_item['wall'] &&
|
||||||
if (!$target_item['private'] &&
|
if (!$target_item['private'] &&
|
||||||
|
@ -279,7 +277,7 @@ class Notifier
|
||||||
$push_notify = false;
|
$push_notify = false;
|
||||||
}
|
}
|
||||||
|
|
||||||
Logger::log("Notify ".$target_item["guid"]." via PuSH: ".($push_notify?"Yes":"No"), Logger::DEBUG);
|
Logger::log('Notify ' . $target_item["guid"] .' via PuSH: ' . ($push_notify ? "Yes":"No"), Logger::DEBUG);
|
||||||
} else {
|
} else {
|
||||||
$followup = false;
|
$followup = false;
|
||||||
|
|
||||||
|
@ -288,7 +286,7 @@ class Notifier
|
||||||
// don't send deletions onward for other people's stuff
|
// don't send deletions onward for other people's stuff
|
||||||
|
|
||||||
if ($target_item['deleted'] && !intval($target_item['wall'])) {
|
if ($target_item['deleted'] && !intval($target_item['wall'])) {
|
||||||
Logger::log('notifier: ignoring delete notification for non-wall item');
|
Logger::log('Ignoring delete notification for non-wall item');
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -313,14 +311,12 @@ class Notifier
|
||||||
|
|
||||||
foreach ($items as $item) {
|
foreach ($items as $item) {
|
||||||
$recipients[] = $item['contact-id'];
|
$recipients[] = $item['contact-id'];
|
||||||
$conversants[] = $item['contact-id'];
|
|
||||||
// pull out additional tagged people to notify (if public message)
|
// pull out additional tagged people to notify (if public message)
|
||||||
if ($public_message && strlen($item['inform'])) {
|
if ($public_message && strlen($item['inform'])) {
|
||||||
$people = explode(',',$item['inform']);
|
$people = explode(',',$item['inform']);
|
||||||
foreach ($people as $person) {
|
foreach ($people as $person) {
|
||||||
if (substr($person,0,4) === 'cid:') {
|
if (substr($person,0,4) === 'cid:') {
|
||||||
$recipients[] = intval(substr($person,4));
|
$recipients[] = intval(substr($person,4));
|
||||||
$conversants[] = intval(substr($person,4));
|
|
||||||
} else {
|
} else {
|
||||||
$url_recipients[] = substr($person,4);
|
$url_recipients[] = substr($person,4);
|
||||||
}
|
}
|
||||||
|
@ -329,16 +325,12 @@ class Notifier
|
||||||
}
|
}
|
||||||
|
|
||||||
if (count($url_recipients)) {
|
if (count($url_recipients)) {
|
||||||
Logger::log('notifier: '.$target_item["guid"].' url_recipients ' . print_r($url_recipients,true));
|
Logger::log('Deliver ' . $target_item["guid"] . ' to _recipients ' . json_decode($url_recipients));
|
||||||
}
|
}
|
||||||
|
|
||||||
$conversants = array_unique($conversants);
|
|
||||||
|
|
||||||
$recipients = array_unique(array_merge($recipients, $allow_people, $allow_groups));
|
$recipients = array_unique(array_merge($recipients, $allow_people, $allow_groups));
|
||||||
$deny = array_unique(array_merge($deny_people, $deny_groups));
|
$deny = array_unique(array_merge($deny_people, $deny_groups));
|
||||||
$recipients = array_diff($recipients, $deny);
|
$recipients = array_diff($recipients, $deny);
|
||||||
|
|
||||||
$conversant_str = DBA::escape(implode(', ',$conversants));
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// If the thread parent is OStatus then do some magic to distribute the messages.
|
// If the thread parent is OStatus then do some magic to distribute the messages.
|
||||||
|
@ -403,7 +395,7 @@ class Notifier
|
||||||
}
|
}
|
||||||
|
|
||||||
if (($cmd == Delivery::RELOCATION)) {
|
if (($cmd == Delivery::RELOCATION)) {
|
||||||
$r = $recipients_relocate;
|
$contacts = $recipients_relocate;
|
||||||
} else {
|
} else {
|
||||||
if ($followup) {
|
if ($followup) {
|
||||||
$recipients = $recipients_followup;
|
$recipients = $recipients_followup;
|
||||||
|
@ -413,37 +405,19 @@ class Notifier
|
||||||
if (!empty($networks)) {
|
if (!empty($networks)) {
|
||||||
$condition['network'] = $networks;
|
$condition['network'] = $networks;
|
||||||
}
|
}
|
||||||
$contacts = DBA::select('contact', ['id', 'url', 'network'], $condition);
|
$result = DBA::select('contact', ['id', 'url', 'network', 'batch'], $condition);
|
||||||
$r = DBA::toArray($contacts);
|
$contacts = DBA::toArray($result);
|
||||||
}
|
}
|
||||||
|
|
||||||
// delivery loop
|
$conversants = [];
|
||||||
if (DBA::isResult($r)) {
|
$batch_delivery = false;
|
||||||
foreach ($r as $contact) {
|
|
||||||
Logger::log("Deliver ".$item_id." to ".$contact['url']." via network ".$contact['network'], Logger::DEBUG);
|
|
||||||
|
|
||||||
Worker::add(['priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true],
|
if ($public_message && !in_array($cmd, [Delivery::MAIL, Delivery::SUGGESTION]) && !$followup) {
|
||||||
'Delivery', $cmd, $item_id, (int)$contact['id']);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// send salmon slaps to mentioned remote tags (@foo@example.com) in OStatus posts
|
|
||||||
// They are especially used for notifications to OStatus users that don't follow us.
|
|
||||||
if (!Config::get('system', 'dfrn_only') && count($url_recipients) && ($public_message || $push_notify) && $normal_mode) {
|
|
||||||
$slap = OStatus::salmon($target_item, $owner);
|
|
||||||
foreach ($url_recipients as $url) {
|
|
||||||
if ($url) {
|
|
||||||
Logger::log('notifier: urldelivery: ' . $url);
|
|
||||||
$deliver_status = Salmon::slapper($owner, $url, $slap);
|
|
||||||
/// @TODO Redeliver/queue these items on failure, though there is no contact record
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if ($public_message) {
|
|
||||||
$r1 = [];
|
$r1 = [];
|
||||||
|
|
||||||
if ($diaspora_delivery) {
|
if ($diaspora_delivery) {
|
||||||
|
$batch_delivery = true;
|
||||||
|
|
||||||
$r1 = q("SELECT `batch`, ANY_VALUE(`id`) AS `id`, ANY_VALUE(`name`) AS `name`, ANY_VALUE(`network`) AS `network`
|
$r1 = q("SELECT `batch`, ANY_VALUE(`id`) AS `id`, ANY_VALUE(`name`) AS `name`, ANY_VALUE(`network`) AS `network`
|
||||||
FROM `contact` WHERE `network` = '%s' AND `batch` != ''
|
FROM `contact` WHERE `network` = '%s' AND `batch` != ''
|
||||||
AND `uid` = %d AND `rel` != %d AND NOT `blocked` AND NOT `pending` AND NOT `archive` GROUP BY `batch`",
|
AND `uid` = %d AND `rel` != %d AND NOT `blocked` AND NOT `pending` AND NOT `archive` GROUP BY `batch`",
|
||||||
|
@ -470,28 +444,51 @@ class Notifier
|
||||||
$r = array_merge($r2, $r1);
|
$r = array_merge($r2, $r1);
|
||||||
|
|
||||||
if (DBA::isResult($r)) {
|
if (DBA::isResult($r)) {
|
||||||
Logger::log('pubdeliver '.$target_item["guid"].': '.print_r($r,true), Logger::DEBUG);
|
|
||||||
|
|
||||||
foreach ($r as $rr) {
|
foreach ($r as $rr) {
|
||||||
// except for Diaspora batch jobs
|
$conversants[] = $rr['id'];
|
||||||
// Don't deliver to folks who have already been delivered to
|
Logger::log('Public delivery of item ' . $target_item["guid"] . ' (' . $item_id . ') to ' . json_encode($rr), Logger::DEBUG);
|
||||||
|
|
||||||
if (($rr['network'] !== Protocol::DIASPORA) && (in_array($rr['id'], $conversants))) {
|
|
||||||
Logger::log('notifier: already delivered id=' . $rr['id']);
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!in_array($cmd, [Delivery::MAIL, Delivery::SUGGESTION]) && !$followup) {
|
|
||||||
Logger::log('notifier: delivery agent: '.$rr['name'].' '.$rr['id'].' '.$rr['network'].' '.$target_item["guid"]);
|
|
||||||
Worker::add(['priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true],
|
Worker::add(['priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true],
|
||||||
'Delivery', $cmd, $item_id, (int)$rr['id']);
|
'Delivery', $cmd, $item_id, (int)$rr['id']);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
|
||||||
|
|
||||||
$push_notify = true;
|
$push_notify = true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// delivery loop
|
||||||
|
if (DBA::isResult($contacts)) {
|
||||||
|
foreach ($contacts as $contact) {
|
||||||
|
// Don't deliver to Diaspora if it already had been done as batch delivery
|
||||||
|
if (($contact['network'] == Protocol::DIASPORA) && $batch_delivery) {
|
||||||
|
Logger::log('Already delivered id ' . $item_id . ' via batch to ' . json_encode($contact), Logger::DEBUG);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Don't deliver to folks who have already been delivered to
|
||||||
|
if (in_array($contact['id'], $conversants)) {
|
||||||
|
Logger::log('Already delivered id ' . $item_id. ' to ' . json_encode($contact), Logger::DEBUG);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
Logger::log('Delivery of item ' . $item_id . ' to ' . json_encode($contact), Logger::DEBUG);
|
||||||
|
Worker::add(['priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true],
|
||||||
|
'Delivery', $cmd, $item_id, (int)$contact['id']);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// send salmon slaps to mentioned remote tags (@foo@example.com) in OStatus posts
|
||||||
|
// They are especially used for notifications to OStatus users that don't follow us.
|
||||||
|
if (!Config::get('system', 'dfrn_only') && count($url_recipients) && ($public_message || $push_notify) && $normal_mode) {
|
||||||
|
$slap = OStatus::salmon($target_item, $owner);
|
||||||
|
foreach ($url_recipients as $url) {
|
||||||
|
if ($url) {
|
||||||
|
Logger::log('Salmon delivery of item ' . $item_id . ' to ' . $url);
|
||||||
|
$deliver_status = Salmon::slapper($owner, $url, $slap);
|
||||||
|
/// @TODO Redeliver/queue these items on failure, though there is no contact record
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
// Notify PuSH subscribers (Used for OStatus distribution of regular posts)
|
// Notify PuSH subscribers (Used for OStatus distribution of regular posts)
|
||||||
if ($push_notify) {
|
if ($push_notify) {
|
||||||
Logger::log('Activating internal PuSH for item '.$item_id, Logger::DEBUG);
|
Logger::log('Activating internal PuSH for item '.$item_id, Logger::DEBUG);
|
||||||
|
@ -500,7 +497,7 @@ class Notifier
|
||||||
PushSubscriber::publishFeed($owner['uid'], $a->queue['priority']);
|
PushSubscriber::publishFeed($owner['uid'], $a->queue['priority']);
|
||||||
}
|
}
|
||||||
|
|
||||||
Logger::log('notifier: calling hooks for ' . $cmd . ' ' . $item_id, Logger::DEBUG);
|
Logger::log('Calling hooks for ' . $cmd . ' ' . $item_id, Logger::DEBUG);
|
||||||
|
|
||||||
if ($normal_mode) {
|
if ($normal_mode) {
|
||||||
Hook::fork($a->queue['priority'], 'notifier_normal', $target_item);
|
Hook::fork($a->queue['priority'], 'notifier_normal', $target_item);
|
||||||
|
|
Loading…
Reference in a new issue