Merge pull request #6471 from annando/ap-bcc

Transmit via BCC if contact is hidden, don't store announces
This commit is contained in:
Tobias Diekershoff 2019-01-19 18:23:44 +01:00 committed by GitHub
commit 892a395eb7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 20 additions and 5 deletions

View file

@ -332,7 +332,10 @@ class Receiver
return;
}
self::storeConversation($object_data, $body);
// Only store content related stuff - and no announces, since they possibly overwrite the original content
if (in_array($object_data['object_type'], self::CONTENT_TYPES) && ($type != 'as:Announce')) {
self::storeConversation($object_data, $body);
}
// Internal flag for thread completion. See Processor.php
if (!empty($activity['thread-completion'])) {

View file

@ -307,6 +307,16 @@ class Transmitter
$last_id = $item['id'];
}
$always_bcc = false;
// Check if we should always deliver our stuff via BCC
if (!empty($item['uid'])) {
$profile = Profile::getByUID($item['uid']);
if (!empty($profile)) {
$always_bcc = $profile['hide-friends'];
}
}
// Will be activated in a later step
// $networks = [Protocol::ACTIVITYPUB, Protocol::DFRN, Protocol::DIASPORA, Protocol::OSTATUS];
@ -349,11 +359,13 @@ class Transmitter
}
foreach ($receiver_list as $receiver) {
$contact = DBA::selectFirst('contact', ['url'], ['id' => $receiver, 'network' => $networks]);
$contact = DBA::selectFirst('contact', ['url', 'hidden'], ['id' => $receiver, 'network' => $networks]);
if (DBA::isResult($contact) && !empty($profile = APContact::getByURL($contact['url'], false))) {
// BCC is currently deactivated, due to Pleroma and Mastodon not reacting like expected
// $data['bcc'][] = $profile['url'];
$data['cc'][] = $profile['url'];
if ($contact['hidden'] || $always_bcc) {
$data['bcc'][] = $profile['url'];
} else {
$data['cc'][] = $profile['url'];
}
}
}
}