Merge pull request #9261 from annando/relay-distribution
Delayed relay distribution
This commit is contained in:
commit
befc8c86ab
|
@ -67,7 +67,7 @@ class Transmitter
|
|||
* @param array $inboxes
|
||||
* @return array inboxes with added relay servers
|
||||
*/
|
||||
public static function addRelayServerInboxes(array $inboxes)
|
||||
public static function addRelayServerInboxes(array $inboxes = [])
|
||||
{
|
||||
$contacts = DBA::select('apcontact', ['inbox'],
|
||||
["`type` = ? AND `url` IN (SELECT `url` FROM `contact` WHERE `uid` = ? AND `rel` IN (?, ?))",
|
||||
|
|
|
@ -781,6 +781,7 @@ class Notifier
|
|||
}
|
||||
|
||||
$inboxes = [];
|
||||
$relay_inboxes = [];
|
||||
|
||||
$uid = $target_item['contact-uid'] ?: $target_item['uid'];
|
||||
|
||||
|
@ -788,7 +789,7 @@ class Notifier
|
|||
$inboxes = ActivityPub\Transmitter::fetchTargetInboxes($target_item, $uid);
|
||||
|
||||
if (in_array($target_item['private'], [Item::PUBLIC])) {
|
||||
$inboxes = ActivityPub\Transmitter::addRelayServerInboxes($inboxes);
|
||||
$relay_inboxes = ActivityPub\Transmitter::addRelayServerInboxes();
|
||||
}
|
||||
|
||||
Logger::log('Origin item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' will be distributed.', Logger::DEBUG);
|
||||
|
@ -802,10 +803,15 @@ class Notifier
|
|||
// Remote items are transmitted via the personal inboxes.
|
||||
// Doing so ensures that the dedicated receiver will get the message.
|
||||
$inboxes = ActivityPub\Transmitter::fetchTargetInboxes($parent, $uid, true, $target_item['id']);
|
||||
|
||||
if (in_array($target_item['private'], [Item::PUBLIC])) {
|
||||
$relay_inboxes = ActivityPub\Transmitter::addRelayServerInboxes([]);
|
||||
}
|
||||
|
||||
Logger::log('Remote item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . ' will be distributed.', Logger::DEBUG);
|
||||
}
|
||||
|
||||
if (empty($inboxes)) {
|
||||
if (empty($inboxes) && empty($relay_inboxes)) {
|
||||
Logger::log('No inboxes found for item ' . $target_item['id'] . ' with URL ' . $target_item['uri'] . '. It will not be distributed.', Logger::DEBUG);
|
||||
return 0;
|
||||
}
|
||||
|
@ -824,6 +830,15 @@ class Notifier
|
|||
}
|
||||
}
|
||||
|
||||
// We deliver posts to relay servers slightly delayed to priorize the direct delivery
|
||||
foreach ($relay_inboxes as $inbox) {
|
||||
Logger::info('Delivery to relay servers via ActivityPub', ['cmd' => $cmd, 'id' => $target_item['id'], 'inbox' => $inbox]);
|
||||
|
||||
if (Worker::add(['priority' => $priority, 'dont_fork' => true], 'APDelivery', $cmd, $target_item['id'], $inbox, $uid)) {
|
||||
$delivery_queue_count++;
|
||||
}
|
||||
}
|
||||
|
||||
return $delivery_queue_count;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue